[FFmpeg-devel] switch indentation
Måns Rullgård
mans
Sat Jan 24 19:17:56 CET 2009
Justin Ruggles <justin.ruggles at gmail.com> writes:
> 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.
I don't know what else it would be.
--
M?ns Rullg?rd
mans at mansr.com
More information about the ffmpeg-devel
mailing list