<div dir="ltr">Ahh..windows.  May be they will fix this once Steve leaves Microsoft -:). This is very good information. Thanks<div><br></div><div style>- Aashish</div><div style><br></div></div><div class="gmail_extra"><br>
<br><div class="gmail_quote">On Thu, Aug 29, 2013 at 11:44 AM, Robert Maynard <span dir="ltr"><<a href="mailto:robert.maynard@kitware.com" target="_blank">robert.maynard@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
RemoveProp is the one that windows.h defines.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<br>
On Thu, Aug 29, 2013 at 11:42 AM, David Gobbi <<a href="mailto:david.gobbi@gmail.com">david.gobbi@gmail.com</a>> wrote:<br>
> I have a vague memory that either AddProp or RemoveProp was defined as<br>
> a macro in some windows header file, making the name unusable.  I<br>
> don't know if that is still the case with Visual Studio 2003 and later<br>
> compilers.<br>
><br>
>  David<br>
><br>
> On Thu, Aug 29, 2013 at 9:16 AM, Aashish Chaudhary<br>
> <<a href="mailto:aashish.chaudhary@kitware.com">aashish.chaudhary@kitware.com</a>> wrote:<br>
>> +1 for AddViewProp all others in legacy. Would have been nice to have it<br>
>> called AddProp (why add View?) .  We have vtkProp and not vtkViewProp as the<br>
>> class name. May be we can add that as the new method? thoughts?<br>
>><br>
>> - Aashish<br>
>><br>
>><br>
>><br>
>> On Thu, Aug 29, 2013 at 11:08 AM, David E DeMarle <<a href="mailto:dave.demarle@kitware.com">dave.demarle@kitware.com</a>><br>
>> wrote:<br>
>>><br>
>>> I'm in Sean's camp. One class to rule them all is easier to get your head<br>
>>> around.<br>
>>><br>
>>><br>
>>> David E DeMarle<br>
>>> Kitware, Inc.<br>
>>> R&D Engineer<br>
>>> 21 Corporate Drive<br>
>>> Clifton Park, NY 12065-8662<br>
>>> Phone: <a href="tel:518-881-4909" value="+15188814909">518-881-4909</a><br>
>>><br>
>>><br>
>>> On Wed, Aug 28, 2013 at 7:25 PM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>><br>
>>> wrote:<br>
>>>><br>
>>>> I think from a user point of view, not being concerned about inheritance,<br>
>>>> the old names make sense.<br>
>>>><br>
>>>><br>
>>>><br>
>>>> On Wed, Aug 28, 2013 at 4:59 PM, Sean McBride <<a href="mailto:sean@rogue-research.com">sean@rogue-research.com</a>><br>
>>>> wrote:<br>
>>>>><br>
>>>>> On Wed, 28 Aug 2013 16:37:05 -0400, Bill Lorensen said:<br>
>>>>><br>
>>>>> >I think AddActor(), AddActor2D(), and AddVolume() are more descriptive<br>
>>>>> > than<br>
>>>>> >AddViewProp even though they are just wrappers.<br>
>>>>><br>
>>>>> Be that as it may, I think it's pretty weird for someone starting with<br>
>>>>> VTK 6.<br>
>>>>><br>
>>>>> Class hierarchy reminder:<br>
>>>>><br>
>>>>> vtkObject -> vtkProp -> vtkProp3D -> vtkActor<br>
>>>>> vtkObject -> vtkProp -> vtkProp3D -> vtkVolume<br>
>>>>> vtkObject -> vtkProp -> vtkActor2D<br>
>>>>><br>
>>>>> The newer AddViewProp() takes a vtkProp*, which makes sense, as it works<br>
>>>>> with all 3 types.<br>
>>>>><br>
>>>>> The older methods AddActor(), AddActor2D(), and AddVolume() strangely<br>
>>>>> all also take a vtkProp*, that's pretty weird given their name.  ex: you'd<br>
>>>>> think AddVolume() would take a vtkVolume*, no?  Why should it accept any<br>
>>>>> vtkProp subclass?<br>
>>>>><br>
>>>>> It's also weird that AddActor() and AddVolume() come from vtkRenderer,<br>
>>>>> while AddActor2D() comes from the superclass vtkViewPort.<br>
>>>>><br>
>>>>> How to explain all this to someone new to VTK 6?<br>
>>>>><br>
>>>>> It seems like cruft to me.  IMNSHO, using only AddViewProp() seems<br>
>>>>> preferable, which is why I'd like to do the cleanup and eventually wrap the<br>
>>>>> old three in VTK_LEGACY.<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" target="_blank">www.rogue-research.com</a><br>
>>>>> Mac Software Developer              Montréal, Québec, Canada<br>
</div></div><div class="HOEnZb"><div class="h5">> _______________________________________________<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>| Aashish Chaudhary <br>| R&D Engineer         <br>| Kitware Inc.            <br>| <a href="http://www.kitware.com">www.kitware.com</a>    
</div>