Sometime, VTK needs to bite the bullet and use KWStyle to produce a consistent code base. We did it for ITK and to be honest, it was not all that difficult. I did most of the updates and I estimate it took about 1 person week during a northeast winter blast.<br>
<br>Once it is done, it is easy to maintain using git hooks.<br><br>Bill<br><br><br><div class="gmail_quote">On Tue, Apr 5, 2011 at 4:21 PM, Marcus D. Hanwell <span dir="ltr"><<a href="mailto:marcus.hanwell@kitware.com">marcus.hanwell@kitware.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div class="im">On Tue, Apr 5, 2011 at 2:33 PM, David Cole <<a href="mailto:david.cole@kitware.com">david.cole@kitware.com</a>> wrote:<br>

> On Tue, Apr 5, 2011 at 2:26 PM, Brian Helba <<a href="mailto:brian.helba@kitware.com">brian.helba@kitware.com</a>> wrote:<br>
>><br>
>> There won't be much drudgery, I already have a bash/sed script to do the<br>
>> necessary edits.<br>
><br>
> But the edits are unnecessary if the script is fixed....<br>
<br>
</div>It seems to be the dominant style in the VTK headers, it would be<br>
great to enforce a more consistent coding style. I thought the newline<br>
before each description was part of that. I agree that we should also<br>
make the Perl script more robust.<br>
<div class="im">>><br>
>> It would be nice to get a quick fix in, so the nightly HTML documentation<br>
>> is more useful.<br>
><br>
> It has been this way for a long, long time. It would be better to get the<br>
> right fix (rather than quick) in and avoid the unnecessary edits.<br>
><br>
> But maybe that's just me....<br>
><br>
> Feel free to resolve this however you think is best.<br>
><br>
</div>As we will potentially have more students coming starting to hack VTK<br>
code over the summer, it would be great to establish what we expect<br>
from them, as well as other contributors. The Perl conversion can be<br>
especially difficult to understand as it is very VTK specific, and so<br>
I think it would be valuable to document these constraints it places<br>
on our coding style, along with the constraints the automated wrapping<br>
also places on our code.<br>
<br>
Just my $0.02 on this - I would rather fix both, but don't have the<br>
time to do either.<br>
<font color="#888888"><br>
Marcus<br>
</font><div><div></div><div class="h5">_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
<br>
</div></div></blockquote></div><br>