[Vv] vv and clitk3 "coding standard"

[Joël Schaerer]

On 04/27/2010 10:32 AM, David Sarrut wrote:
>
> astyle ? ok can we do what has been proposed at once ?

Yes, if everyone agrees. Note that astyle cannot do any of these:


>>- all member functions start with an upper case (like vtk and itk, but
unlike QT !) : SetToto(a) and GetToto()
- all member variables start with m, eg : mToto or mTiti (unlike vtk,
unlike qt, unlike itk)
- delimitate the function (in cxx and txx) with comment like
//------------ (see for example clitkBinarizeImageGenericFilter.txx)


>
>
> On Tue, Apr 27, 2010 at 10:30, Joël Schaerer <joel.schaerer at gmail.com
> <mailto:joel.schaerer at gmail.com>> wrote:
>
>     I don't really have a preference, but I suggest that we change all the
>     sources at once with astyle once we converge on a standard. Of course
>     using the default astyle layout would be the simplest :)
>
>     On a completely unrelated topic, if you use vv a lot from the command
>     line, I suggest adding the following line to your .bashrc:
>
>     complete -A file -X '*.raw' vv
>
>     It makes dealing with mhd files a lot more pleasant.
>
>     joel
>
>     On 04/27/2010 09:36 AM, Simon Rit wrote:
>     > After working a bit on clitk3, I have the feeling that there is
>     > absolutely no consistency between (and even within) files. I don't
>     > really think we should adopt a coding standard because we should keep
>     > our own coding styles but it would be nice if we could just set a
>     > minimal set of rules for the sake of readability. I suggest these
>     ones
>     > first:
>     > - indentation: 2 spaces
>     > - tabs are replaced by spaces
>     > - opening and closing characters of a block are aligned
>     vertically, e.g.
>     > if (true)
>     > {
>     > }
>     > - types are suffixed with Type (as in ITK if I'm not mistaking)
>     > - preprocessor defines and macros in upper case
>     >
>     > For the rest (e.g. naming of functions and variables), we can try to
>     > follow the rules of our third-party libraries (itk and vtk). What do
>     > you think?
>     > Simon
>     > _______________________________________________
>     > vv mailing list
>     > vv at creatis.insa-lyon.fr <mailto:vv at creatis.insa-lyon.fr>
>     > http://www.creatis.insa-lyon.fr/mailman/listinfo/vv
>     >
>
>
>     --
>     Joël Schaerer, PhD
>     Research Engineer
>
>     Centre de lutte contre le cancer Léon Bérard
>     Service de radiothérapie
>     28 rue Laënnec
>     69373 LYON CEDEX 08
>
>     Tel: 04 78 78 51 50
>         06 26 65 29 54
>
>     http://www.creatis.insa-lyon.fr/rio
>     http://www.creatis.insa-lyon.fr/rio/vv
>
>     _______________________________________________
>     vv mailing list
>     vv at creatis.insa-lyon.fr <mailto:vv at creatis.insa-lyon.fr>
>     http://www.creatis.insa-lyon.fr/mailman/listinfo/vv
>
>
>
>
> -- 
> David Sarrut, Phd
> Chargé de recherche CNRS
> Centre de lutte contre le cancer Léon Bérard
> 28 rue Laënnec, 69373 Lyon cedex 08
> Laboratoire CREATIS-LRMN UMR CNRS 5220, Inserm U 630
> Tel : 04 78 78 51 51 / 06 74 72 05 42
> http://www.creatis.insa-lyon.fr/rio
> ______________________________________________
> "2 + 2 = 5,  for extremely large values of 2"
> ______________________________________________
>
>
> _______________________________________________
> vv mailing list
> vv at creatis.insa-lyon.fr
> http://www.creatis.insa-lyon.fr/mailman/listinfo/vv
>   


-- 
Joël Schaerer, PhD
Research Engineer

Centre de lutte contre le cancer Léon Bérard
Service de radiothérapie
28 rue Laënnec
69373 LYON CEDEX 08

Tel: 04 78 78 51 50
     06 26 65 29 54

http://www.creatis.insa-lyon.fr/rio
http://www.creatis.insa-lyon.fr/rio/vv

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.creatis.insa-lyon.fr/pipermail/vv/attachments/20100427/bba83080/attachment.html>