<div dir="ltr">I've used it.  It has an unforgivable flaw: it doesn't count the whitespace at the beginning of the line as part of the line length when it does its maximum line length check.<div><br></div><div>My experience with style checkers has been bad.  I find that people use them as a crutch, and write messy code because "the style checker can fix it before I commit".  In any case, my main concern is that people write tidy code, I'm less concerned about strictly following a specific style as long as each module is internally self-consistent.</div><div><br></div><div> - David</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jul 1, 2016 at 9:59 AM, Utkarsh Ayachit <span dir="ltr"><<a href="mailto:utkarsh.ayachit@kitware.com" target="_blank">utkarsh.ayachit@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Maybe a little off topic, but has anyone looked at "Artistic Style" (<a href="http://astyle.sourceforge.net/astyle.html" target="_blank">http://astyle.sourceforge.net/astyle.html</a>).<div><br></div><div>Seems like a project especially designed for changing formatting.</div><span><font color="#888888"><div><br></div></font></span><div><span><font color="#888888">Utkarsh</font></span><div><div><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jul 1, 2016 at 9:49 AM, Bill Lorensen <span dir="ltr"><<a href="mailto:bill.lorensen@gmail.com" target="_blank">bill.lorensen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Once the files are changed, I suggest that a KWStyle expert<br>
<a href="https://kitware.github.io/KWStyle/" rel="noreferrer" target="_blank">https://kitware.github.io/KWStyle/</a> take a stab at creating a VTK style<br>
file. That way we can enforce the style in the future. We use it in<br>
ITK and it is very effective, although it can take some effort to<br>
correct all of the existing defects.<br>
<div><div><br>
<br>
On Fri, Jul 1, 2016 at 9:23 AM, Ben Boeckel <<a href="mailto:ben.boeckel@kitware.com" target="_blank">ben.boeckel@kitware.com</a>> wrote:<br>
> On Thu, Jun 30, 2016 at 17:09:18 -0600, David Gobbi wrote:<br>
>> The scripts just modify the VTK source files. They don't call any git<br>
>> commands so I don't think they can, by themselves, be idempotent.  Do you<br>
>> have any insights into what we can do to avoid conflicts?<br>
><br>
> A function f is idempotent iff f(f(x)) == f(x). If the formatter is<br>
> idempotent, it can be safely run over any branch during a rebase to make<br>
> it not add any style violations.<br>
><br>
> --Ben<br></div></div></blockquote></div></div></div></div></div></div>
</blockquote></div><br></div></div>