On Sun, Jan 29, 2012 at 11:01:01PM +0530, Aneesh Dogra wrote:
> The DEBUG macro has no significance in the code and thus can be removed
> we dont need the comment either. Its of no use unless we actually use that 
> macro
> for good.
> 
> --- a/libavcodec/pngenc.c
> +++ b/libavcodec/pngenc.c
> @@ -29,8 +29,6 @@
>  
>  #include <zlib.h>
>  
> -//#define DEBUG

I like the patch, but I see little point in removing this from just
one file.  Either eliminate it everywhere or don't.  But cluttering
the history with lots of "remove #define DEBUG" is IMO pointless.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to