[FFmpeg-devel] [PATCH] Provide a context to the ffmpeg -r option

[Michael Niedermayer]

On Mon, Aug 11, 2008 at 11:16:46AM +0200, Stefano Sabatini wrote:
> On date Friday 2008-08-08 22:42:19 +0200, Stefano Sabatini encoded:
> > Hi,
> > as in subject, consistent with most options and clearer IMO.
> > 
> > Regards.
> > -- 
> > FFmpeg = Frightening & Forgiving MultiPurpose Erudite Goblin
> 
> > Index: ffmpeg.c
> > ===================================================================
> > --- ffmpeg.c	(revision 14670)
> > +++ ffmpeg.c	(working copy)
> > @@ -2309,12 +2309,13 @@
> >      return 0;
> >  }
> >  
> > -static void opt_frame_rate(const char *arg)
> > +static int opt_frame_rate(const char *opt, const char *arg)
> >  {
> >      if (av_parse_video_frame_rate(&frame_rate, arg) < 0) {
> > -        fprintf(stderr, "Incorrect frame rate\n");
> > +        fprintf(stderr, "Incorrect value for %s: %s\n", opt, arg);
> >          av_exit(1);
> >      }
> > +    return 0;
> >  }
> >  
> >  static int opt_bitrate(const char *opt, const char *arg)
> > @@ -3581,7 +3582,7 @@
> >          opt_format("vcd");
> >  
> >          opt_frame_size(norm ? "352x240" : "352x288");
> > -        opt_frame_rate(frame_rates[norm]);
> > +        opt_frame_rate(NULL, frame_rates[norm]);
> >          opt_default("gop", norm ? "18" : "15");
> >  
> >          opt_default("b", "1150000");
> > @@ -3609,7 +3610,7 @@
> >          opt_format("svcd");
> >  
> >          opt_frame_size(norm ? "480x480" : "480x576");
> > -        opt_frame_rate(frame_rates[norm]);
> > +        opt_frame_rate(NULL, frame_rates[norm]);
> >          opt_default("gop", norm ? "18" : "15");
> >  
> >          opt_default("b", "2040000");
> > @@ -3631,7 +3632,7 @@
> >          opt_format("dvd");
> >  
> >          opt_frame_size(norm ? "720x480" : "720x576");
> > -        opt_frame_rate(frame_rates[norm]);
> > +        opt_frame_rate(NULL, frame_rates[norm]);
> >          opt_default("gop", norm ? "18" : "15");
> >  
> >          opt_default("b", "6000000");
> > @@ -3652,7 +3653,7 @@
> >          opt_frame_size(norm ? "720x480" : "720x576");
> >          opt_frame_pix_fmt(!strncmp(arg, "dv50", 4) ? "yuv422p" :
> >                                               (norm ? "yuv411p" : "yuv420p"));
> > -        opt_frame_rate(frame_rates[norm]);
> > +        opt_frame_rate(NULL, frame_rates[norm]);
> >  
> >          audio_sample_rate = 48000;
> >          audio_channels = 2;
> > @@ -3798,7 +3799,7 @@
> >      { "b", OPT_FUNC2 | HAS_ARG | OPT_VIDEO, {(void*)opt_bitrate}, "set bitrate (in bits/s)", "bitrate" },
> >      { "vb", OPT_FUNC2 | HAS_ARG | OPT_VIDEO, {(void*)opt_bitrate}, "set bitrate (in bits/s)", "bitrate" },
> >      { "vframes", OPT_INT | HAS_ARG | OPT_VIDEO, {(void*)&max_frames[CODEC_TYPE_VIDEO]}, "set the number of video frames to record", "number" },
> > -    { "r", HAS_ARG | OPT_VIDEO, {(void*)opt_frame_rate}, "set frame rate (Hz value, fraction or abbreviation)", "rate" },
> > +    { "r", OPT_FUNC2 | HAS_ARG | OPT_VIDEO, {(void*)opt_frame_rate}, "set frame rate (Hz value, fraction or abbreviation)", "rate" },
> >      { "s", HAS_ARG | OPT_VIDEO, {(void*)opt_frame_size}, "set frame size (WxH or abbreviation)", "size" },
> >      { "aspect", HAS_ARG | OPT_VIDEO, {(void*)opt_frame_aspect_ratio}, "set aspect ratio (4:3, 16:9 or 1.3333, 1.7777)", "aspect" },
> >      { "pix_fmt", HAS_ARG | OPT_EXPERT | OPT_VIDEO, {(void*)opt_frame_pix_fmt}, "set pixel format, 'list' as argument shows all the pixel formats supported", "format" },
> 
> If there are no objections I'm going to apply it on Wednesday, regards.

I object, not strongly but i think this is not good for anything.
Like if it aint broken dont fix it ...

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

Opposition brings concord. Out of discord comes the fairest harmony.
-- Heraclitus
-------------- 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/20080811/b4aa30ba/attachment.pgp>