Re: Add log bug

[Masatake YAMATO]

> There is a bug in add-log.el, in the function
> `add-log-current-defun'.  There was a fix; but it is not complete.
> The old description of the bug can be found at
> 
>     http://lists.gnu.org/archive/html/emacs-devel/2006-09/msg00857.html
> 
> The problem exists yet now.  If you put point in the body of the C++
> function below:
> 
> void
> web_slim::slim_dirent::clear ()
> {
>   for (slim_dirent_seq::iterator i = sub.begin (); i != sub.end (); ++i)
>     i->clear ();
>   sub.clear ();
> }
> 
> `add-change-log-entry-other-window' would try to add a log entry for
> function `slim_dirent::clear', rather than the desired
> `web_slim::slim_dirent::clear'.
> 
> The old fix changed `forward-word' into `forward-sexp', but the latter
> one only moves one sexp.  So the point is put before `slim_dirent' in
> the above example.  A complete fix could be:

You are/were right. My patch was wrong.
Do you have a write-access to the CVS repository?

Masatake YAMATO