[Kmymoney-devel] Tweaks to astyle script?

[Thomas Baumgart]

On Tuesday 25 May 2010 14:38:29 Ian Neal wrote:
>   Hi all,
> 
> I've just been looking through mymoneyscheduled.cpp and noticed that our
> current settings for astyle don't make a very good job of styling
> switch/case code statements/blocks.
> I propose that we add --indent-switches and --indent-cases to the script.
> Any objections?
> 
> If there are no objections I'll do a series of pushes to the tree:
> 1) Any changes needed from running current astyle script.
> 2) The change to the astyle script.
> 3) Any changes needed from running the tweaked astyle script (plus some
> tidying up of some switch/case code blocks - e.g. where a case block is
> on the same line as the case statement).

I am not so happy with the current style myself, but I would suggest you 
simply drop an example of the current and the new style here, so that we have 
a clearer picture.

Also, what does each of the options achieve? Links to external references are 
fine.

-- 

Regards

Thomas Baumgart

GPG-FP: E55E D592 F45F 116B 8429   4F99 9C59 DB40 B75D D3BA
-------------------------------------------------------------
There are only 10 types of people in the world: those who
understand binary arithmetic and those who don't.
-------------------------------------------------------------
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 225 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/kmymoney-devel/attachments/20100525/bf617a16/attachment.sig