[FFmpeg-devel] [PATCH] FLAC parser

Michael Chinen mchinen
Wed Dec 1 03:28:59 CET 2010


Hi,

On Tue, Nov 30, 2010 at 1:59 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Wed, Nov 24, 2010 at 11:44:34PM +0100, Michael Chinen wrote:
>> On Tue, Nov 23, 2010 at 10:25 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
>> > On Mon, Nov 22, 2010 at 05:19:23PM -0500, Justin Ruggles wrote:
> [...]
>> >> As for the AV_LOG_MAX patch, I think it's good but Michael N. needs to
>> >> ok it.
>> >
>> > probably a max of 127 should be more than enough
>>
>> Ok, here is the updated patches with comment to flac_read_fifo_wrap,
>> changelog, and minor version bump.
>>
>> Michael
>
> [...]
>> @@ -109,6 +109,12 @@ typedef struct {
>> ?#define AV_LOG_DEBUG ? ?48
>>
>> ?/**
>> + * The maximum AV_LOG value.
>> + * This can be useful as an offset to silence the log messages
>> + **/
>> +#define AV_LOG_MAX ? ? ?255
>
> thats not 127 but rethinking i see no need for this, offseting by appropriate
> values for the code used is better because with several levels of calls there can
> be no globally correct max/min offset

Ok, here are the patches without AV_LOG_MAX.

Michael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-move-decode_frame_header-from-flacdec.c-to-flac.c-h.patch
Type: application/octet-stream
Size: 8119 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101201/4cbf148c/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Add-error-codes-for-FLAC-header-parsing-and-move-log.patch
Type: application/octet-stream
Size: 5864 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101201/4cbf148c/attachment-0001.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-Add-FLAC-Parser.patch
Type: application/octet-stream
Size: 40176 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101201/4cbf148c/attachment-0002.obj>



More information about the ffmpeg-devel mailing list