[FFmpeg-devel] [PATCH] Use symbolic names for PAL/NTSC rather than numeric constants in ffmpeg.c:opt_target

Stefano Sabatini stefano.sabatini-lala
Sun Dec 13 18:05:29 CET 2009


On date Sunday 2009-12-13 16:40:39 +0100, Michael Niedermayer encoded:
> On Sun, Dec 13, 2009 at 04:31:51PM +0100, Stefano Sabatini wrote:
> > Hi, as in subject, improve readability.
> > -- 
> > FFmpeg = Fierce & Friendly Mastodontic Programmable Epic Generator
> 
> >  ffmpeg.c |   33 +++++++++++++++++----------------
> >  1 file changed, 17 insertions(+), 16 deletions(-)
> > 65a70f82a8b3a2dc7cfb61d2e7f1d387f93da6ce  ffmpeg-use-symbol-for-pal-ntsc.patch
> > Index: ffmpeg.c
> > ===================================================================
> > --- ffmpeg.c	(revision 20821)
> > +++ ffmpeg.c	(working copy)
> > @@ -3598,26 +3598,27 @@
> >  
> >  static void opt_target(const char *arg)
> >  {
> > +    enum { PAL, NTSC, FILM };
> 
> >      int norm = -1;
> 
> should be enum not int except that this is a nice improvment and ok

Right, so UNKNOWN defined explicitely as well.
-- 
FFmpeg = Faboulous and Fantastic Merciful Programmable Extended Gorilla
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ffmpeg-use-symbol-for-pal-ntsc.patch
Type: text/x-diff
Size: 4166 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091213/6becf0d2/attachment.patch>



More information about the ffmpeg-devel mailing list