[Ffmpeg-devel] av_log() bloatedness

Michael Niedermayer michaelni
Mon Jul 24 21:42:45 CEST 2006


Hi

On Mon, Jul 24, 2006 at 10:43:47AM -0400, Rich Felker wrote:
> On Mon, Jul 24, 2006 at 03:23:55PM +0200, Michel Bardiaux wrote:
> > Michael Niedermayer wrote:
> > >Hi
> > >
> > >instead of directly using av_log_callback() we have av_log() and av_vlog()
> > >instead of directly setting av_log_level we have av_log_get/set_level()
> > >
> > >why?
> > >is there some deeper reason behind this? or just the CS/OOP/java 
> > >braindamage
> > >if later i will remove it under #if LIBAVUTIL_VERSION_INT < ... so that
> > >it will disaprear the next time we break compatibility
> > >
> > >
> > >
> > Now this *had* to happen while I (as primary author for av_log) was on 
> > vacation and unable to defend my design! Oh well... as we say in French 
> > "Les absents ont toujours tort"...
> > 
> > I guess using shared ffmpeg libs under windows will become even more 
> > difficult, but its hard to say before the change is stabilized.
> 
> Yes, see the other thread. I kinda expect this change will be
> reversed, 

yes, my idea was not good, but iam not sure yet what would be best


> though perhaps via the macro method I suggested.
> 
> BTW could you comment in the other thread on why so many calls to
> av_log lack a context?

i think 20%-30% or so are just av_logs which are commented out ...

and whichever way we will change (or not change) the av_log() api
its certainly a good idea to try to reduce the number of context-less
av_logs (= patch welcome)

[...]
-- 
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