<div dir="ltr">A flag is additional complication that we need to maintain pretty much forever. I'd say break backwards compatibility and go with names over attributes. We made the switch to favoring array names over attributes long time ago. This somehow fell through cracks so consider it fixing an old bug.<div>
<br></div><div><div>-berk</div><div><br></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 2, 2014 at 12:18 PM, Cory Quammen <span dir="ltr"><<a href="mailto:cory.quammen@kitware.com" target="_blank">cory.quammen@kitware.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Consider Backwards Compatibility?<br>
<br>
Yes, there is that to consider. The most backwards compatible-friendly<br>
ways to make any changes would be to either<br>
<br>
1) add a flag that determines whether to merge by active attribute<br>
(current behavior) or merge by name only, defaulting to the first<br>
option to retain the current behavior, or<br>
<br>
2) create a separate filter that merges only by name.<br>
<br>
Option 2 seems too bloaty to me. Option 1 keeps the element of<br>
surprising behavior by default, but maintains backwards compatibility,<br>
and appropriate documentation of the merging options should hopefully<br>
make it easy for someone to choose the behavior they might have been<br>
expecting.<br>
<br>
Option 1 sounds the best to me.<br>
<span class="HOEnZb"><font color="#888888"><br>
Cory<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Fri, May 2, 2014 at 12:08 PM, David Thompson<br>
<<a href="mailto:david.thompson@kitware.com">david.thompson@kitware.com</a>> wrote:<br>
>> CBC.<br>
><br>
> Complete Blood Count? Canadian Broadcasting Company?<br>
><br>
> David<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></div>