> Ian's options for astyle, in the order he wrote them today:

> #4: astyle -a -C -S    -m0 -U -t8 -s2 -p    -o -O -k3

This is the only one I'd "recommend" - and even then only slightly...


> My quick reference:
> 
> $ astyle        Ian's options
> -----------------------------------------------------------------
>          -a      --brackets=attach
>          -C      --indent-classes
>          -S      --indent-switches
>          -M      --max-instatement-indent [ =40 ] ( < 80 )
>          -m0     --min-conditional-indent=#
>          -U      --unpad-paren
>          -t8     tab == 8 spaces \ _ ordering is significant (?)
>          -s2     indent 2 spaces /

It seems I have to put the t8 before the s2 or I get the first indent
being 8 spaces...
Maybe that's just me!

>          -p      --pad-oper              ??? [removed]

I like this option. Not sure how it flies with the coding standard, but
I like my operators padded in this style... Personal taste I guess.

>          -x      --delete-empty-lines    ??? [removed]

Definitely remove. I was wrong to put this one in.

>          -o      --keep-one-line-statements
>          -O      --keep-one-line-blocks
>          -k3     --align-pointer=name
> -----------------------------------------------------------------
>                  More options to test
> -----------------------------------------------------------------
>          -T      --indent-force-tab [ =8 ]

Hmm, tried this. Did nto like the result.

>          -l      --brackets=linux  (break 
> functions,classes,namespaces,
>                                     attach otherwise (statements))

Tried this too, think that the -a option was better for our style.

>          -K      --indent-cases

Tried this. Looked odd to me. Personal tatse again.

>          -z#     --lineend=... 1=windows, 2=linux, 3=macold

Astyle claimes to match the file's current lineend if you leave it
alone, so I think we should let it. Then svn will sort the line ending
on the next check in anyway, if all files have the "native" prop set.

> Please note that Cygwin's astyle (1.23) doesn't support -k3, maybe
> that's one of the problems I saw.

I used astyle 1.24 built from sources on mingw. 




SELEX Galileo Ltd
Registered Office: Sigma House, Christopher Martin Road, Basildon, Essex SS14 
3EL
A company registered in England & Wales.  Company no. 02426132
********************************************************************
This email and any attachments are confidential to the intended
recipient and may also be privileged. If you are not the intended
recipient please delete it from your system and notify the sender.
You should not copy it or use it for any purpose nor disclose or
distribute its contents to any other person.
********************************************************************

_______________________________________________
fltk-dev mailing list
[email protected]
http://lists.easysw.com/mailman/listinfo/fltk-dev

Reply via email to