<div dir="ltr">Following discussion with marmoute on IRC, I'll drop the rawrevision() method for now.<div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, 30 Dec 2016 at 11:12 Augie Fackler <<a href="mailto:raf@durin42.com">raf@durin42.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br class="gmail_msg">
> On Dec 30, 2016, at 10:46 AM, Rémi Chaintron <<a href="mailto:remi.chaintron@gmail.com" class="gmail_msg" target="_blank">remi.chaintron@gmail.com</a>> wrote:<br class="gmail_msg">
><br class="gmail_msg">
> The actual rawrevision() method is simply a personal preference as I like to have simpler apis on operations with optional arguments and I've seen this in other places, but I do not feel strongly about moving it to use the raw argument in calls to revision directly. Let me know what you prefer.<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://silkandspinach.net/2004/07/15/avoid-boolean-parameters/" rel="noreferrer" class="gmail_msg" target="_blank">https://silkandspinach.net/2004/07/15/avoid-boolean-parameters/</a><br class="gmail_msg">
<br class="gmail_msg">
In this particular case, my opinion is that it’s a clear win to avoid the boolean parameter since raw=False is actually defined in terms of raw=True, so it makes sense to express the one case in terms of the other using a function call to expose the structure to the reader.<br class="gmail_msg">
<br class="gmail_msg">
Not a requirement, but that’s my reasoning for the choice. :)</blockquote></div><div dir="ltr">-- <br></div><div data-smartmail="gmail_signature"><div dir="ltr">Rémi</div></div>