[Top][All Lists]

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

Re: Add log bug

From: Masatake YAMATO
Subject: Re: Add log bug
Date: Tue, 26 Dec 2006 11:06:26 +0900 (JST)

> 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

reply via email to

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