[FFmpeg-devel] [PATCH] Cleaner parity checking code in mlp_parser.c

Ramiro Polla ramiro
Wed Jun 25 01:45:26 CEST 2008


Michael Niedermayer wrote:
> On Tue, Jun 24, 2008 at 08:48:08PM +0100, Ramiro Polla wrote:
>> Hello,
>>
>> In http://article.gmane.org/gmane.comp.video.ffmpeg.devel/59805 Michael 
>> suggested a way to minimize the code that does parity checking on 
>> mlp_parser.c. This leads to the code being harder to understand.
>>
>> The for() loop deals with the substream headers, and the smaller code 
>> parity checked the access unit header in the same loop. It is confusing 
>> since it looks like the loop is only doing the parity check on the 
>> substream headers, and not on the access unit header. It takes closer 
>> inspection to see that the access unit header is also being checked.
>>
>> Is attached patch ok to desimplify the code to make it easier to 
>> understand?
> 
> iam slightly against the patch

Then what about your original idea on that thread? A -1 in there already 
helps a bit in understanding the code is not only dealing with substream 
headers.

Ramiro Polla
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cleaner_parity_2.diff
Type: text/x-diff
Size: 688 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080625/25c58ea4/attachment.diff>



More information about the ffmpeg-devel mailing list