<div class="gmail_extra">This sounds like a good candidate to live in openstack-common, rather than being limited to Swift.<br><br><div class="gmail_quote">On Sat, Apr 21, 2012 at 12:22 PM, John Dickinson <span dir="ltr"><<a href="mailto:me@not.mn" target="_blank">me@not.mn</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I like what you are trying to do here. Can you please submit this as a patch through gerrit so we can get the rest of the core devs to look at it?<br>

<span class="HOEnZb"><font color="#888888"><br>
--John<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On Apr 20, 2012, at 12:14 PM, Victor Rodionov wrote:<br>
<br>
> There are many place in Swift code where used hard coded values, such<br>
> as response statuses (200, 201, 404, ...) which can replaced with<br>
> constants HTTP_OK, HTTP_CREATED, HTTP_NOT_FOUND. Also there is widlly<br>
> used idiom 200 <= status < 300, that can be replaced as well with<br>
> something like this is_success(status). I want add modules for<br>
> defining all required constants (Swift, HTTP).<br>
><br>
> So I think this changes will improve Swift code readability.<br>
><br>
> PS: this is an initial changes in github<br>
> <a href="https://github.com/vitoordaz/swift/commit/7163d5df13ceaf8fc7b53ba812fe16bd7dd31131" target="_blank">https://github.com/vitoordaz/swift/commit/7163d5df13ceaf8fc7b53ba812fe16bd7dd31131</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">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>
</div></div><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">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></blockquote></div><br></div>