[FFmpeg-devel] [PATCH] area changed:in cfhd height initialization was buggy for chroma plane

Carl Eugen Hoyos ceffmpeg at gmail.com
Fri Mar 16 16:56:49 EET 2018


2018-03-16 11:27 GMT+01:00, Gagandeep Singh <deepgagan231197 at gmail.com>:
> From: Gagandeep Singh <sunsingh at iitk.ac.in>

Thank you for the important patch!

The first line of the commit message should not start with "area changed"
but something similar to "lavc/cfhd: " which means you can also remove
"cfhd" from the rest of the first line.

> description:when the chroma_y_shift was not present, the FFALIGN used to
> round the height was unnecessary for 0 chroma shift in y direction.

The word description is unnecessary.
Please mention ticket #6675, without it is impossible to ever do a
regression test.

> ---
>  libavcodec/cfhd.c           |   6 +++---

>  libavcodec/tests/codec_desc | Bin 0 -> 189776 bytes

You should not add the file codec_desc to your commit...

Thank you, Carl Eugen


More information about the ffmpeg-devel mailing list