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

Panagiotis Issaris takis.issaris
Sat Sep 9 12:34:22 CEST 2006


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.

 libavcodec/opt.c    |   37 +++++++++++++++++++++++++++++++++++++
 libavcodec/opt.h    |    1 +
 libavcodec/utils.c  |    3 +++
 libavformat/utils.c |    4 ++++
 4 files changed, 45 insertions(+)

With friendly regards,
Takis
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pi-20060909T123108-ffmpeg-enable_usage_of_default_avoptions-v3.diff
Type: text/x-patch
Size: 2696 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20060909/e941e863/attachment.bin>



More information about the ffmpeg-devel mailing list