<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jun 29, 2020 at 10:45 PM Alex Henrie <<a href="mailto:alexhenrie24@gmail.com">alexhenrie24@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, Jun 25, 2020 at 3:29 AM Francois Gouget <<a href="mailto:fgouget@free.fr" target="_blank">fgouget@free.fr</a>> wrote:<br>
><br>
> On Wed, 24 Jun 2020, Alex Henrie wrote:<br>
><br>
> > The big change here is rewriting the patches to avoid the term<br>
> > "blacklist", which I have replaced with "naughty list".<br>
><br>
> In this context it's not clear what naughty means. What in a file<br>
> extension is "badly behaved, disobedient or mildly rude or indecent"?<br>
<br>
Several built-in Wine programs are badly behaved in the sense that<br>
they associate themselves with file types that native desktop programs<br>
are better suited to open.<br></blockquote><div><br></div><div>Would it be a problem to remove such integration completely? I can't think of a good scenario when it would be useful.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> I want for proof that you have to explain what it means in the code:<br>
> > +            if (on_naughty_list(extensionW, commandW))<br>
> > +                /* command is on naughty list => desktop integration is not desirable */<br>
><br>
> Also, this being a key name, is "List" even needed? After all the<br>
> "Classes" key is not called "ClassesList".<br>
><br>
> So I think something like "FileOpenIgnore" would be clearer,<br>
> although a wording more explicitly related to desktop integration<br>
> may be even better. "FileOpenInWineOnly", "NotExportedExtensions" or<br>
> "WineOnlyExtensions"?<br>
<br>
The word "extension" is problematic because the naughty list includes<br>
both extensions (which start with a dot) and protocols (which do not).<br>
How about "FileOpenNoIntegration"?<br>
<br>
-Alex<br>
<br>
</blockquote></div></div>