[FFmpeg-devel] [PATCH] remove unused and broken test program in libavutil/base64.c

Marc Mason mpeg.blue
Thu Jan 29 10:01:23 CET 2009


Michael Niedermayer wrote:

> Stefano Sabatini wrote:
> 
>>  if (strcmp(encoded, t->encoded) != 0) {
> 
> superflous != 0

Redundant (or superfluous) information sometimes make the code or the 
coder's intention clearer.

>>     printf("\n");
>>     printf("Encoding/decoding tests on random data\n");
> 
> can be merged

For what gain? Cosmetic or computational?
GCC is likely to merge the two statements.

-- 
Regards.




More information about the ffmpeg-devel mailing list