implemented [TC 11.50b3] Internal 7-Zip packer unavailable in Windows XP

[Usher]

Run TC in Windows XP. Go to Configuration -> Options -> 7-Zip packer tab. Option "Use internal 7-Zip packer" is unchecked and grayed out as unavailable.
Probably the same state is in Windows 2000, but it should be disabled in Windows 9x only.

[Dalai]

Confirmed. However, I very much doubt that the internal 7-Zip will work on Win2k because tcbit7z.dll imports DecodePointer() function from kernel32.dll which is available starting in WinXP.

[Usher]

2Dalai
There were some kernel extensions available for Windows 2000. However, I won't be able to check them in the nearest future.

[ghisler(Author)]

Not a bug, I had to disable it because tcbit7z.dll doesn't work on XP although I compiled it for XP. The problem seems to be that try..catch for C++ exceptions doesn't work on XP, resulting in an access violation in Total Commander. I could not find a reason for why XP behaves differently than newer Windows versions. tcbit7zip uses throw to return errors in 44 places, no chance to rewrite that to other methods in a reasonable time frame.

[Fla$her]

in a reasonable time frame.

Is this feasible before the release of the final version?

[Usher]

Not a bug

You may call it a missing feature and move the topic to Suggestions forum, but for me it's a bug as it breaks original 7zip.dll compatibility.

I could not find a reason for why XP behaves differently than newer Windows versions.

There have been remarks in older topics that XP compatibility options for compiler should be set after every VS update, otherwise you will get errors or Vista compatibility. There have been also explanations about access violation:
viewtopic.php?p=360529&hilit=subsystem#p360529
viewtopic.php?f=6&t=36578&p=349994&hilit=thread%2A#p349994

[ghisler(Author)]

Is this feasible before the release of the final version?

I would have to walk through every single step of what tcbit7z does, and try to figure out how to do it myself. That would take many weeks.

[Horst.Epp]

Is this feasible before the release of the final version?

I would have to walk through every single step of what tcbit7z does, and try to figure out how to do it myself. That would take many weeks.

That's time which shouldn't be spent for a more than outdated OS.

[Fla$her]

The answer was given in another thread:

I hope to find a solution for XP before the final release.

That's time which shouldn't be spent for a more than outdated OS.

All OS will become obsolete sooner or later. When the world is overwhelmed by the boom of mini gadgets with super AI, PCs with software will become a rarity for gourmets.

[JOUBE]

PCs with software will become a rarity for gourmets.

Ah, that's the interest? So it's not really about 7-zip and XP? That's an important information about your real intentions.

It is certainly not a bug, at most a wish, which does not really concern the TC...

[ghisler(Author)]

Moderator message from: ghisler(Author) » 2024-10-28, 08:59 UTC

Moved to suggestions forum

[Fla$her]

Ah, that's the interest? So it's not really about 7-zip and XP? That's an important information about your real intentions.

It's not worth trying your slanderous intentions on someone else's. And since the topic is not about this, why are you wasting your time on an answer that concerned the attitude of a private person to the obsolescence of versions?

It is certainly not a bug, at most a wish, which does not really concern the TC...

This is not a bug, as the author explained the reason. But how it could have occurred to anyone that this does not concern TC remains a big mystery...

[Fla$her]

It didn't take a long wait.

30.10.24 Added: Internal 7-Zip packer in 32-bit version now also works on Windows XP (32)

[ghisler(Author)]

I also managed to make it work in 64-bit now! It was a lot of work because I can't debug (not even remote debug!) on XP with Visual Studio 2017, so I had to put a lot of MessageBox calls in the dll to find the location where it crashes. In the end it was a declaration in the ghc library:
The following line crashes on XP:

Code: Select all

static const impl_string_type seps = GHC_PLATFORM_LITERAL("\\:");

while the following works fine:

Code: Select all

const impl_string_type seps = GHC_PLATFORM_LITERAL("\\:");

[Usher]

2ghisler(Author)
Thanks a lot for your efforts. Have you posted bugfix to the ghc author?