[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