all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Stefan Guath <stefan@automata.se>
To: Stefan Monnier <monnier@iro.umontreal.ca>
Cc: 17685@debbugs.gnu.org
Subject: bug#17685: 24.3.91.1; help-C-file-name failes to create temp buffer " *DOC*"
Date: Thu, 5 Jun 2014 08:46:57 +0200	[thread overview]
Message-ID: <632050FA-D476-417C-8956-3C9A8508164F@automata.se> (raw)
In-Reply-To: <jwvsinkhlfe.fsf-monnier+emacsbugs@gnu.org>

I understand. But it seems an unsatisfactory solution to demand callers of find-lisp-object-file-name to pre-evaluate (get-buffer-create " *DOC*") in order to activate its c-source search ability (i.e. convoluted code, code breaks when buffer name changes etc). Maybe just add an optional argument in find-lisp-object-file-name? Something like enable-c-search with the explanation "Please note that this will be memory consuming."?

/Stefan Guath

On 4 jun 2014, at 15:15, Stefan Monnier <monnier@iro.umontreal.ca> wrote:

>> emacs -Q
>> (find-lisp-object-file-name 'visible-bell 'defvar) ;visible-bell is just an
>> example and can be replaced with any variable.
> 
>> RESULT:
>> C-source
> 
>> EXPECTED RESULT:
>> "src/dispnew.c"
> 
> Here's the reason for the behavior:
> the DOC file is moderately large, so we don't want to load it into
> memory just because the user did C-h v visible-bell RET.
> 
> So we only populate the *DOC* buffer when the user actually clicks on
> the "C-source" link to jump to the source code.  And once the buffer is
> populated, then there's no reason not to use it, so if *DOC* exists then
> we do use it.
> 
> We could probably change it so that we always populate the *DOC* buffer
> in the case where the C sources are available (so the behavior would
> stay unchanged for users running pre-compiled Emacs, while it would
> work better for those users who compile it themselves and keep the
> source in place).  But I'm not sure it'd really be an improvement.
> 
> 
>        Stefan






  parent reply	other threads:[~2014-06-05  6:46 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-04  8:47 bug#17685: 24.3.91.1; help-C-file-name failes to create temp buffer " *DOC*" Stefan Guath
2014-06-04 13:15 ` Stefan Monnier
2014-06-04 14:08   ` Drew Adams
2014-06-04 14:21     ` Stefan Monnier
2014-06-05  6:46   ` Stefan Guath [this message]
2014-06-05  7:48     ` martin rudalics
2014-06-05  8:35       ` Stefan Guath
2014-06-05  8:45         ` martin rudalics
2014-06-05  9:00           ` Stefan Guath
2014-06-05 13:51         ` Stefan Monnier
2022-02-08 10:11       ` Lars Ingebrigtsen

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

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

  git send-email \
    --in-reply-to=632050FA-D476-417C-8956-3C9A8508164F@automata.se \
    --to=stefan@automata.se \
    --cc=17685@debbugs.gnu.org \
    --cc=monnier@iro.umontreal.ca \
    /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 external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.