[libav-devel] [PATCH 7/8] checkasm: add HEVC MC tests

Luca Barbato lu_zero at gentoo.org
Fri Aug 21 06:31:22 CEST 2015


On 20/08/15 22:37, Henrik Gramner wrote:
>> +        report("%s", "qpel");
>> +        report("%s", "epel");
>> +        report("%s", "unweighted_pred");
>> +        report("%s", "weighted_pred");
> 
> The "%s" is redundant with string literals.

It is good practice do always do that and recent compilers do warn you
if you pass string literals directly to printf-like functions.

Incidentally printf("%s", "string") is faster than printf("string")
since the parser needs to go through 2 chars for those that are versed
to speed optimization ^^.

lu


More information about the libav-devel mailing list