[PATCH] mpatch: unify mpatchError (issue5182)

Pierre-Yves David pierre-yves.david at ens-lyon.org
Mon Apr 4 23:12:07 EDT 2016



On 04/04/2016 07:06 PM, timeless wrote:
> # HG changeset patch
> # User timeless <timeless at mozdev.org>
> # Date 1459389928 0
> #      Thu Mar 31 02:05:28 2016 +0000
> # Node ID 3f972305323b084f81b2566f9612db17cf0d92e3
> # Parent  ff0d3b6b287f89594bd8d0308fe2810d2a18ea01
> mpatch: unify mpatchError (issue5182)
>
> The pure version was mpatch was throwing struct.error or ValueError
> for errors, whereas the C version was throwing an "mpatch.mpatchError".
>
> Introducing an mpatch.mpatchError into pure and using it consistently
> is fairly easy, but the actual form for it is mercurial.mpatch.mpatchError,
> so with this commit, we change the C implementation to match the naming
> convention too.

Thanks a lot for tackling this quickly.

-- 
Pierre-Yves David


More information about the Mercurial-devel mailing list