[FFmpeg-devel] [PATCH] Cleaner parity checking code in mlp_parser.c
Tue Jun 24 22:34:57 CEST 2008
On Tue, Jun 24, 2008 at 08:48:08PM +0100, Ramiro Polla wrote:
> 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
iam slightly against the patch
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Opposition brings concord. Out of discord comes the fairest harmony.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel