[Top][All Lists]

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

bug#24449: Emacs 25.1 RC2: Byte compiler reports error in wrong place.

From: Alan Mackenzie
Subject: bug#24449: Emacs 25.1 RC2: Byte compiler reports error in wrong place.
Date: Fri, 16 Sep 2016 16:08:33 +0000
User-agent: Mutt/1.5.24 (2015-08-30)

Hello, Eli.

On Fri, Sep 16, 2016 at 06:09:38PM +0300, Eli Zaretskii wrote:
> > Date: Fri, 16 Sep 2016 13:33:52 +0000
> > Cc: address@hidden
> > From: Alan Mackenzie <address@hidden>

> > > If you didn't, you should, because after you do, you will never again
> > > wonder why an incorrect line number is reported.  In fact, now that I
> > > did look there, I'm surprised it reports a correct line number at all,
> > > let alone as often as it does.  It's sheer luck.

> > This is not a Good Thing.  Even its own comment describes itself as a
> > "gross hack".  Surely we can do better?

> I certainly hope we can.  But, unless I misunderstood something, the
> way it's designed makes that really hard.

Yes.  I understand it better now, though the comments before
`byte-compile-set-symbol-position' are not as helpful as they might be.

Given that the byte compiler works by first reading an entire top-level
form, and only then going to work on it, the only handle the compiler
has on the original source is this list `read-symbol-positions-list'
produced by the reader.  (It was probably invented for the byte

So without rewriting one or both of the byte compiler and the reader,
there doesn't seem to be a different strategy available for determining
the position in the raw source.  Correction: there might be: On
processing a symbol, at the moment the earliest occurrence of that
symbol in `read-symbol-positions-list' is removed from it.  Instead, we
could remove everything up to and including that symbol.  Maybe.

What I'm guessing happened in my particular case is that several
instances of 'eq in that list failed to get removed because it's a
function that undergoes compiler optimisation.  Or something like that.

So, the thing for me to check first is that
`byte-compile-set-symbol-position' gets called for _everything_ it
should be.

Alan Mackenzie (Nuremberg, Germany).

reply via email to

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