<div dir="ltr">Hi Folks, <div><br></div><div>Thanks for commenting :) I will update Slicer code base and deprecate the use of vtkStdString.</div><div>It could also make sens to deprecate it in VTK itself. By for example adding a "/deprecated" comment so that doxygen list it.</div>

<div style><br></div><div style>Jc</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Feb 21, 2013 at 10:12 AM, Aashish Chaudhary <span dir="ltr"><<a href="mailto:aashish.chaudhary@kitware.com" target="_blank">aashish.chaudhary@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 class="im">On Thu, Feb 21, 2013 at 9:59 AM, Marcus D. Hanwell<br>
<<a href="mailto:marcus.hanwell@kitware.com">marcus.hanwell@kitware.com</a>> wrote:<br>
> It does exist in some public API, and some functions return it too. We<br>
> also have vtkStringArray - should that be changed to use std::string<br>
> instead of vtkStdString? We also have some code to support<br>
> vtkUnicodeString in public API, although not very much.<br>
><br>
> I am totally in favor of moving to std::string - although I don't know<br>
> if we want to do a mass renaming - are we talking changing<br>
> recommendation here, or trying to go through VTK and entirely remove<br>
> vtkStdString? Berk expressed support for using std::string last year<br>
> along with limited use of other STL types in headers outside of<br>
> Common, although we never found the time to do much.<br>
<br>
</div>I am in favor of removing vtkStdString altogether, unless there is a<br>
technical issue (seems none now), for the sake of clarity and<br>
consistency (time permitting).<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
> On Thu, Feb 21, 2013 at 9:35 AM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>> wrote:<br>
>> I'm not sure if vtkStdString exists in any of the public API.<br>
>><br>
>><br>
>> On Thu, Feb 21, 2013 at 9:24 AM, Aashish Chaudhary<br>
>> <<a href="mailto:aashish.chaudhary@kitware.com">aashish.chaudhary@kitware.com</a>> wrote:<br>
>>><br>
>>> +1 for std::string<br>
>>><br>
>>> On Thu, Feb 21, 2013 at 9:03 AM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>><br>
>>> wrote:<br>
>>> > +1 for std::string as long as it does not break the public API<br>
>>><br>
>>> Bill, can you elaborate  "as long as it does not break the public<br>
>>> API"? For the public API we stick to char*. Is that what you meant<br>
>>> here?<br>
>>><br>
>>> Thanks,<br>
>>><br>
>>><br>
>>> ><br>
>>> ><br>
>>> > On Thu, Feb 21, 2013 at 8:12 AM, Kyle Lutz <<a href="mailto:kyle.lutz@kitware.com">kyle.lutz@kitware.com</a>><br>
>>> > wrote:<br>
>>> >><br>
>>> >> +1 for std::string.<br>
>>> >><br>
>>> >> -kyle<br>
>>> >><br>
>>> >> On Wed, Feb 20, 2013 at 10:42 PM, David Thompson<br>
>>> >> <<a href="mailto:david.thompson@kitware.com">david.thompson@kitware.com</a>> wrote:<br>
>>> >> > Hi JC,<br>
>>> >> ><br>
>>> >> > At least one reason vtkStdString exists is because old versions of<br>
>>> >> > SunCC<br>
>>> >> > could not compile std::map<std::string,std::string> as the template<br>
>>> >> > expansions resulted in class names exceeding a fixed string length in<br>
>>> >> > the<br>
>>> >> > compiler (strings were templated on character type in Sun's STL).<br>
>>> >> > That is no<br>
>>> >> > longer a concern and when we dropped support for old compilers a<br>
>>> >> > while back,<br>
>>> >> > std::string became allowed.<br>
>>> >> ><br>
>>> >> >         David<br>
>>> >> ><br>
>>> >> > On Feb 20, 2013, at 5:56 PM, Jean-Christophe Fillion-Robin<br>
>>> >> > <<a href="mailto:jchris.fillionr@kitware.com">jchris.fillionr@kitware.com</a>> wrote:<br>
>>> >> ><br>
>>> >> >> Hi Folks,<br>
>>> >> >><br>
>>> >> >> What is the advantages of one vs the other? Looking at the current<br>
>>> >> >> code<br>
>>> >> >> base, seems both are widely used.<br>
>>> >> >><br>
>>> >> >><br>
>>> >> >> $ rm -rf Common/Core/vtkStdString.*<br>
>>> >> >> $ ack -c -l -h "std\:\:string" | cut -d":" -f2 | awk '{s+=$1} END<br>
>>> >> >> {print s}'<br>
>>> >> >> 1583<br>
>>> >> >> $ ack -c -l -h "vtkStdString" | cut -d":" -f2 | awk '{s+=$1} END<br>
>>> >> >> {print<br>
>>> >> >> s}'<br>
>>> >> >> 2104<br>
>>> >> >><br>
>>> >> >><br>
>>> >> >> Are the following reason still valid:<br>
>>> >> >>     - allow forward declaration hence reducing the number of header<br>
>>> >> >> inclusion, and also speed compilation.<br>
>>> >> >>     - shorter symbol name. See [1]<br>
>>> >> >><br>
>>> >> >><br>
>>> >> >> As a best practice, should we recommend one vs the other ?<br>
>>> >> >><br>
>>> >> >> Thanks<br>
>>> >> >> Jc<br>
>>> >> >><br>
>>> >> >> [1] <a href="http://www.vtk.org/pipermail/vtkusers/2010-January/105450.html" target="_blank">http://www.vtk.org/pipermail/vtkusers/2010-January/105450.html</a><br>
>>> >> >><br>
>>> >> >> --<br>
>>> >> >> <a href="tel:%2B1%20919%20869%208849" value="+19198698849">+1 919 869 8849</a><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<br>
>>> >> >> <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>
>>> >> ><br>
>>> >> > David Thompson<br>
>>> >> > R&D Engineer – Kitware, Inc.<br>
>>> >> > <a href="mailto:david.thompson@kitware.com">david.thompson@kitware.com</a><br>
>>> >> > <a href="tel:%28919%29869-8868" value="+19198698868">(919)869-8868</a><br>
>>> >> > Suite G-4<br>
>>> >> > 101 E. Weaver St.<br>
>>> >> > Carrboro, NC 27510<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<br>
>>> >> > <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>
>>> >> _______________________________________________<br>
>>> >> Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
>>> >><br>
>>> >> Visit other Kitware open-source projects at<br>
>>> >> <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>
>>> ><br>
>>> ><br>
>>> ><br>
>>> > --<br>
>>> > Unpaid intern in BillsBasement at noware dot com<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<br>
>>> > <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>
>>> ><br>
>>><br>
>>><br>
>>><br>
>>> --<br>
>>> | Aashish Chaudhary<br>
>>> | R&D Engineer<br>
>>> | Kitware Inc.<br>
>>> | <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
>><br>
>><br>
>><br>
>><br>
>> --<br>
>> Unpaid intern in BillsBasement at noware dot com<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<br>
>> <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>
>><br>
<br>
<br>
<br>
--<br>
| Aashish Chaudhary<br>
| R&D Engineer<br>
| Kitware Inc.<br>
| <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><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><br clear="all"><div><br></div>-- <br>+1 919 869 8849<br>
</div>