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
