[FFmpeg-devel] [PATCH] Fix inclusion in pixdesc.h of the non-public header intreadwrite.h

Måns Rullgård mans
Tue Feb 16 10:35:28 CET 2010


Michael Niedermayer <michaelni at gmx.at> writes:

> On Mon, Feb 15, 2010 at 01:00:42PM +0000, M?ns Rullg?rd wrote:
>> Michael Niedermayer <michaelni at gmx.at> writes:
>> 
>> > On Sun, Feb 14, 2010 at 11:39:35PM +0100, Stefano Sabatini wrote:
>> >> Hi all,
>> >> 
>> >> intreadwrite.h is not public so we should not include it in a public
>> >> header, this also fixes a bunch of warnings during compilation.
>> >> 
>> >> Also read_line() and write_line() are just meant for
>> >> testing/debugging/pedagogical purposes, so the fact that they're not
>> >> defined inline shouldn't be relevant.
>> >
>> > no, read/write_line() are an essential and important part of pixdescs
>> > They where intended as fallback for very rarely used convertions in swscale
>> > having a specific optimized converter for each pixel format is overkill
>> > especially on CONFIG_SMALL targets
>> 
>> Having them inline in the public header doesn't exactly fit with that
>> description, so I think the patch is fine.  No functionality is being
>> removed.
>
> well, if its important to you then iam ok with moving them to the c file

Regardless of the fate of intreadwrite.h, can we please just apply
this patch?  It fixes an immediate error, and seems to be the right
thing to do either way.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list