[FFmpeg-devel] Subject: [PATCH] Process compressed id3v2 tags.

Michael Niedermayer michaelni at gmx.at
Sat Dec 3 20:21:59 CET 2011


On Sat, Dec 03, 2011 at 08:14:29PM +0100, Reimar Döffinger wrote:
> On Sat, Dec 03, 2011 at 10:46:46AM -0800, Adrian Drzewiecki wrote:
> > +            char *type;
> 
> "const char *" is more correct.
> 
> > +                for (i = 0, j = 0; i < dlen; i++, j++) {
> > +                    if (tcomp)
> > +                        buffer[j] = avio_r8(s->pb);
> 
> Did you test this to work? Because I'm quite convinced this should be
> "!tcomp"...

I think Adrian said he wanted to be CC-ed

also the patch is a bit messed up in terms of newlines, likely your
MUA wraps lines

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

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111203/5f9326c8/attachment.asc>


More information about the ffmpeg-devel mailing list