[FFmpeg-devel] [PATCH V3 1/3] avutil: add ROI data struct and bump version

Derek Buitenhuis derek.buitenhuis at gmail.com
Thu Dec 27 18:42:13 EET 2018


On 27/12/2018 11:05, Guo, Yejun wrote:
>  enum AVActiveFormatDescription {
> @@ -200,6 +206,19 @@ typedef struct AVFrameSideData {
>      AVBufferRef *buf;
>  } AVFrameSideData;
>  
> +typedef struct AVROI {
> +    /* coordinates at frame pixel level.
> +     * It will be extended internally if the codec requires an alignment.
> +     * If the regions overlap, the last value in the list will be used.
> +     */

Should probaly be doxygen above the typedef, and mention the offset, too.

> +    size_t top;
> +    size_t bottom;
> +    size_t left;
> +    size_t right;
> +    // quant offset is encoder dependent
> +    int qoffset;
> +} AVROI;

Nit: Technically it could be a float, but I do't feel strongly about it
one way or another.

Cheers,
- Derek


More information about the ffmpeg-devel mailing list