[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