[vtk-developers] Removing the vtkstd headers from VTK master - using STL directly

David Gobbi david.gobbi at gmail.com
Tue Dec 13 16:15:47 EST 2011


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.

 - David


On Tue, Dec 13, 2011 at 2:11 PM, Sean McBride <sean at rogue-research.com>wrote:

> On Tue, 13 Dec 2011 11:49:12 -0500, Marcus D. Hanwell said:
>
> >I just pushed a topic to reviewtest that removes vtkstd from VTK. For
> >testing purposes I removed the generation of vtkstd altogether, but
> >that could be modified.
> >
> >http://reviewtest.source.kitware.com:81/#/t/31/
> >
> >This seems to compile for me, and was done largely using a Python
> >script and a little regex. Are there objections to me merging this
> >topic in? Should the vtkstd headers be generated for legacy purposes,
> >I know at least ParaView (which I can fix) is using vtkstd in some
> >places.
>
> "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.
>
> Was the motivation for this change just cleaning up old cruft?
>
> --
> ____________________________________________________________
> Sean McBride, B. Eng                 sean at rogue-research.com
> Rogue Research                        www.rogue-research.com
> Mac Software Developer              Montréal, Québec, Canada
>
>
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
>
> Follow this link to subscribe/unsubscribe:
> http://www.vtk.org/mailman/listinfo/vtk-developers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/vtk-developers/attachments/20111213/9414bf79/attachment.html>


More information about the vtk-developers mailing list