I'll just add... if we do go with compatibility, then the "vtkstd::string" lines in vtkWrapPython.c and vtkParseExtras.c must remain as-is, i.e. no modifications should be done to either file.<div><br></div><div>
- David</div><div><br><div><br><div class="gmail_quote">On Tue, Dec 13, 2011 at 2:11 PM, Sean McBride <span dir="ltr"><<a href="mailto:sean@rogue-research.com">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">
<div class="im">On Tue, 13 Dec 2011 11:49:12 -0500, Marcus D. Hanwell said:<br>
<br>
>I just pushed a topic to reviewtest that removes vtkstd from VTK. For<br>
>testing purposes I removed the generation of vtkstd altogether, but<br>
>that could be modified.<br>
><br>
><a href="http://reviewtest.source.kitware.com:81/#/t/31/" target="_blank">http://reviewtest.source.kitware.com:81/#/t/31/</a><br>
><br>
>This seems to compile for me, and was done largely using a Python<br>
>script and a little regex. Are there objections to me merging this<br>
>topic in? Should the vtkstd headers be generated for legacy purposes,<br>
>I know at least ParaView (which I can fix) is using vtkstd in some<br>
>places.<br>
<br>
</div>"vtkstd::" appears 105 times in our code. Personally, I'm happy to change them all when we upgrade VTK, but it might be helpful, as Bill argues, to have some compatibility route.<br>
<br>
Was the motivation for this change just cleaning up old cruft?<br>
<span class="HOEnZb"><font color="#888888"><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" target="_blank">www.rogue-research.com</a><br>
Mac Software Developer Montréal, Québec, Canada<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
_______________________________________________<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></div></div>