<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Jun 10, 2015 at 10:29 AM, Augie Fackler <span dir="ltr"><<a href="mailto:raf@durin42.com" target="_blank">raf@durin42.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, Jun 10, 2015 at 09:50:31AM -0700, Gregory Szorc wrote:<br>
> # HG changeset patch<br>
> # User Gregory Szorc <<a href="mailto:gregory.szorc@gmail.com">gregory.szorc@gmail.com</a>><br>
> # Date 1433955009 25200<br>
> #      Wed Jun 10 09:50:09 2015 -0700<br>
> # Branch stable<br>
> # Node ID 1b4b923551c0a37c129b1d1c530448685e5ca2a2<br>
> # Parent  7298da81f5a9f64ebbdef2b2195585a65da0f99e<br>
> parsers: raise an unambiguous type during index lookup (issue4451)<br>
<br>
</span>The patch as stated looks fine, however, I do have one proposal below<br>
that might be of interest.<br>
<br>
[snip]<br>
<span class=""><br>
><br>
> diff --git a/mercurial/parsers.c b/mercurial/parsers.c<br>
> --- a/mercurial/parsers.c<br>
> +++ b/mercurial/parsers.c<br>
> @@ -1482,45 +1482,8 @@ static int index_find_node(indexObject *<br>
>               return rev;<br>
>       return -2;<br>
>  }<br>
><br>
> -static PyObject *raise_revlog_error(void)<br>
> -{<br>
> -     static PyObject *errclass;<br>
<br>
</span>We could just dispense with static and load the error every time. This<br>
doesn't look like a path we should hit super-frequently in normal<br>
operations.</blockquote><div><br></div><div>Perhaps. But as I mentioned in the commit message, the actual type doesn't really matter. Raising RevlogError from C feels like extra, unjustified complexity to me. But I'll restore it if that's what is wanted.<br></div></div></div></div>