[FFmpeg-devel] [PATCH 2/2] MxPEG decoder

[Michael Niedermayer]

On Mon, Nov 08, 2010 at 05:27:18PM +0300, Anatoly Nenashev wrote:
> On 07.11.2010 16:28, Michael Niedermayer wrote:
>> On Sat, Nov 06, 2010 at 05:10:41AM +0300, Anatoly Nenashev wrote:
>>    [...]
>>> +        /* search for start marker - 0xff */
>>> +        while (mxg->current_pos + FF_INPUT_BUFFER_PADDING_SIZE<  mxg->buffer_size) {
>>> +            uint32_t x;
>>> +            uint8_t *p = mxg->buffer + mxg->current_pos;
>>> +
>>> +            ret = get_partial_buffer(s->pb, p, 4);
>>>      
>> calling this every 4 bytes is slow
>>
>>
>>    
>
> Replaced by 4-times get_byte calls.

i suggest you look at START/STOP_TIMER
they are quite usefull to find out what is fast and what is slow

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Republics decline into democracies and democracies degenerate into
despotisms. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101109/a8a7a956/attachment.pgp>