2018-01-24 4:04 GMT+01:00 Jun Zhao <mypopy...@gmail.com>: > - procamp_params[i].type = VAProcFilterColorBalance; > - procamp_params[i].attrib = VAProcColorBalanceBrightness; > - procamp_params[i].value = map(ctx->bright, BRIGHTNESS_MIN, > BRIGHTNESS_MAX, > - > procamp_caps[VAProcColorBalanceBrightness-1].range.min_value, > - > procamp_caps[VAProcColorBalanceBrightness-1].range.max_value); > - i++;
> - procamp_params[i].type = VAProcFilterColorBalance; > - procamp_params[i].attrib = VAProcColorBalanceContrast; > - procamp_params[i].value = map(ctx->contrast, CONTRAST_MIN, > CONTRAST_MAX, > - > procamp_caps[VAProcColorBalanceContrast-1].range.min_value, > - > procamp_caps[VAProcColorBalanceContrast-1].range.max_value); > - i++; > - procamp_params[i].type = VAProcFilterColorBalance; > - procamp_params[i].attrib = VAProcColorBalanceHue; > - procamp_params[i].value = map(ctx->hue, HUE_MIN, HUE_MAX, > - > procamp_caps[VAProcColorBalanceHue-1].range.min_value, > - > procamp_caps[VAProcColorBalanceHue-1].range.max_value); > - i++; > - procamp_params[i].type = VAProcFilterColorBalance; > - procamp_params[i].attrib = VAProcColorBalanceSaturation; > - procamp_params[i].value = map(ctx->saturation, SATURATION_MIN, > SATURATION_MAX, > - > procamp_caps[VAProcColorBalanceSaturation-1].range.min_value, > - > procamp_caps[VAProcColorBalanceSaturation-1].range.max_value); > - i++; Please do not reindent these lines in the same patch that removes the conditions, this has two advantages: It makes reviewing your changes easier now, and allows somebody who looks at your change in the future to understand it more quickly. Send an independent patch for the re-indentation. Carl Eugen _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
