unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Alan Mackenzie <acm@muc.de>
Cc: emacs-devel@gnu.org
Subject: Re: How does one find out what file a library has been loaded from?
Date: Thu, 21 Jul 2022 20:53:28 +0300	[thread overview]
Message-ID: <838rommjxj.fsf@gnu.org> (raw)
In-Reply-To: <YtmOz+0n1Ah9kTse@ACM> (message from Alan Mackenzie on Thu, 21 Jul 2022 17:37:19 +0000)

> Date: Thu, 21 Jul 2022 17:37:19 +0000
> Cc: emacs-devel@gnu.org
> From: Alan Mackenzie <acm@muc.de>
> 
> > > +This function returns a file name associated with the file that
> > > +defined @var{symbol} (@pxref{eln files}).  If @var{type} is
> > > +@code{nil}, then any kind of definition is acceptable.  If @var{type}
> > > +is @code{defun}, @code{defvar}, or @code{defface}, that specifies
> > > +function definition, variable definition, or face definition only.
> 
> > This change is for the worse: it introduces a vague and confusing
> > notion of "file name associated with the file that defines" a symbol.
> > This should be removed from the patch, as it doesn't add any useful
> > information, just muddies the waters.
> 
> It's accurate, though.

No, it isn't accurate, because it doesn't say anything definitive.

What exactly did you want to say here, and why?  (See, I didn't even
understand you intention, from reading that text.)

> > >                                        If you want to find the actual
> > > +file loaded from, and you suspect if may really be a native compiled
> > > +file, something like the following should help.  You need to know the
> > > +name of a function which hasn't been advised, say @var{foo}, defined
> > > +in the suspected native compiled file.  Then
> > > +
> > > +@lisp
> > > +(let ((foo-fun (symbol-function #'FOO)))
> > > +       (and foo-fun (subr-native-elisp-p foo-fun)
> > > +            (native-comp-unit-file (subr-native-comp-unit foo-fun))))
> > > +@end lisp
> > > +
> > > +@noindent
> > > +will return either the name of the native compiled file defining
> > > +@var{foo}, or @code{nil} if there is no such file.
> 
> > This is not a good way of documenting some technique in this manual.
> > The way we describe such stuff is by documenting the functions a
> > program needs to use, not by giving a random example which calls the
> > functions without any documentation of the functions themselves.
> 
> OK.  But I think here could be an exception.  Describing the functions
> separately on their own page will not help users to get the loaded file
> name without a great deal of research.

You can describe them, and then show the example.  Or fill in the
blanks as part of the functions' description.

>                                         I've tried out this recipe and
> it works, but I don't yet know what these native-comp-unit functions are
> for, what they do in any detail, or even what a compilation-unit is.
> The functions are not already in the Elisp manual, and their doc strings
> are somewhat terse.

If you cannot figure it out from the code, feel free to ask questions.

> I still think it would be a good thing to be able to get the name of an
> actual load file from the .elc name stored in load-history without
> having to go through the intermediate step of knowing a function name
> defined by it.

Did you try comp-el-to-eln-filename?

> > Also, native-comp-unit-file doesn't exist in a build without native
> > compilation support, so some feature test is missing.
> 
> Do you mean a test in the TexInfo sources which would test whether it's
> necessary to include that example in the finished manual?

No, I mean a test in the @example code.  You want that snippet to be
complete, so that readers could copy it into their programs, right?



  parent reply	other threads:[~2022-07-21 17:53 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-07-19 10:52 How does one find out what file a library has been loaded from? Alan Mackenzie
2022-07-19 12:39 ` Eli Zaretskii
2022-07-19 15:01   ` Alan Mackenzie
2022-07-19 15:32     ` Andrea Corallo
2022-07-24 16:07       ` Eli Zaretskii
2022-07-24 17:46         ` Andrea Corallo
2022-07-31 12:52           ` Eli Zaretskii
2022-08-01  9:23             ` Andrea Corallo
2022-08-02  8:43               ` Andrea Corallo
2022-08-02 12:12                 ` Eli Zaretskii
2022-08-02 14:13                   ` Andrea Corallo
2022-08-03 14:19                     ` Eli Zaretskii
2022-08-01 19:31             ` Alan Mackenzie
2022-08-03 14:16               ` Eli Zaretskii
2022-07-19 15:50     ` Eli Zaretskii
2022-07-19 17:07       ` Alan Mackenzie
2022-07-19 19:13         ` Eli Zaretskii
2022-07-20 11:47           ` Alan Mackenzie
2022-07-20 15:31             ` Stefan Monnier
2022-07-20 20:34             ` Alan Mackenzie
2022-07-21  6:13               ` Eli Zaretskii
2022-07-21 17:37                 ` Alan Mackenzie
2022-07-21 17:52                   ` Stefan Monnier
2022-07-21 18:24                     ` Alan Mackenzie
2022-07-21 18:37                       ` Stefan Monnier
2022-07-21 21:03                         ` Alan Mackenzie
2022-07-21 23:15                           ` Stefan Monnier
2022-07-21 17:53                   ` Eli Zaretskii [this message]
2022-07-21 20:39                     ` Alan Mackenzie
2022-07-23 10:11                       ` Eli Zaretskii
2022-07-24 11:27                         ` Alan Mackenzie
2022-07-24 12:16                           ` Eli Zaretskii
2022-07-24 15:37                             ` Eli Zaretskii
2022-07-24 15:42                               ` Eli Zaretskii
2022-07-24 15:32                           ` Stefan Monnier
2022-07-24 15:49                             ` T.V Raman
2022-07-24 16:11                               ` Stefan Monnier
2022-07-24 18:21                                 ` T.V Raman
2022-07-24 18:50                                   ` Stefan Monnier
2022-07-24 16:19                               ` Eli Zaretskii
2022-07-19 16:27     ` Stefan Monnier
2022-07-20 18:36       ` Andrea Corallo
2022-07-21  7:20         ` Eli Zaretskii

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=838rommjxj.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=acm@muc.de \
    --cc=emacs-devel@gnu.org \
    /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).