[FFmpeg-devel] [PATCH] Re: How can I determine color_range from a filter?

[Michael Niedermayer]

On Mon, Jan 10, 2011 at 10:11:45AM -0800, Larry Robinson wrote:
> On 1/10/2011 6:52 AM, Michael Niedermayer wrote:
>> On Sun, Jan 09, 2011 at 09:20:21PM -0800, Larry Robinson wrote:
>>> On 1/9/2011 2:30 PM, Michael Niedermayer wrote:
>>>> On Sun, Jan 09, 2011 at 08:26:25PM +0100, Stefano Sabatini wrote:
>>>>> On date Sunday 2011-01-09 19:42:27 +0100, Michael Niedermayer encoded:
>>>>>> On Sun, Jan 09, 2011 at 12:11:02AM +0100, Stefano Sabatini wrote:
>>>>>>> On date Friday 2011-01-07 11:12:59 -0800, Larry Robinson encoded:
>> [...]
>>>>> compile/install libavfilter without installing libavcodec (configure
>>>>> --enable-libavfilter --disable-libavcodec),
>>>> no problem here
>>>>
>>>>
>>>>> in a distro you could have
>>>>> libavfilter-dev and yet not libavcodec-dev.
>>>> it wouldnt work, we dont have 4 independant configure scripts, and
>>>> quadruplicating the build system across 5 split -dev packages seems like a
>>>> bad idea
>>>> and if you meant the distro simply physically removd libavcodec.
>>>> We should nozt support this castration and if someone still wanted he can
>>>> leave the header there as well if he already manually cuts things randomly
>>>> away
>>>>
>> [...]
>>> #ifndef AVCORE_COLORSPACE_H
>>> #define AVCORE_COLORSPACE_H
>> The enums are not just colorspace related some enums are related to the
>> positions of samples.
>> if at all this could be moved into teh main public header of avcore
>>
>> [...]
>>
>>
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at mplayerhq.hu
>> https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel
> Alrightie then...
> In the interest of progress, please allow me to sidestep this question  
> (and the opt.h question) as follows:
>
> 1) For now, I will include avcodec.h in avfilter.h; if the color  
> rendering enums are moved out, only this line would need to be changed.
>
> 2) I will encode color_primaries, etc. into the `args' string currently  
> being passed to the vsrc_buffer init routine, which will now expect 10  
> colon-separated arguments rather than 5.  This affects only the `buffer'  
> filter.
>

> I think there is agreement to place the 5 enums into the AVFilterLink  
> structure.

I would try to place them in AVFilterBufferRefVideoProps, if it works its
more flexible

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Frequently ignored awnser#1 FFmpeg bugs should be sent to our bugtracker. User
questions about the command line tools should be sent to the ffmpeg-user ML.
And questions about how to use libav* should be sent to the libav-user ML.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20110110/92e8fb02/attachment.pgp>