[FFmpeg-devel] [PATCH 1/2] avidec: get rid of some magic numbers in read_gab2_sub()

[Michael Niedermayer]

On Sun, Feb 06, 2011 at 03:17:57PM +0100, Reimar D?ffinger wrote:
> On Sun, Feb 06, 2011 at 03:12:25PM +0100, Anton Khirnov wrote:
> > On Sun, Feb 06, 2011 at 11:47:54AM +0100, Reimar D?ffinger wrote:
> > > On Sun, Feb 06, 2011 at 11:07:05AM +0100, Anton Khirnov wrote:
[...]
> > > Also in case the AV_RL16 part is some kind of version number I would
> > > consider this significantly worse, magic number or not.
> > TBH I don't know what that number means, this "format" seems like an
> > undocumented hack (and we can hope no new versions of it will ever be
> > created).
> > Anyway, I don't care much about this patch, I can drop it if you think
> > it does more harm than good.
> 
> I don't care either, it just seems a bit pointless to me...

Id like to second this, and IMHO pointless changes should be avoided for
keeping log & blame clean

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

Avoid a single point of failure, be that a person or equipment.
-------------- 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/20110206/86b26b46/attachment.pgp>