<br><br><div class="gmail_quote">On Tue, Aug 2, 2011 at 4:48 PM, Vishvananda Ishaya <span dir="ltr"><<a href="mailto:vishvananda@gmail.com" target="_blank">vishvananda@gmail.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div>
<br>
</div>-1  The original purpose of the test was that people would add strings with multiple positional replacement strings.  This is hard to translate in some cases, because the translators may need control over word order.  We had a series of 10 or so reviews, which was please convert your positional replacement strings to keyword replacement strings.  After a bunch of reviews, someone finally wrote a test to verify it so we didn't have to repeat it a million times.<br>


</blockquote><div><br></div><div>as a data point, I was ignorant of this positional replacement string issue, my code tripped this test, and I learned without a core dev having to explain.  Definitely valuable.</div><div>

 </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
The original naive code was fine for that application.  Your refactor is very useful if you want to do some of the things you mentioned in 1., otherwise the existing test was solving the one issue that we had.<br>
<div><div></div><div><br>
><br>
> Comments welcome.<br>
><br>
> --<br>
> Zed A. Shaw<br>
> <a href="http://zedshaw.com/" target="_blank">http://zedshaw.com/</a><br>
><br>
> _______________________________________________<br>
> Mailing list: <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
> Post to     : <a href="mailto:openstack@lists.launchpad.net" target="_blank">openstack@lists.launchpad.net</a><br>
> Unsubscribe : <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
> More help   : <a href="https://help.launchpad.net/ListHelp" target="_blank">https://help.launchpad.net/ListHelp</a><br>
<br>
<br>
_______________________________________________<br>
Mailing list: <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
Post to     : <a href="mailto:openstack@lists.launchpad.net" target="_blank">openstack@lists.launchpad.net</a><br>
Unsubscribe : <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
More help   : <a href="https://help.launchpad.net/ListHelp" target="_blank">https://help.launchpad.net/ListHelp</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>~~~~~~~~~~~~~~~~~~~~~~~~~~~<br>Dan Wendlandt <br>Nicira Networks, Inc. <br><a href="http://www.nicira.com" target="_blank">www.nicira.com</a> | <a href="http://www.openvswitch.org" target="_blank">www.openvswitch.org</a><br>


Sr. Product Manager <br>cell: <a href="tel:650-906-2650" value="+16509062650" target="_blank">650-906-2650</a><br>~~~~~~~~~~~~~~~~~~~~~~~~~~~<br><br>