Bug 489815

Summary: Shell Extension loads Qt6Core.dll and other DLLs.
Product: [Applications] kdiff3 Reporter: lesliejlopez-bugzilla
Component: applicationAssignee: michael <reeves.87>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: 1.11.2   
Target Milestone: ---   
Platform: Microsoft Windows   
OS: Microsoft Windows   
Latest Commit: Version Fixed In:
Sentry Crash Report:

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.