On Wed, 29 May 2013 17:06:14 +0100 Joe Orton <[email protected]> wrote:
> On Wed, May 29, 2013 at 11:37:14AM -0400, Matthew Steele wrote: > > Oops, yes, RUN_ALL semantics are desired; the misleading API > > description is my fault, sorry. (I confess I never really > > understood why RUN_ALL hooks accept both OK and DECLINED values, > > but then don't actually treat them any differently.) So probably > > we should update the doc comments. I'd be happy to draft new > > versions for those if that would be helpful, or not, as you prefer. > > (Really attached this time) This all looks great, thanks Joe! Pull my comments along with my the old backport proposal, because I'm certainly ready to support this change in 2.2 and 2.4. Addressed all of my concerns.
