[FFmpeg-devel] [PATCH] 4/6 Move APE tag handling to separate module

Michael Niedermayer michaelni
Mon Aug 10 21:54:50 CEST 2009


On Mon, Aug 10, 2009 at 07:33:17PM +0300, Kostya wrote:
> On Mon, Aug 10, 2009 at 12:35:56PM +0300, Matti Hamalainen wrote:
> > 
> > Move APE tag handling to apetag.[ch] and add ff_-prefix to 
> > ape_parse_tag().
> 
> Looks ok, if Michael approves this split too, somebody can apply it.

iam not maintainer of ape*

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Its not that you shouldnt use gotos but rather that you should write
readable code and code with gotos often but not always is less readable
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090810/51e4a025/attachment.pgp>



More information about the ffmpeg-devel mailing list