[FFmpeg-devel] switch indentation

Justin Ruggles justin.ruggles
Sat Jan 24 19:03:00 CET 2009


M?ns Rullg?rd wrote:
> Justin Ruggles <justin.ruggles at gmail.com> writes:
> 
>> Hi,
>>
>> Quick question. Would reindenting a switch block to make the "case:"
>> line up vertically with the "switch()" be considered a code style change
>> or should it be in a separate commit?
> 
> If all that changes is whitespace, I see no harm in including it with
> other whitespace fixes.  Is that what you were asking?

Specifically, I am making the coding style in flacdec.c consistent. Per
Diego's suggestion, I will do it in a single commit.  But I'm not sure
if reindenting the switch statement really counts as a coding style change.

-Justin





More information about the ffmpeg-devel mailing list