[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