<div dir="ltr">A little late, but here is the patch to put this into hacking.<div><br></div><div><a href="https://review.openstack.org/#/c/64584/">https://review.openstack.org/#/c/64584/</a><br></div><div><br></div><div><br>

</div><div>And here is it running against nova: <a href="http://logs.openstack.org/84/64584/1/check/gate-hacking-integration-nova/b31c47e/console.html">http://logs.openstack.org/84/64584/1/check/gate-hacking-integration-nova/b31c47e/console.html</a></div>

</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Nov 18, 2013 at 5:23 PM, Vishvananda Ishaya <span dir="ltr"><<a href="mailto:vishvananda@gmail.com" target="_blank">vishvananda@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"><br>
On Nov 14, 2013, at 10:00 AM, Monty Taylor <<a href="mailto:mordred@inaugust.com">mordred@inaugust.com</a>> wrote:<br>
<br>
><br>
><br>
> On 11/13/2013 08:08 PM, Robert Collins wrote:<br>
>> On 14 November 2013 13:59, Sean Dague <<a href="mailto:sean@dague.net">sean@dague.net</a>> wrote:<br>
>><br>
>>> This is an area where we actually have consensus in our docs (have had<br>
>>> for a while), the reviewer was being consistent with them, and it feels<br>
>>> like you are reopening that for personal preference.<br>
>><br>
>> Sorry that it feels that way. My personal code also uses ()<br>
>> overwhelmingly - so this isn't a personal agenda issue. I brought it<br>
>> up because the person that wrote the code had chosen to use \, and as<br>
>> far as I knew we didn't have a hard decision either way - and the<br>
>> style guide we have talks preference not requirement, but the review<br>
>> didn't distinguish between whether it's a suggestion or a requirement.<br>
>> I'm seeking clarity so I can review more effectively and so that our<br>
>> code doesn't end up consistent but hard to read.<br>
><br>
> I'd say we've got an expression of clarity here - which means<br>
> potentially a patch to the hacking guide to clarify the language on what<br>
> our choice is, as well as the addition of a hacking check to enforce it<br>
> would be in bounds.<br>
<br>
</div>+1 to sticking something in hacking. FWIW I would probably do the following<br>
to avoid the debate altogether:<br>
<br>
result = self._path_file_exists(ds_browser, folder_path, file_name)<br>
folder_exists, file_exists, file_size_in_kb, disk_extents = result<br>
<br>
Vish<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
><br>
>>> Honestly I find \ at the end of a line ugly as sin, and completely<br>
>>> jarring to read. I actually do like the second one better. I don't care<br>
>>> enough to change a policy on it, but we do already have a policy, so it<br>
>>> seems pretty pedantic, and not useful.<br>
>><br>
>> Ok, thats interesting. Readability matters, and if most folk find that<br>
>> even this case - which is pretty much the one case where I would argue<br>
>> for \ - is still easier to read with (), then thats cool.<br>
>><br>
>>> Bringing up for debate the style guide every time it disagrees with your<br>
>>> personal preference isn't a very effective use of people's time.<br>
>>> Especially on settled matters.<br>
>><br>
>> Totally not what I'm doing. I've been told that much of our style<br>
>> guide was copied lock stock and barrel from some Google Python style<br>
>> guide, so I can't tell what is consensus and what is 'what someone<br>
>> copied down one day'. Particularly when there is no rationale included<br>
>> against the point - its a black box and entirely opaque.<br>
>><br>
>> -Rob<br>
>><br>
><br>
> _______________________________________________<br>
> OpenStack-dev mailing list<br>
> <a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div>