[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