Bug 489815 - Shell Extension loads Qt6Core.dll and other DLLs.
Summary: Shell Extension loads Qt6Core.dll and other DLLs.
Status: RESOLVED FIXED
Alias: None
Product: kdiff3
Classification: Applications
Component: application (show other bugs)
Version: 1.11.2
Platform: Microsoft Windows Microsoft Windows
: NOR normal
Target Milestone: ---
Assignee: michael
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-07-06 06:19 UTC by lesliejlopez-bugzilla
Modified: 2024-09-23 01:22 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description lesliejlopez-bugzilla 2024-07-06 06:19:31 UTC
The KDiff3 shell extension (kdiff3ext.dll) loads Qt6Core.dll and potentially other DLLs unnecessarily when executing system applications like Manage Computer. This behavior is undesirable as it increases resource consumption and can lead to performance degradation.

If Qt6Core.dll and other potentially loaded DLLs are indeed unnecessary for the core functionality of the KDiff3 shell extension, please remove these dependencies entirely. This will result in a leaner, more performant extension that minimizes resource consumption and avoids potential conflicts.

Please also let users disable the shell extension via a Kdiff3 settings option.
Comment 1 michael 2024-07-06 18:47:04 UTC
To change this would require support from kde's automated translation string exaction scripts for an alternate translated string function  such as gettext. The presence of translated strings and to need to be compatible for kd3 automation is the primary reason for qt being used at all in the hell extension. I don't want to hand code po file and translation support as was originally done.
Comment 2 michael 2024-07-06 18:55:49 UTC
Disabling the extension may something I can do. Because craft starts with a generic template implement additional options is something I may not have a way to do easily. The current release also includes a version built to be side loaded via the windows store. This version does not have the shell extension as they are not allowed for such apps. Looks I forgot to upload it this go around I can correct that. These builds are part of the CI.
Comment 3 michael 2024-09-23 01:14:36 UTC
The exention is now optional if it causes an issue.