<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Mar 24, 2016 at 8:03 AM, Sean McBride <span dir="ltr"><<a href="mailto:sean@rogue-research.com" target="_blank">sean@rogue-research.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">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>
</span>No opinion per se, but as to your last question: it's impossible for us to know, and polling this list is too small a sample size.  I just grepped our codebase and found 2 uses of InsertValue().<br></blockquote><div><br></div><div>Yes, I prefer to get the opinions of other developers before opening it up to everyone :)</div><div><br></div><div>But since you took the time to answer: for those two uses you found, was the usage of InsertValue() easy to understand, and was it a good match for the intent of the code?</div><div><br></div><div> - David </div><div> </div></div></div></div>