[Kde-pim] New Maintainer of KMail

[Thomas McGuire]

Hi,

On Tuesday 05 February 2008, Thorsten Staerk wrote:
> > I'll also continue to nag about coding style and incorrect indentation, I
> > hope
> > that this is not too annoying.
>
> No, especially if you just change the policy so it fits to:
>
> QString bar(int i)
> {
>   mycommand(no_capital_letters);
>   if (whatever) oneliner();
>   if (stuff)
>   {
>     firstcommand();
>     nextcommand();
>   }
> }
>
> this is just my opinion of course. Now is the time to do a change!
> innerCapitals are annoying because you never know if command-parts are
> withIn one word or if they are two words.
> { and } on the same indentation increase readAbility.

The coding style will stay the same. Reasons:
- Changing coding style means we would also need to change all code so it fits 
  to the new coding style. This could probably be done with astyle, but I 
  think much of the formatting would be messed up.
- Changing all old code breakes svn annotations
- Not changing code and having inconsistent style is not good
- I've actually used the style so much that I've been brainwashed into  
  liking it

Regards,
Thomas

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 194 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20080206/b8543a51/attachment.sig>
-------------- next part --------------
_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/