Re: change in call-next-method

[Eric Abrahamsen <eric@ericabrahamsen.net>]

Stephen Leake <stephen_leake@stephe-leake.org> writes:

> Andreas Schwab <schwab@linux-m68k.org> writes:
>
>> Stephen Leake <stephen_leake@stephe-leake.org> writes:
>>
>>> I'm updating JDEE to Emacs 25. It has several functions like this:
>>>
>>> (cl-defmethod initialize-instance ((this jdee-jddocset) &rest rest)
>>>   (apply 'call-next-method rest)
>>>   (unless (oref this description)
>>>     (oset this :description
>>> 	  (if (oref this jdkp)
>>> 	      (format "JDK %s Javadoc" (oref this version))
>>> 	    (let ((file (jdee-url-file (oref this url))))
>>> 	      (if (string-match ".*\\/\\(.*?\\)\\/doc\\/api" file)
>>> 		  (match-string 1 file)
>>> 		(jdee-url-name (oref this url))))))))
>>>
>>> This breaks if I just change call-next-method to cl-call-next-method.
>>
>> How do you change it?  
>
> I tried:
>
> (apply 'cl-call-next-method rest)
> (cl-call-next-method rest)
>
> They both give the error:
>
> cl-call-next-method: cl-call-next-method only allowed inside primary and around methods
>
>> I think you are supposed to call it as
>> (cl-call-next-method).
>
> That gives the same error.
>
> Note that I don't actually need to call the superclass init; I just need
> to process the args in `rest', storing them in the slots.

I think if you replace any of the args, you need to replace *all* of the
args. Ie, either:

(cl-call-next-method) ; in which case your changes to REST are ignored

or

(cl-call-next-method this rest)

Otherwise, it's just the wrong number of arguments.

I may be missing something, but...