GNU bug report logs - #60644
30.0.50; eval-defun: in specific case finds wrong function below the current one

Previous Next

Full log

Message #14 received at 60644 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Jean Louis <bugs <at> gnu.support>,
 Stefan Monnier <monnier <at> iro.umontreal.ca>
Cc: 60644 <at> debbugs.gnu.org
Subject: Re: bug#60644: 30.0.50; eval-defun: in specific case finds wrong
 function below the current one
Date: Sun, 08 Jan 2023 14:03:06 +0200

> Date: Sun, 8 Jan 2023 14:21:12 +0300
> From: Jean Louis <bugs <at> gnu.support>
> Cc: 60644 <at> debbugs.gnu.org
> 
> * Eli Zaretskii <eliz <at> gnu.org> [2023-01-08 13:37]:
> > > Date: Sun, 08 Jan 2023 09:19:05 +0300
> > > From: Jean Louis <bugs <at> gnu.support>
> > > 
> > > Find in the file two functions, go somewhere in the middle of
> > > `hyperscope-tag-hyperdocuments-by-type' and try invoking C-M-x and you
> > > will see `rcd-db-recurse-on-table' instead of
> > > `hyperscope-tag-hyperdocuments-by-type'. In general I can't instrument
> > > this function interactively.
> > 
> > Its indentation is wrong.  Reindent the line where you have
> > "(interactive)", and your problems will likely disappear.
> 
> I get it, it works.
> 
> I was not aware that detection system should rely on indentation, but
> rather on sexp.

AFAIU, the top-level form (which is what C-M-x is about) is detected
in a way that depends on indentation.

Stefan, any further comments, or should we close this?

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.