[FFmpeg-devel] [PATCH 2/2] iff demuxer: include DEEP TVDC lookup table in extradata buffer
pross at xvid.org
Mon Dec 3 12:29:57 CET 2012
On Sat, Dec 01, 2012 at 08:52:16AM +0100, Reimar Döffinger wrote:
> On Sat, Dec 01, 2012 at 05:41:31PM +1100, Peter Ross wrote:
> > + if ((res = avio_read(pb, iff->tvdc, sizeof(iff->tvdc))) < 0)
> I guess I won't be able to convince you, but I can't stop myself
> from say that munging this all into a single line IMHO is really
> bad for readability, the assignment could just as well be in its own
Sounds fair. I will fix that.
> > #include "libavutil/avutil.h"
> > #define LIBAVFORMAT_VERSION_MAJOR 54
> > -#define LIBAVFORMAT_VERSION_MINOR 41
> > -#define LIBAVFORMAT_VERSION_MICRO 100
> > +#define LIBAVFORMAT_VERSION_MINOR 42
> > +#define LIBAVFORMAT_VERSION_MICRO 0
> But my real comment is that the our micro version by policy starts at
> 100, to distinguish from Libav.
If there are no other objections, I will push in 24 hrs.
(A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel