[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