[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: change in call-next-method

From: Eric Abrahamsen
Subject: Re: change in call-next-method
Date: Sun, 03 Jan 2016 22:48:26 +0800
User-agent: Gnus/5.130014 (Ma Gnus v0.14) Emacs/25.1.50 (gnu/linux)

Stephen Leake <address@hidden> writes:

> Andreas Schwab <address@hidden> writes:
>> Stephen Leake <address@hidden> 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


(cl-call-next-method this rest)

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

I may be missing something, but...

reply via email to

[Prev in Thread] Current Thread [Next in Thread]