[vtk-developers] brace style
Simon Warfield
warfield at crl.med.harvard.edu
Thu Sep 10 13:21:11 EDT 2015
The original brace styles were also the best:
http://public.kitware.com/pipermail/vtk-developers/2001-May/015916.html
--Simon
>
> On Thu, Sep 10, 2015 at 11:46 AM, Biddiscombe, John A. <biddisco at cscs.ch>
> wrote:
>
>> Berk,
>>
>> I was not trying to re-propose my suggestion, just emphasising how much I
>> dislike the current style.
>>
>> When I looked at the archives, I found this classic rant from 2001 on the
>> same subject. 14 years I?ve been whining!
>> http://public.kitware.com/pipermail/vtk-developers/2001-May/015906.html
>>
>> Watch out Berk - Now that Kitware has abandoned gerrit, and is changing
>> code style, I might start start cleaning up my CV and looking for job
>> openings ;)
>>
>> JB
>>
>>
>> From: Berk Geveci <berk.geveci at kitware.com>
>> Date: Thursday 10 September 2015 15:24
>> To: cscs <biddisco at cscs.ch>
>> Cc: VTK Developers <vtk-developers at vtk.org>
>> Subject: Re: [vtk-developers] PROPOSAL: Changing VTK's indentation style
>>
>> I am still against your proposal John. For reference, it was this:
>>
>>> I would like to reserve the right to use my own coding style on the
>> contributed class(es)
>>
>> It was not changing VTK's style to something different. Looking at that
>> thread, it also looked like everyone that contributed to it except you were
>> against it. Also looking at this thread, it sounds like many people are
>> still against mixing multiple styles in the same codebase.
>>
>> Best,
>> -berk
>>
>>
>> On Thu, Sep 10, 2015 at 2:49 AM, Biddiscombe, John A. <biddisco at cscs.ch>
>> wrote:
>>
>>> +lots
>>>
>>>
>>>
>>> Like the Scottish who say ?Anyone but England?, I would like to
>>> emphatically vote in favour of changing VTK?s code style to ?Anything, but
>>> what we have now?, it certainly can?t get worse.
>>>
>>>
>>>
>>> Yours
>>>
>>>
>>>
>>> JB
>>>
>>> PS. It feels like 2006! I made my views felt then (and many times
>>> previously), and Berk has criticised my views on it for the last 10 years,
>>> so I shan?t say any more!
>>>
>>>
>>> http://vtk.1045678.n5.nabble.com/vtk-Coding-style-Proposed-change-td1251702.html
>>>
>>>
>>>
>>>
>>>
>>> *From:* vtk-developers [mailto:vtk-developers-bounces at vtk.org] *On
>>> Behalf Of *Berk Geveci
>>> *Sent:* 09 September 2015 15:09
>>> *To:* VTK Developers
>>> *Subject:* [vtk-developers] PROPOSAL: Changing VTK's indentation style
>>>
>>>
>>>
>>> Hi folks,
>>>
>>>
>>>
>>> What would you think of changing VTK's indentation style from:
>>>
>>>
>>>
>>> if ()
>>>
>>> {
>>>
>>> // ..
>>>
>>> }
>>>
>>>
>>>
>>> to
>>>
>>>
>>>
>>> if ()
>>>
>>> {
>>>
>>> // ...
>>>
>>> }
>>>
>>>
>>>
>>> ?
>>>
>>>
>>>
>>> Our current style (modified Whitesmiths I am told) is used only by very
>>> few projects,
>>>
>>> mostly related to VTK ones (ITK, CMake). It is poorly supported by some
>>> of the newer editors
>>>
>>> that are widely popular (Sublime, TextMate etc.) and IMO foreign to the
>>> new generation
>>>
>>> C++ developers.
>>>
>>>
>>>
>>> The style I propose is modified Allman style - with indentation changed
>>> from 4 to 2 spaces.
>>>
>>> It is much more common.
>>>
>>>
>>>
>>> If approved by the community, we would change VTK wholesale. This should
>>> not be
>>>
>>> an issue for history as git blame/annotate can be told to skip white
>>> spaces.
>>>
>>>
>>>
>>> Best,
>>>
>>> -berk
>>>
More information about the vtk-developers
mailing list