[PATCH v2 4/5] gdiplus: Fix hatch brush patterns that require anti-aliasing.

Jeff Smith whydoubt at gmail.com
Sun Jun 14 23:28:37 CDT 2020


On Sun, Jun 14, 2020 at 9:15 AM Esme Povirk (they/them)
<vincent at codeweavers.com> wrote:
>
> On Sun, Jun 14, 2020 at 2:16 AM Jeff Smith <whydoubt at gmail.com> wrote:
> > > Why not blend them as ARGB?
> >
> > What are you suggesting that is not being done now?
>
> Use blend_colors. It's intended to blend two ARGB colors. If it
> doesn't work correctly, fix it.

Thank you for clarifying.  I just want to make sure that I am
addressing the right concerns.  As you suggest, I will simply use
blend_colors for now, and leave further blending improvements out of
this patchset.  I guess I'll also make the test accept inexact matches
for now.

> If the color values are slightly off from what native produces, but
> there's no clear reason ours are worse, I don't think we should care.

If we have an alternative that matches what native produces, and
there's no clear reason it is worse, perhaps we should.

> Matching the output perfectly is not necessary for application
> compatibility.

True, but I think this is an area where small errors can compound quickly,
so I do think fidelity with native is warranted here if practical.

> I don't see a reason why blending should work differently in this case
> from other cases that use blend_colors.

I agree that it shouldn't in principle, and I hope that holds true in practice.



More information about the wine-devel mailing list