[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-pim
Subject: Re: [Kde-pim] [kde-dev-scripts] /: Make astyle-kdelibs match the Qt/kdelibs coding style closer.
From: Allen Winter <winter () kde ! org>
Date: 2013-11-27 17:43:47
Message-ID: 1702709.zlRCzvCar4 () zazzy
[Download RAW message or body]
On Wednesday, November 27, 2013 11:24:11 AM David Faure wrote:
> On Tuesday 26 November 2013 16:48:06 guy-kde wrote:
> > BUT I still have a problem with the parenthesis/spaces we wish to have
> > at kdepim
> > such as
> > a = b( c, d, e );
>
> I think kdepim decided to adopt the Qt/kdelibs coding style now, which would
> mean b(c, d, e). Please get confirmation from a real kdepim developer though.
>
> > BUT NOT for the SIGNAL and SLOT keywords, and the C-casting mode:
> > a = (int) b;
> > and NOT:
> > a = ( int ) b;
> >
> > Any proposal for that?
>
> Yes, using the same coding style everywhere :-)
>
> I'm happy to invest time to make astyle work out of the box for the kdelibs
> coding style, but I won't spend time making it work for 10 different coding
> styles.
>
I agree with David.
Those small differences are not worth the effort.
Let's keep 1 coding style == the kdelibs coding style
_______________________________________________
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]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic