[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: [Kde-pim] Coding style: block braces in switch cases
From:       Sergio Martins <iamsergio () gmail ! com>
Date:       2014-01-13 17:49:17
Message-ID: 1564374.7JguaV95RH () desktop
[Download RAW message or body]

On Monday, January 13, 2014 17:05:04 Kevin Krammer wrote:
> 
> I've looked around and found quite a variation:
> 
> e.g. qdatetime.cpp:
> switch (...) {
> case Foo: {
>     }
> }
> 
> e.g. qrasterizer.cpp:
> switch (...) {
> case Foo:
>     {
>     }
> }



> e.g. qcommandlineparser.cpp:
> switch (...) {
> case Foo:
> {
> }
> }
> 
> e.g. qlocale.cpp:
> switch (...) {
> case Foo: {
> }
> }

Hi,

Personally I think it doesn't matter and we shouldn't enforce such fine grained 
policy, it makes it hard to remember and I don't want to read the guidelines 
whenever I code.

In any case, if something is to be improved in the style guidelines, please 
submit it to Qt first. We took the steps to get rid of the kdepim style to get 
close to the Qt style, so lets not start diverging again.




Regards,
Sérgio Martins

[prev in list] [next in list] [prev in thread] [next in thread]