<div dir="ltr">I commented on the gitlab issue, but I'll reiterate here. I'm worried about backwards incompatibilities this might introduce. In most cases, you should be fine. But I have suspicions things may be more complicated than they first appear.<div><br></div><div>With ITK a few years back, there was a seemingly innocent change of a common parameter from 'int' to 'unsigned int'. In principle this made sense because this parameter represented a number of threads, but it caused numerous problems that were hard to foresee.</div><div><br></div><div>We'll need to test ParaView build with this change at the very least...</div><div><br></div><div>Thanks,</div><div>Cory</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 16, 2015 at 11:35 PM, Sean McBride <span dir="ltr"><<a href="mailto:sean@rogue-research.com" target="_blank">sean@rogue-research.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
<br>
I'm working on a patch that changes some VTK APIs that use 'int' to instead use 'bool'.  I've used a bunch of regexes to find vtkBooleanMacro uses, ex:<br>
<br>
  vtkSetMacro\((.*),int\);\r  vtkGetMacro\(\1,int\);\r  vtkBooleanMacro\(\1,int\);<br>
<br>
I've discovered there's a variety of a) whitespace b) ordering, ex:<br>
<br>
  vtkSetMacro(foo, bool);<br>
  vtkGetMacro(foo, bool);<br>
  vtkBooleanMacro(foo, bool);<br>
<br>
vs<br>
<br>
  vtkBooleanMacro(bar,bool);<br>
  vtkSetMacro(bar,bool);<br>
  vtkGetMacro(bar,bool);<br>
<br>
Shall I take this opportunity to make everything uniform?<br>
<br>
The most common form is:<br>
<br>
  vtkSetMacro(foo,int);<br>
  vtkGetMacro(foo,int);<br>
  vtkBooleanMacro(foo,int);<br>
<br>
The 2nd most common is the same order, but with a space after the comma.  The style guide seems to have examples of both.  Is one preferred?<br>
<br>
Here's my WIP:<br>
<<a href="https://gitlab.kitware.com/vtk/vtk/merge_requests/917" rel="noreferrer" target="_blank">https://gitlab.kitware.com/vtk/vtk/merge_requests/917</a>><br>
<br>
Cheers,<br>
<br>
--<br>
____________________________________________________________<br>
Sean McBride, B. Eng                 <a href="mailto:sean@rogue-research.com">sean@rogue-research.com</a><br>
Rogue Research                        <a href="http://www.rogue-research.com" rel="noreferrer" target="_blank">www.rogue-research.com</a><br>
Mac Software Developer              Montréal, Québec, Canada<br>
<br>
<br>
_______________________________________________<br>
Powered by <a href="http://www.kitware.com" rel="noreferrer" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" rel="noreferrer" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" rel="noreferrer" target="_blank">http://markmail.org/search/?q=vtk-developers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/vtk-developers" rel="noreferrer" target="_blank">http://public.kitware.com/mailman/listinfo/vtk-developers</a><br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Cory Quammen<br>R&D Engineer<br>Kitware, Inc.</div>
</div>