[Bug 760024] jpegdec: Extra buffers and memcpy() overhead

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=760024

--- Comment #3 from Sachin Kumar Chauhan <sachin.kc at samsung.com> ---
The jpeg_read_raw_data() function needs an array 3 pointers, each further
constituting an array of 16 gpointer's which further point to actual data. By
convention, a local copy of buffers was made to pass to jpeg_read_raw_data(). 

But while optimizing the jpegdec for even dimensional pictures, these memcpy()
kind of overheads were removed from gst_jpeg_dec_decode_direct(). As
gst_jpeg_dec_decode_indirect() was meant to be used for odd dimensional
pictures, it may have been overlooked for optimization at that moment.

As for the point of usage, these array of gpointer's are used only to read
actual data and hence a local copy is not necessary. Also, there is no parallel
decoding going on, so underlying pointers are not meant to change, so local
copy is unnecessary.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.