[Ffmpeg-devel] [PATCH] move utf8 writing to libavutil

Justin Ruggles jruggle
Sun Jul 9 03:12:11 CEST 2006


M?ns Rullg?rd wrote:
> Justin Ruggles <jruggle at earthlink.net> writes:
> 
> 
>>+#define PUT_UTF8(val)\
>>+    do {\
>>+    if(val < 0x80){\
> 
> 
> This could do with a little indentation.  Sure, it's not a big deal
> being a macro definition and all.  It's only that bad indentation is
> one of those things I just can't stand...
> 

Yeah, I debated both ways. I figured that if the do{}while(0) was added
to GET_UTF8, indenting the whole block would be against the cosmetic
changes policy...and I wanted the 2 macros to match... If the patch is
okay otherwise, I don't have any strong feelings one way or the other.

-Justin




More information about the ffmpeg-devel mailing list