On Fri, Mar 4, 2016 at 2:52 PM, Søren Sandmann <[email protected]> wrote:
> > > On Wed, Feb 10, 2016 at 1:25 AM, <[email protected]> wrote: > >> From: Bill Spitzak <[email protected]> >> >> Rearranged so that the entire block of memory for the filter pair >> is allocated first, and then filled in. Previous version allocated >> and freed two temporary buffers for each filter and did an extra >> memcpy. >> >> v8: small refactor to remove the filter_width function >> >> v10: Restored filter_width function but with arguments changed to >> match later patches >> >> v11: Removed unused arg and pointer from filter_width function >> Whitespace fixes. >> >> Signed-off-by: Bill Spitzak <[email protected]> >> Reviewed-by: Oded Gabbay <[email protected]> >> > > I'm not opposed to this patch if it is correct (and I'll believe the > existing Reviewed-by for correctness), but I'm also don't think there is > much point to this -- these malloc() and free() calls are unlikely to be > performance bottlenecks. > > Acked-by: Soren Sandmann <[email protected]> > This is certainly correct, it would have been obvious in my tests if it did not work.
_______________________________________________ Pixman mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/pixman
