[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openmcl-bug
Subject:    Re: [Bug-openmcl] %active-restart
From:       Alan Ruttenberg <alanr-l () mumble ! net>
Date:       2005-09-08 13:30:12
Message-ID: f793324dde85e58c61e2bb7cd1cadcbb () mumble ! net
[Download RAW message or body]

Good analysis! I concur.

So this is a documentation bug for :test-function.

In fact http://www.lisp.org/HyperSpec/Issues/iss075-writeup.html 
explicitly says:

7. Add a :TEST keyword to RESTART-CASE [...] then functions such as 
FIND-RESTART, COMPUTE-RESTARTS, and INVOKE-RESTART [see #4 above] will 
not consider the restart to be active. The argument to the test 
function is the value of the optional condition argument most of these 
functions accept (or nil for invoke-restart, since it doesn't have that 
argument)...

Clearest thing, IMHO, would have been to define the test function to 
take an *optional* argument, a condition, to parallel the other 
definitions.

Thanks!

-Alan


[prev in list] [next in list] [prev in thread] [next in thread]