unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Robert Weiner <rsw@gnu.org>
To: Rocky Bernstein <rocky@gnu.org>
Cc: emacs-devel <emacs-devel@gnu.org>
Subject: Re: A couple of things that I think should be in byte bytecode meta comments
Date: Sat, 23 Dec 2017 13:30:49 -0500	[thread overview]
Message-ID: <CA+OMD9j=DJPEd1mN36vjs+oScNquXHw=_E4FicKbNScJOgkVWg@mail.gmail.com> (raw)
In-Reply-To: <CANCp2gYNBgeknpxBi8oK7F_5QU6HZWJPZ0B9QR4E12Nc2uc=-w@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 2785 bytes --]

On Sat, Dec 23, 2017 at 12:16 PM, Rocky Bernstein <rocky@gnu.org> wrote:

>
> My suggestion was that when a relative path is given, (which is always in
> your world), also turn that into an absolute path and store in the bytecode
> file as well.
>

​People are just sharing their thoughts on whether that appears to be
helpful or not.  You have acknowledged that at runtime, the absolute path
may be invalid, but you have not provided your use case to show where and
when it might be helpful.

​​
>   In my experience in working in debugging, debuggers and in showing where
> errors are, sometimes the absolute path can be useful in addition to the
> relative path, for informative purposes.
>
​​
​Only when it is accurate.  But you haven't yet made a suggestion about how
to make absolute paths in .elc files accurate during runtime, as far as I
recall.
Maybe a static absolute path would provide a heuristic for finding the
run-time absolute path, maybe not.  You haven't shown anything either way
yet.

> ​​
> ​I know others will not believe that, and furthermore claim most
> emphatically that were these paths included as meta comments  in the
> bytecode file, that would be either useless, confusing and harmful and who
> knows what other bad things could happen.  (By the way, as something
> similar, when I enter Emacs, I am shown a build string for a system is not
> mine, which so far has not caused such havoc, and I even find mildly
> interesting).
>

​It is a matter of utility, not of havoc.
​

> ​Ok. this was a just suggestion. That it has caused such negative
> reaction, I've seen before in other venues, and I'm sorry.
> ​​
> For my part, I know what I can do to handle my concerns when or if I
> decide to improve the error reporting and debugging situation on Emacs.
> Yes, I am sorry I didn't say at the outset that this is were I envision
> this getting used.
>

​Yes, an easy to follow use case would be helpful.  I am very much in favor
of improving Emacs' error messages, especially anything that leads to the
source of an error when a backtrace is not produced.  If you can explain
how something you envision could make that happen, then you'll likely see
feedback change.

BTW, I think there might be a good idea in there about using hash codes to
verify valid use of a file, though my personal experience is that
incompatible byte codes are well reported by Emacs and have not caused me
any problems to date.  The much bigger problem is tracing an error raised
from C code back to the source function that raised it when running without
a C debugger active.  Without that, users can't provide much of a bug
report except possibly how to trigger the problem.

Bob

[-- Attachment #2: Type: text/html, Size: 5468 bytes --]

  reply	other threads:[~2017-12-23 18:30 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-22 17:54 A couple of things that I think should be in byte bytecode meta comments Rocky Bernstein
2017-12-22 17:55 ` Rocky Bernstein
2017-12-22 18:30   ` Eli Zaretskii
2017-12-22 18:49     ` Rocky Bernstein
2017-12-22 20:25       ` Eli Zaretskii
2017-12-22 20:55         ` Rocky Bernstein
2017-12-23  8:25           ` Eli Zaretskii
2017-12-23  8:44             ` Rocky Bernstein
2017-12-23 15:32               ` Robert Weiner
2017-12-23 17:16                 ` Rocky Bernstein
2017-12-23 18:30                   ` Robert Weiner [this message]
2017-12-24  2:44                     ` Rocky Bernstein
2018-01-13 19:44                       ` Thien-Thi Nguyen
2017-12-22 21:27 ` Andreas Schwab
2017-12-22 21:43   ` Rocky Bernstein
2017-12-23  8:25     ` Andreas Schwab
2017-12-23  8:35       ` Rocky Bernstein

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CA+OMD9j=DJPEd1mN36vjs+oScNquXHw=_E4FicKbNScJOgkVWg@mail.gmail.com' \
    --to=rsw@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=rocky@gnu.org \
    --cc=rswgnu@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).