emacs-devel
[Top][All Lists]
Advanced

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

Re: scratch/accurate-warning-pos: Solid progress: the branch now bootstr


From: Alan Mackenzie
Subject: Re: scratch/accurate-warning-pos: Solid progress: the branch now bootstraps.
Date: Fri, 30 Nov 2018 18:55:03 +0000
User-agent: Mutt/1.10.1 (2018-07-13)

Hello, Paul.

On Fri, Nov 30, 2018 at 09:50:09 -0800, Paul Eggert wrote:
> On 11/29/18 2:05 PM, Alan Mackenzie wrote:
> > We're not talking about macros generating messages for invalid
> > input.  we're talking about generating message positions for the
> > source being compiled.

> I'm not sure what you mean here. The point of this exercise, as I 
> understand it, is to generate better diagnostics for Lisp code that has 
> problems in it (or is "invalid input" in some sense).

Then you've completely misunderstood.

The point of the exercise is specific, not vague as you have
misformulated it.  It is to print the correct source locations in
compiler warnings which are currently being output as wrong locations.
For example, read bug #9109.

> Such code may define macros, and it may use those macros or may use
> macros defined elsewhere. If you're saying that we should talk only
> about problems in the part of the code that uses macros, and not about
> problems in the part of the code that defines macros, then that sounds
> like a conversation that would be too narrow.

Feel free to have that wider conversation, somewhere.  I might even join
in.  It is orthogonal to the current set of bugs.  

> >
> >> I'm afraid we'll have to disagree here, as I can see examples where
> >> symbols-with-pos fails ....
> > I can't.  Please give an example, so I can see what you're talking about.

> I attempted to give one in 
> <https://lists.gnu.org/r/emacs-devel/2018-11/msg00530.html>, but you 
> dismissed it as not relevant to the discussion. As mentioned above, I 
> don't see why it's irrelevant.

I hope you now see why it is irrelevant, and will take back the
insinuation that symbols-with-pos fails, in the sense of bug #9101.

> > Symbols-with-pos exists, works, and works well.  Conses-with-pos
> > doesn't exist, hence doesn't work.  I doubt it can work.  Can you
> > create it?

> I think I could, with time.

But you're not going to?

> I hope someone else will do it, though, as the problem does not seem
> that urgent. I would suggest that whoever does it, looks into Stefan's
> ideas in this area, e.g.,
> <https://lists.gnu.org/r/emacs-devel/2018-11/msg00532.html>.

Nobody else is going to do it, and why should they?  I've already
explained to you why the approach you suggest won't work, you being
about the only person insisting that it will.  If you actually tried it,
as I have, you'd soon find out that I'm right, and why.

And if we only fixed "urgent" bugs, not a lot would get done on Emacs.
These bugs have been hanging around Emacs for many, many years, and it's
about high time they finally got fixed.  I have fixed them.

> > is the idea of the two instances of code still live?
> It's live only in the sense that it's better than
> scratch/accurate-warning-pos because of performance issues. I'd rather
> that we did something better than either.

There are no performance "issues".  Emacs with this bug fix is minutely
slower than without it, not enough to matter.  If you insist there are
"issues", then please perform some timings and report them here.

-- 
Alan Mackenzie (Nuremberg, Germany).



reply via email to

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