Ruediger Pluem wrote:
Hm. I am slightly confused here. The documentation states that "abort" should be the default behaviour (that is the else branch), but if nothing is set in the config onfail defaults to -1 which would cause to get us here (the remove case). So either the documentation or the code is wrong.
Fixed in r731388, which is included in the backport proposal. Thanks for reviewing. -- Nick Kew
