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