[prev in list] [next in list] [prev in thread] [next in thread] 

List:       vtk-developers
Subject:    Re: [vtk-developers] Popular class name: vtkArrayMap
From:       "Timothy M. Shead" <tshead () sandia ! gov>
Date:       2009-09-17 15:27:20
Message-ID: 4AB25558.7010809 () sandia ! gov
[Download RAW message or body]

Berk Geveci wrote:
> I would leave the one in ParaView alone since KWCommon is (possibly)
> used by other projects. If you wanted to get rid of both, we should
> consider getting rid of the use of KWCommon in ParaView.
> 
> -berk
> 
> On Wed, Sep 16, 2009 at 12:07 PM, Jeff Baumes <jeff.baumes@kitware.com> wrote:
>> PV ArrayMap : Made April 2002, in several PV releases, used by
>> vtkPVGeometryFilter, has companion class vtkArrayMapIterator whose
>> name would also need to be changed.
>>
>> VTK ArrayMap : Made April 2008, in VTK 5.2 and VTK 5.4, used by
>> vtkGraphMapper (which is deprecated).
>>
>> Yes, I'd say the PV one wins, and we should change the name of the VTK one.
>>
>> Jeff
>>
>> On Wed, Sep 16, 2009 at 11:50 AM, Timothy M. Shead <tshead@sandia.gov> wrote:
>>> Gang:
>>>
>>> We have two classes named vtkArrayMap, one in VTK/Rendering, and another in
>>> ParaView/Common/KWCommon.  I'm inclined to rename the former to something
>>> along the lines of "vtkMapArrayValues", any thoughts?
>>>
>>> Cheers,
>>> Tim

I left the ParaView class alone and converted the VTK vtkArrayMap to 
vtkMapArrayValues ... it's in CVS now.

Cheers,
Tim

_______________________________________________
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

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic