On Wed, 24 Jun 2020, Alex Henrie wrote:
The big change here is rewriting the patches to avoid the term "blacklist", which I have replaced with "naughty list".
In this context it's not clear what naughty means. What in a file extension is "badly behaved, disobedient or mildly rude or indecent"? I want for proof that you have to explain what it means in the code:
+ if (on_naughty_list(extensionW, commandW)) + /* command is on naughty list => desktop integration is not desirable */
Also, this being a key name, is "List" even needed? After all the "Classes" key is not called "ClassesList". So I think something like "FileOpenIgnore" would be clearer, although a wording more explicitly related to desktop integration may be even better. "FileOpenInWineOnly", "NotExportedExtensions" or "WineOnlyExtensions"? +HKCU,Software\Wine\FileOpenInWineOnly\.gif,"iexplore",,"""%16422%\Internet Explorer\iexplore.exe"" -nohome" +HKCU,Software\Wine\FileOpenInWineOnly\.jpe,"iexplore",,"""%16422%\Internet Explorer\iexplore.exe"" -nohome" -- Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/ Sufficently advanced incompetence is indistinguishable from malice.