Re: Some problems in `add-log-current-defun'

[Herbert Euler]

> I would like to install your fix for problems I, II and III,
> but it is big enough that we need legal papers for it.
> I will send you what is needed to do this, and in the mean time,
> I will save that patch.

I am trying to get the postal address and will send this mail
later.  Thank you.

Here I'd like to say something for the fix of IV.  This change

2006-12-26  Guanpeng Xu  <address@hidden>  (tiny change)

        * add-log.el (add-log-current-defun): Call `forward-sexp'
        multiple times to pick a member function name defined as
        part of nested classes/namespaces.

created a bug.  For the following example,

   struct abc {
   -!-};

If point is at -!-, `add-log-current-defun' will return ``abc'' rather
than the desired, old ``struct abc''.  I found it when trying to
solve IV.  So, please consider to either revert the above change,
or install the fix of IV.  I think it is necessary to keep the behavior
correct for at least C, and based on my test I think there is little
risk of breaking something else to install the fix of IV.

Regards,
Guanpeng Xu

_________________________________________________________________
FREE pop-up blocking with the new MSN Toolbar - get it now! 
http://toolbar.msn.click-url.com/go/onm00200415ave/direct/01/