[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