[PATCH STABLE] templater: catch parsing error for sub

Kevin Bullock kbullock+mercurial at ringworld.org
Mon May 13 13:09:05 CDT 2013


On May 13, 2013, at 12:46 PM, Sean Farley wrote:

> # HG changeset patch
> # User Sean Farley <sean.michael.farley at gmail.com>
> # Date 1368464142 18000
> #      Mon May 13 11:55:42 2013 -0500
> # Branch stable
> # Node ID 1e85222b91951651ea8f8fc3de1af4320a79bd7c
> # Parent  12dbdd348bb0977366200bf96cb6d2afa85faf13
> templater: catch parsing error for sub
> 
> This follows d8d548d868d3, which added template expansion for the sub function,
> to fix the case where the description contained a curly brace (i.e. not part of
> any template). This patch simply wraps the templater call into a try statement.
> 
> Test coverage has been added.

LGTM, but waiting on Matt to ack since he's got the original context on this.

pacem in terris / мир / शान्ति / ‎‫سَلاَم‬ / 平和
Kevin R. Bullock



More information about the Mercurial-devel mailing list