Re: [FFmpeg-devel] [PATCH 2/7] Removed unecessary if/else branch when getting QP.

2015-03-27 Thread Michael Niedermayer
On Sat, Mar 28, 2015 at 12:24:29AM +, Timothy Gu wrote: > For all patches in this series, please use the following format as commit > message: > > postproc: blah added the prefix and applied as its easier than repostng and reviewing again if theres no other reason to redo things for futu

Re: [FFmpeg-devel] [PATCH 2/7] Removed unecessary if/else branch when getting QP.

2015-03-27 Thread Timothy Gu
For all patches in this series, please use the following format as commit message: postproc: blah Remember, FFmpeg has a giant code base and it is often useful to see what part is being modified in the patch in the first word of the title. See https://www.ffmpeg.org/developer.html#Developmen

[FFmpeg-devel] [PATCH 2/7] Removed unecessary if/else branch when getting QP.

2015-03-27 Thread Tucker DiNapoli
From: Tucker DiNapoli There's still an if, as QP needs to be modified if isColor=0, but it still removes a unecessary branch. --- libpostproc/postprocess_template.c | 12 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/libpostproc/postprocess_template.c b/libpostproc/