[Pixman] [PATCH 2/2] pixman: Use maximum precision for pixman-bits-image, v2.

Maarten Lankhorst maarten.lankhorst at linux.intel.com
Mon Feb 11 12:40:20 UTC 2019


Op 09-02-2019 om 13:01 schreef Basile Clement:
> On 1/8/19 11:42 AM, Maarten Lankhorst wrote:
>
>> pixman-bits-image's wide helpers first obtains the 8-bits image,
>> then converts it to float. This destroys all the precision that
>> the wide path was offering.
>>
>> Fix this by making get_pixel() take a pointer instead of returning
>> a value. Floating point will fill in a argb_t, while the 8-bits path
>> will fill a 32-bits ARGB value. This also requires writing a floating
>> point bilinear interpolator. With this change pixman can use the full
>> floating point precision internally in all paths.
>>
>> Changes since v1:
>> - Make accum and reduce an argument to convolution functions,
>>   to remove duplication.
>>
>> Signed-off-by: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
> Apologies for the late reply; I have been quite busy with other things
> recently.
>
> The patch looks good to me now!  I have not figured out how to visually
> test the filters however.

Thanks, pushed.

Can you rebase the dithering patches so I can take a look?

~Maarten



More information about the Pixman mailing list