[FFmpeg-devel] [PATCH 2] add tag/comment support to the raw flac demuxer

Reimar Döffinger Reimar.Doeffinger
Wed Dec 10 10:08:15 CET 2008


On Tue, Dec 09, 2008 at 01:38:33PM -0800, Jim Radford wrote:
> On Thu, 4 Dec 2008 at 21:44:51, Reimar D?ffinger wrote:
> > On Mon, Dec 01, 2008 at 02:18:02PM -0800, Jim Radford wrote:
> > > This patch adds support for parsing vorbis comments in plain flac
> > > streams.  Only metadata packets are parsed leaving the frame data to
> > > be parsed in raw 1024 byte chunks like before.
> 
> > > +        init_get_bits(&gb, header, ret*8);
> > > +        flac->metadata_done = get_bits(&gb, 1);
> > > +        type = get_bits(&gb, 7);
> > > +        length = get_bits(&gb, 24);
> 
> > Seems very much like overkill to use get_bits here. Actually
> > get_be32 to me seems much more appropriate than get_buffer to get
> > the header anyway.
> 
> I thought so too, but given that I need to copy the data back into the
> stream where it needs to be in its original endian, so the code
> doesn't get any simpler.

Nonsense, AV_WB32 is simpler and possibly a lot faster than memcpy
(depends on whether the compiler inlines his own memcpy variant).




More information about the ffmpeg-devel mailing list