On Thu, 08 Dec 2011 16:08:27 +0100, Diego Biurrun <[email protected]> wrote: > On Thu, Dec 08, 2011 at 03:03:15PM +0000, Måns Rullgård wrote: > > Diego Biurrun <[email protected]> writes: > > > --- > > > ffmpeg.c | 2 +- > > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > > > --- a/ffmpeg.c > > > +++ b/ffmpeg.c > > > @@ -502,7 +502,7 @@ void exit_program(int ret) > > > > > > - exit(ret); /* not all OS-es handle main() return value */ > > > + exit(ret); > > > } > > > > 15:02 <@elenril> srsly, removing _comments_ from the code clearly marked as > > deprecated and not to be developed > > Removing the comment took substantially less time than the previous > discussion about return vs. exit() that led to it, so it was clearly > worth the trouble for me. >
ffmpeg.c is deprecated and not developed anymore. It should not be touched for any other reason than keeping it compatible with cmdutils changes and possibly fixing critical bugs. And since it'll be removed soon anyway (I'm thinking shortly after the release/before the bump), there's no point to this patch. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
