[Ffmpeg-devel] [PATCH/RFC] change (f)printf calls in libpostproc to av_log

[Diego Biurrun]

On Tue, Oct 10, 2006 at 02:39:42PM +0200, Michael Niedermayer wrote:
> 
> 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

OK, applied.

Diego