bytereader: Use pointer instead of index access
authorSungho Bae <baver.bae@lge.com>
Tue, 10 Jun 2014 13:35:38 +0000 (09:35 -0400)
committerNicolas Dufresne <nicolas.dufresne@collabora.co.uk>
Tue, 10 Jun 2014 13:35:38 +0000 (09:35 -0400)
commitd3b2f6e4b85a45213ed2878f411f7c66ebb15e33
treed98385edcbddfdfaf11c09304258bfa067a1c7c4
parent12088d87bba3cb88212ac87b0ccef1608fd52c00
bytereader: Use pointer instead of index access

Currently the scan uses Boyer-moore method and its performance is good.
but, it can be optimized from an implementation of view.

The original scan code is implemented by byte array and index-based access.
In _scan_for_start_code(), the index is increasing from start to end and the
base address of the byte array is referred to as return value.

In the case, index-based access can be replaced by pointer access, which
improve the performance by removing index-related operations.

Its performace is enhanced by approximately 8% on arm-based embedded devices.
Although it seems trivial, it can affect the overall performance because the
_scan_for_start_code() function is very often called when H.264/H.265 video is
played.

In addition, the technique can apply for all architectures and it is good in
view of readability and maintainability.

https://bugzilla.gnome.org/show_bug.cgi?id=731442
libs/gst/base/gstbytereader.c