[vtk-developers] PROPOSAL: Changing VTK's indentation style

[Aashish Chaudhary]

+1

On Wed, Sep 9, 2015 at 9:20 AM, David Gobbi <david.gobbi at gmail.com> wrote:

> +1
>
>
> On Wed, Sep 9, 2015 at 7:14 AM, David E DeMarle <dave.demarle at kitware.com>
> wrote:
>
>> +1
>>
>>
>>
>> David E DeMarle
>> Kitware, Inc.
>> R&D Engineer
>> 21 Corporate Drive
>> Clifton Park, NY 12065-8662
>> Phone: 518-881-4909
>>
>> On Wed, Sep 9, 2015 at 9:08 AM, Berk Geveci <berk.geveci at kitware.com>
>> wrote:
>>
>>> 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
>>>
>>
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
>
> Search the list archives at: http://markmail.org/search/?q=vtk-developers
>
> Follow this link to subscribe/unsubscribe:
> http://public.kitware.com/mailman/listinfo/vtk-developers
>
>
>


-- 



*| Aashish Chaudhary | Technical Leader         | Kitware Inc.            *
*| http://www.kitware.com/company/team/chaudhary.html
<http://www.kitware.com/company/team/chaudhary.html>*
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/vtk-developers/attachments/20150909/fd597822/attachment.html>