<div><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 7 Oct 2020 at 18:35, Jeff Smith <<a href="mailto:whydoubt@gmail.com">whydoubt@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, Oct 7, 2020 at 2:24 AM Huw Davies <<a href="mailto:huw@codeweavers.com" target="_blank">huw@codeweavers.com</a>> wrote:<br>
><br>
> On Tue, Oct 06, 2020 at 11:22:29PM -0500, Jeff Smith wrote:<br>
> > Signed-off-by: Jeff Smith <<a href="mailto:whydoubt@gmail.com" target="_blank">whydoubt@gmail.com</a>><br>
> > ---<br>
> > Testing revealed that SetBkMode() does not treat 'invalid' values as one<br>
> > might assume.  Discovered this when working on another issue.  Note that<br>
> > fixing this to match Windows simplifies one of the tests for that bug.<br>
><br>
> Which then begs the question: How should these invalid modes be<br>
> treated visually?<br>
<br>
FWIW, testing In Windows points to the default being transparent.<br>
That implies that any wine code that checks this value should check<br>
for equality/non-equality with OPAQUE to be "safe".<br>
</blockquote><div dir="auto"><br></div><div dir="auto">What ends up in metafile record for invalid modes?</div><div dir="auto"><br></div><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
> Since there doesn't appear to be a real application that<br>
> depends on this, we should keep things as they are.<br>
<br>
Would be nice (and removes 8 unnecessary lines of code), but no<br>
real harm in leaving it as it is.<br>
<br>
Thanks,<br>
Jeff<br>
<br>
</blockquote></div></div>