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

List:       kde-pim
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 gra=
ined =

policy, it makes it hard to remember and I don't want to read the guideline=
s =

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=E9rgio Martins

_______________________________________________
KDE PIM mailing list kde-pim@kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/
[prev in list] [next in list] [prev in thread] [next in thread]