Re: [CEDET-devel] eieio and edebug
Brought to you by:
zappo
From: Eric M. L. <er...@si...> - 2011-04-29 00:40:34
|
On 04/26/2011 03:46 PM, David Engster wrote: > Eric M. Ludlam writes: >> On 03/22/2011 03:02 AM, Darren Hoo wrote: >>> eieio has support for documentation so that describe-variable or >>> describe-function can be used on class definition and method definition >>> , but where they are defined are not trackered, so directcly jumping from >> >>> the from the help cross ref to the file position they are defined is >>> impossible. I am really missing this feature. >>> >>> load-history only records defun, defface and defvar, but not defclass or >>> defmethod..., so if these info is to be trackered, it should be done by >>> eieio itself. I wonder if this can be improved. > > [...] > >> I tried fixing that some years back. It is also related to >> 'unload-library" which is broken with any eieio based program. I was >> not able to figure things out from that side (adding items to the load >> history). I looked into augmenting aspects of find-function. That >> works ok with mode-local, but at the time I didn't get it working w/ >> eieio classes. After all, which method of many do you want to jump to? >> >> I don't doubt it is a solvable problem since eieio generates the help >> text of all methods, so it could create links to the ones that match. I >> think that's a matter of time for someone to tuck in and do it. > > Since Darren's mail reminded me that this is actually something which > has annoyed me for ages, I now implemented this. It's a bit hacky > because - as Darren wrote - we cannot use load-history, but I simply > save the filename(s) for the different methods in a symbol property. > > I attached a patch; it will display a link for every > implementation. It's currently only for methods, not for classes, but I > first want to know if you agree with the general direction of the > patch. Adding the rest should be straightforward. Hi David, Your solution was not what I had been thinking of, which is probably a good idea. Getting the links into the doc for EIEIO sounds like a clever idea. The thing I had been stuck on which I described above was getting 'find-function' to work on methods. Anyway, I like your patch. I think it's a good idea. Eric |