<div dir="ltr">Hello,<br><br>As a user, I believe that kind of change makes perfect sense, these functions are just confusing.<br>Simon<br></div><div class="gmail_extra"><br><div class="gmail_quote">2016-03-24 16:53 GMT+01:00 Sean McBride <span dir="ltr"><<a href="mailto:sean@rogue-research.com" target="_blank">sean@rogue-research.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thu, 24 Mar 2016 08:11:40 -0600, David Gobbi said:<br>
<br>
>On Thu, Mar 24, 2016 at 8:03 AM, Sean McBride <<a href="mailto:sean@rogue-research.com">sean@rogue-research.com</a>><br>
>wrote:<br>
><br>
>> On Wed, 23 Mar 2016 22:09:55 -0600, David Gobbi said:<br>
>><br>
>> >Also related to data arrays: I'd love to see the deprecation of<br>
>> >InsertValue() and InsertTuple().  The problems with them are:<br>
>> ><br>
>> >1) They are redundant.  As far as I'm concerned, their usage can<br>
>> >always be replaced with SetTuple() or InsertNextTuple() and the<br>
>> >resulting code will become easier to read.<br>
>> ><br>
>> >2) They don't actually "insert" tuples except at the end of the<br>
>> >array, and we have InsertNextTuple() or that purpose.  Anywhere<br>
>> >else in the array, they perform a replacement operation.<br>
>> ><br>
>> >Any thoughts?  Are these methods widely used anywhere except<br>
>> >for VTK's own source tree?<br>
>><br>
>> No opinion per se, but as to your last question: it's impossible for us to<br>
>> know, and polling this list is too small a sample size.  I just grepped our<br>
>> codebase and found 2 uses of InsertValue().<br>
>><br>
><br>
>Yes, I prefer to get the opinions of other developers before opening it up<br>
>to everyone :)<br>
><br>
>But since you took the time to answer: for those two uses you found, was<br>
>the usage of InsertValue() easy to understand, and was it a good match for<br>
>the intent of the code?<br>
<br>
</span>In my 2 cases, it's always appending to the end of the data array.  I guess I could use InsertNextValue() instead, if I understand it correctly.<br>
<br>
I think your point #2 is quite valid.  Since it's not *inserting*, but rather *replacing*, then the API name is very confusing.  A deprecate and rename is in order IMHO.<br>
<div class="HOEnZb"><div class="h5"><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" rel="noreferrer" target="_blank">www.rogue-research.com</a><br>
Mac Software Developer              Montréal, Québec, Canada<br>
<br>
<br>
_______________________________________________<br>
Powered by <a href="http://www.kitware.com" rel="noreferrer" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" rel="noreferrer" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" rel="noreferrer" target="_blank">http://markmail.org/search/?q=vtk-developers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/vtk-developers" rel="noreferrer" target="_blank">http://public.kitware.com/mailman/listinfo/vtk-developers</a><br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div>------------------------------------------------------------------<br>Simon Esneault<div>Rennes, France<br>------------------------------------------------------------------</div></div></div></div>
</div>