[Vv] vv and clitk3 "coding standard"

Simon Rit simon.rit at gmail.com
Tue Apr 27 10:38:52 CEST 2010


What is astyle?

On Tue, Apr 27, 2010 at 10:37 AM, Joël Schaerer <joel.schaerer at gmail.com> wrote:
> 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>
> 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
>> > 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
>> 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
>
> _______________________________________________
> vv mailing list
> vv at creatis.insa-lyon.fr
> http://www.creatis.insa-lyon.fr/mailman/listinfo/vv
>
>



More information about the vv mailing list