[FFmpeg-devel] [PATCH] Some lavf renames

Stefano Sabatini stefano.sabatini-lala
Tue Jan 20 23:40:37 CET 2009


On date Tuesday 2009-01-20 12:09:20 -0800, Baptiste Coudurier encoded:
> Hi Ramiro,
> 
> Ramiro Polla wrote:
> > On Tue, Jan 20, 2009 at 6:00 PM, Baptiste Coudurier
> > <baptiste.coudurier at gmail.com> wrote:
> >> Hi Stefano,
> >>
> >> Stefano Sabatini wrote:
> >>> On date Friday 2009-01-09 01:44:00 +0100, Diego Biurrun encoded:
> >>>> On Thu, Jan 08, 2009 at 10:59:40PM +0100, Stefano Sabatini wrote:
> >>>>> attached patches propose the renames:
> >>>>> av_alloc_format_context -> avformat_alloc_context (more similar to avcodec_alloc_context)
> >>>>>
> >>>>> av_register_all -> avformat_register_all (avcodec_register_all, avfilter_register_all, avdevice_register_all)
> >>>> FWIW, I'm in favor..
> >>> Ping.
> >> I'm in favor too, also some attention could be paid to "guess_format" :>
> > 
> > Wasn't there some discussion some time ago about renaming a bunch of
> > functions? I think if we're going to rename some functions, we might
> > as well rename all functions to be consistent and break API only once.
> 
> Yes, indeed. We will break API during next major dump, this is not going
> to happen anytime soon.

I propose also these renames, people can add to the list:

lavc:
register_avcodec          -> avcodec_register()

lavf:
ff_reduce_index           -> av_reduce_index
av_register_input_format  -> avformat_register_input
av_register_output_format -> avformat_register_output
av_iformat_next           -> avformat_next_input
av_oformat_next           -> avformat_next_output

Check the attached patches for the first ones.

And yes we can avoid A[PB]I break with the usual
deprecation/change/ifversioning trick.

Regards.
-- 
FFmpeg = Fundamental Fierce Multimedia Prodigious Embarassing Gospel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rename-av-alloc-format-context.patch
Type: text/x-diff
Size: 1482 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090120/cf2aeff8/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rename-av-register-all.patch
Type: text/x-diff
Size: 1257 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090120/cf2aeff8/attachment-0001.patch>



More information about the ffmpeg-devel mailing list