[FFmpeg-devel] [PATCH 1/2] lavu/frame: add new side data type for ICC profiles
atomnuker at gmail.com
Tue Jul 25 22:18:26 EEST 2017
On 22 July 2017 at 21:17, Rostislav Pehlivanov <atomnuker at gmail.com> wrote:
> On 21 July 2017 at 14:36, Nicolas George <george at nsup.org> wrote:
>> Le tridi 3 thermidor, an CCXXV, Derek Buitenhuis a écrit :
>> > +1 on exposing it as side data; it's consistent with previous APIs and
>> I will not oppose further; I do not buy the "Yet Another Field" argument
>> but I will debunk it another time.
>> > FWIW, literally every other library exposes ICC data the same way (as a
>> > buffer), and it's IMO the most reasonable. It's not reasonable to put
>> an actual
>> > description of ICC data in the doxy (see: 300 page spec... it is
>> > Just a reference to the spec is fine.
>> I fully agree. I suggest something along the lines:
>> The data contains an ICC profile as an opaque octet buffer
>> following the format described at $url with an optional name
>> defined in the metadata entry "name".
> Thanks for the suggestion, changed the description to what you suggested.
> I'll push both patches tomorrow unless someone finds something else wrong.
Thanks for your reviews, pushed.
Replaced $url with the ISO spec the format follows.
More information about the ffmpeg-devel