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

[Tim Thirion]

+1

I still find the current brace style hard to read after a year of
development.

TT

On Wed, Sep 9, 2015 at 2:17 PM, Dan Lipsa <dan.lipsa at kitware.com> wrote:

> +1
> I found the VTK formatting very foreign when I first start using VTK - now
> I got used with it.
>
> For everything else I used the format you propose.
> Dan
>
>
> 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
>>
>>
>>
>
> _______________________________________________
> 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
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/vtk-developers/attachments/20150909/2d35dc12/attachment-0001.html>