<div dir="ltr">LGTM.<div><br></div><div style>-eric</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jan 25, 2013 at 11:42 AM, Eli Bendersky <span dir="ltr"><<a href="mailto:eliben@google.com" target="_blank">eliben@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Here's a full patch that addresses your comments, adds flags for all<br>
sections and some testing.<br>
<span class="HOEnZb"><font color="#888888"><br>
Eli<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On Fri, Jan 25, 2013 at 8:48 AM, Eli Bendersky <<a href="mailto:eliben@google.com">eliben@google.com</a>> wrote:<br>
> On Fri, Jan 25, 2013 at 8:44 AM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br>
>><br>
>><br>
>>> There were two reasons for this change:<br>
>>><br>
>>> 1. General coding conventions consistency.<br>
>>> 2. The first couple of times I saw that variable I had a slight wtf<br>
>>> moment reading it "dict x" and getting confused. (yeah, too much<br>
>>> Python, I know).<br>
>>><br>
>>> However, I don't see it as a big deal, so if you feel strongly about<br>
>>> keeping the old name, I'll remove it from the patch. No worries.<br>
>>><br>
>><br>
>> Not particularly against it, if you want to just go ahead and commit it now<br>
>> and not part of the other patch.<br>
>><br>
><br>
> OK, I'll do that. Thanks.<br>
><br>
> Eli<br>
</div></div></blockquote></div><br></div>