[Ffmpeg-devel] [PATCH/RFC] change (f)printf calls in libpostproc to av_log
Michael Niedermayer
michaelni
Tue Oct 10 14:39:42 CEST 2006
Hi
On Tue, Oct 10, 2006 at 02:19:33PM +0200, Diego Biurrun wrote:
> On Tue, Oct 10, 2006 at 08:29:31AM +0200, Michael Niedermayer wrote:
> >
> > On Tue, Oct 10, 2006 at 01:12:10AM +0200, Diego Biurrun wrote:
> > > On Sun, Oct 08, 2006 at 11:36:00PM +0200, Michael Niedermayer wrote:
> > > >
> > > > On Sun, Oct 08, 2006 at 01:58:13PM +0200, Diego Biurrun wrote:
> > > > > This changes all printf/fprintf occurrences in libpostproc to av_log. I
> > > > > used NULL as context, if there is a better alternative, let me know.
> > > >
> > > > yes use the PPContext (add a AVClass as first field) like AVCodecContexts
> > >
> > > More or less like the attached patch? Thanks to Oded for some help.
> >
> > yes, looks good
>
> Do I need to update the version number of libpostproc for this change?
> Since I add a member to the PPContext struct, the API has changed,
> right? IIUC I need to bump the minor version, i.e. the second number..
no, PPContext is never directly accessed by the user
(its even in postproc_internal.h), so no version change is needed
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is
More information about the ffmpeg-devel
mailing list