[FFmpeg-devel] [PATCH 1/2] lavu/frame: add new side data type for ICC profiles
george at nsup.org
Fri Jul 21 16:36:30 EEST 2017
Le tridi 3 thermidor, an CCXXV, Derek Buitenhuis a écrit :
> +1 on exposing it as side data; it's consistent with previous APIs and doesn't
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 dumb
> 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 complictated).
> 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".
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 833 bytes
Desc: Digital signature
More information about the ffmpeg-devel