[Ffmpeg-devel] [PATCH 1/2] Enable usage of AVOption default value

Michael Niedermayer michaelni
Sat Sep 9 23:56:28 CEST 2006


Hi

On Sat, Sep 09, 2006 at 12:34:22PM +0200, Panagiotis Issaris wrote:
> Hi,
> 
> On za, 2006-09-09 at 10:29 +0200, Panagiotis Issaris wrote:
> > On za, 2006-09-09 at 00:21 +0200, Aurelien Jacobs wrote:
> > > [...]
> > > Any reason why you implemented this as a macro instead of a function ?
> > Ah! You are absolutely right! In fact, I _had_ a reason during the
> > writing of the patch... but it is gone now.
> > [...]
> > After this, I finally got how flags in AVOption worked, and rewrote
> > the flag handling of the patch, so in fact the reason for writing it as
> > a macro was gone... I will turn it into a function ASAP and resend a
> > patch.
> 
> Attached to this e-mail you'll find the updated patch. Regression tests
> passed fine on my system.

looks ok

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