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

Berk Geveci berk.geveci at kitware.com
Thu Sep 10 15:51:51 EDT 2015


OK. Let's go with:

if ()
{
  // ...
}

as it will have minimal impact on our ability to use git blame/annotate
easily. We'll make this happen on our end. We'll give warning a while
before doing it because it will create conflicts when merging branches that
predate the change.

-berk

On Thu, Sep 10, 2015 at 1:36 PM, Bill Lorensen <bill.lorensen at gmail.com>
wrote:

> Can we please put this to bed and concentrate on the 58 gitlab merge
> requests?
>
> Bill
>
>
>
> On Thu, Sep 10, 2015 at 1:24 PM, Ben Boeckel <ben.boeckel at kitware.com>
> wrote:
> > On Thu, Sep 10, 2015 at 13:14:40 -0400, Marcus D. Hanwell wrote:
> >> Once we are done with this...C++11, Doxygen comments, and typed
> enums?!?!
> >
> > Oh dear. If anyone wants to take a whack, here's (some of) my VTK wish
> > list:
> >
> >   - CMake 3-based module system;
> >   - replace "T *foo = T::New(); /* ... */ foo->Delete();" patterns with
> >     "vtkNew<T> foo;";
> >   - increase use of std::algorithm (making a vtk*Iterator -> STL
> >     Iterator adaptor would help *wonders* here);
> >   - make file readers more robust (using the afl[1] fuzzer as a guide).
> >
> > --Ben
> >
> > [1]http://lcamtuf.coredump.cx/afl/
>
>
>
> --
> Unpaid intern in BillsBasement at noware dot com
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/vtk-developers/attachments/20150910/8da9db7e/attachment.html>


More information about the vtk-developers mailing list