[9.20b4] CtrlArrowOpenArchive=4 tries to analyze archive

Bug reports will be moved here when the described bug has been fixed

Moderators: white, Hacker, petermad, Stefan2

Post Reply
User avatar
MVV
Power Member
Power Member
Posts: 8702
Joined: 2008-08-03, 12:51 UTC
Location: Russian Federation

[9.20b4] CtrlArrowOpenArchive=4 tries to analyze archive

Post by *MVV »

When CtrlArrowOpenArchive=4 is set (never open archives on Ctrl+Arrow, always focus), TC still tries to find a packer plugin for an archive on Ctrl+Arrow.
It is easy to notice with Total7zip plugin: create an empty file.7z and use Ctrl+Arrow on it, and you will see an error message. Some plugins cause hangs (e.g. lsop sometimes).
User avatar
Horst.Epp
Power Member
Power Member
Posts: 6449
Joined: 2003-02-06, 17:36 UTC
Location: Germany

Post by *Horst.Epp »

Confirmed
Windows 11 Home x64 Version 23H2 (OS Build 22631.3374)
TC 11.03 x64 / x86
Everything 1.5.0.1371a (x64), Everything Toolbar 1.3.2, Listary Pro 6.3.0.69
QAP 11.6.3.2 x64
User avatar
ghisler(Author)
Site Admin
Site Admin
Posts: 48021
Joined: 2003-02-04, 09:46 UTC
Location: Switzerland
Contact:

Post by *ghisler(Author) »

This should be fixed in beta 5, please test it!
Author of Total Commander
https://www.ghisler.com
User avatar
MVV
Power Member
Power Member
Posts: 8702
Joined: 2008-08-03, 12:51 UTC
Location: Russian Federation

Post by *MVV »

I still do a Ctrl+Arrow on empty file and get a error message "Error in packed file D:\Stuff\1.7z!"...
User avatar
ghisler(Author)
Site Admin
Site Admin
Posts: 48021
Joined: 2003-02-04, 09:46 UTC
Location: Switzerland
Contact:

Post by *ghisler(Author) »

NOT confirmed. Which 7zip plugin do you use?
Author of Total Commander
https://www.ghisler.com
User avatar
MVV
Power Member
Power Member
Posts: 8702
Joined: 2008-08-03, 12:51 UTC
Location: Russian Federation

Post by *MVV »

I use Total7zip 0.8.5.6. I've tried with clean wincmd.ini (only associated 7z extension with plugin and added CtrlArrowOpenArchive=4 to INI) and the message still appears.

Detailed test with clean installation:
1. Extract TC installer to some dir.
2. Copy Total7zip.wcx to test TC dir (just this one file).
3. Create wincmd.ini:

Code: Select all

[Configuration]
UseIniInProgramDir=7
CtrlArrowOpenArchive=4
[PackerPlugins]
7z=%COMMANDER_PATH%\Total7zip.wcx
4. Start TOTALCMD.exe, close first-time windows.
5. Focus empty .7z file and press Ctrl+Arrow.
6. Total7zip error appears:
Total7zip
---------------------------
7-zip library is not found.
Download 7-Zip
http://www.7-zip.org/
---------------------------
ОК
7. Click OK, another error appears:
Total Commander
---------------------------
Error in packed file d:\Stuff\1.7z!
---------------------------
ОК
User avatar
Horst.Epp
Power Member
Power Member
Posts: 6449
Joined: 2003-02-06, 17:36 UTC
Location: Germany

Post by *Horst.Epp »

MVV wrote:I still do a Ctrl+Arrow on empty file and get a error message "Error in packed file D:\Stuff\1.7z!"...
Confirmed
Still the same error using Total7zip 0.8.5.6
I only get

Total Commander
---------------------------
Error in packed file c:\temp\test\x.7z!
---------------------------
ОК
Windows 11 Home x64 Version 23H2 (OS Build 22631.3374)
TC 11.03 x64 / x86
Everything 1.5.0.1371a (x64), Everything Toolbar 1.3.2, Listary Pro 6.3.0.69
QAP 11.6.3.2 x64
User avatar
ghisler(Author)
Site Admin
Site Admin
Posts: 48021
Joined: 2003-02-04, 09:46 UTC
Location: Switzerland
Contact:

Post by *ghisler(Author) »

I cannot reproduce it for Ctrl+Left arrow or Ctrl+Right arrow.

Maybe you used Ctrl+Up arrow or Ctrl+Shift+Up arrow? Then I can reproduce it.
Author of Total Commander
https://www.ghisler.com
User avatar
Horst.Epp
Power Member
Power Member
Posts: 6449
Joined: 2003-02-06, 17:36 UTC
Location: Germany

Post by *Horst.Epp »

ghisler(Author) wrote:I cannot reproduce it for Ctrl+Left arrow or Ctrl+Right arrow.

Maybe you used Ctrl+Up arrow or Ctrl+Shift+Up arrow? Then I can reproduce it.
No, I tested with Ctrl+Left and Ctrl+Right arrow and get the error.
If I use Ctrl-Up arrow an new tab with the name of the archive is opened.
If I enter this tab the error message is shown.
Windows 11 Home x64 Version 23H2 (OS Build 22631.3374)
TC 11.03 x64 / x86
Everything 1.5.0.1371a (x64), Everything Toolbar 1.3.2, Listary Pro 6.3.0.69
QAP 11.6.3.2 x64
User avatar
ghisler(Author)
Site Admin
Site Admin
Posts: 48021
Joined: 2003-02-04, 09:46 UTC
Location: Switzerland
Contact:

Post by *ghisler(Author) »

Sorry, cannot reproduce with CtrlArrowOpenArchive=4. Are you sure you have that in your wincmd.ini?

Edit:
I found one special case where I still get the error: When the file has a size of 0 bytes. Is this the case in your test?
Author of Total Commander
https://www.ghisler.com
User avatar
Horst.Epp
Power Member
Power Member
Posts: 6449
Joined: 2003-02-06, 17:36 UTC
Location: Germany

Post by *Horst.Epp »

ghisler(Author) wrote:Sorry, cannot reproduce with CtrlArrowOpenArchive=4. Are you sure you have that in your wincmd.ini?

Edit:
I found one special case where I still get the error: When the file has a size of 0 bytes. Is this the case in your test?
Yes of course, MVV started this thread with such a file.
Windows 11 Home x64 Version 23H2 (OS Build 22631.3374)
TC 11.03 x64 / x86
Everything 1.5.0.1371a (x64), Everything Toolbar 1.3.2, Listary Pro 6.3.0.69
QAP 11.6.3.2 x64
User avatar
ghisler(Author)
Site Admin
Site Admin
Posts: 48021
Joined: 2003-02-04, 09:46 UTC
Location: Switzerland
Contact:

Post by *ghisler(Author) »

MVV didn't write anything about a 0 bytes file. When I tested this error, I simply renamed a small text file from name.txt to name.7z.
Author of Total Commander
https://www.ghisler.com
User avatar
MVV
Power Member
Power Member
Posts: 8702
Joined: 2008-08-03, 12:51 UTC
Location: Russian Federation

Post by *MVV »

Well, I call "0-byte" file as "empty file", so yes, this is the case. :)
Why this case is a special one?
User avatar
ghisler(Author)
Site Admin
Site Admin
Posts: 48021
Joined: 2003-02-04, 09:46 UTC
Location: Switzerland
Contact:

Post by *ghisler(Author) »

TC checked parameters by content for being archives if the size was >0 bytes, because 0 byte files couldn't contain anything to check by content. The 0 byte files were passed directly to the directory read function, where a separate check was made.
Author of Total Commander
https://www.ghisler.com
User avatar
MVV
Power Member
Power Member
Posts: 8702
Joined: 2008-08-03, 12:51 UTC
Location: Russian Federation

Post by *MVV »

Fixed in 9.20b6, thank you!
Post Reply