unofficial mirror of help-gnu-emacs@gnu.org
 help / color / mirror / Atom feed
From: Eric Abrahamsen <eric@ericabrahamsen.net>
To: help-gnu-emacs@gnu.org
Subject: Re: Adding and running a major mode hook
Date: Tue, 29 Apr 2014 08:45:55 +0800	[thread overview]
Message-ID: <87vbttq7y4.fsf@ericabrahamsen.net> (raw)
In-Reply-To: 96032cfe-f440-4eca-9d44-07fc14fb7371@googlegroups.com

Jacob Gerlach <jacobgerlach@gmail.com> writes:

>> If you've got the run-hooks inside the mode declaration, then the hooks
>> 
>> won't run until the user activates the mode in some particular buffer.
>> 
>> They will then run *every* time your mode is activated.
>> 
>> 
>> 
>> If you've got it at the top level of the file, the hooks will run once,
>> 
>> *while* the file is being loaded for the first time: ie quite likely
>> 
>> during emacs startup, possibly before the user's relevant customization
>> 
>> is loaded, probably before any relevant file are opened.
>
> Ahhh, this is the key piece of information that I was missing. I was
> confused by the fact that the mode declaration wasn't bothered by the
> fact that it came before the function definitions further on in the
> file.
>
>> >From what I can see from your examples above, it might be enough just to
>> 
>> have:
>> [...]
>> Then, *every time* the user activates this
>> 
>> mode, constructs are created out of whatever's in `content-list'.
>  
> The creation uses add-to-list, which checks for duplicates, so that
> won't be a problem necessarily, but it does seem like unnecessary
> overhead that I can get rid of. I will play with some conditionals to
> see what I can do about this.
>
>> Hope that didn't make it worse,
> On the contrary, much clearer now!
>
> To add a wrinkle to this, can someone explain how autoload changes this sequence? I tried to implement autoload by putting something like this in my file:
>
> (autoload 'my-mode "my-name" "My Editing Mode" t)
>
> I copied it from somewhere, but now that I read the documentation more carefully, I think I have two problems with this:
>
> First, "my-name" should actually be the filename: "mymode.el". Does
> this argument care about paths?

`autoload' uses `load', which searches for files in your 'load-path. If
the file you want to load isn't in your load-path, you can give an
absolute path, but that's probably a bad idea. You don't need the ".el"
suffix -- load will try with ".el", ".elc" and no suffix. (see
'load-suffixes)

> second, I put this line after the mode definition, but the help says:
>
> If FUNCTION is already defined other than as an autoload,
> this does nothing and returns nil.
>
> So I think it's doing nothing (and perhaps that's why it doesn't
> matter that I didn't include the file name as I should have). It also
> seems it would be much easier to make the mode autoloaded using a
> magic comment, which is how I'm fixing it.
>
> With regards to the order of code execution, how does autoloading the
> mode definition change what Eric already described? Users will still
> have(require 'my-mode) in their .emacs, which I think executes
> everything except the mode definition.
>
> Is it then redundant to put autoload on a function definition?

It's redundant to have an autoload for a function that is defined in the
same file, yes. autoloads should be put elsewhere, either in the user's
configuration or in other libraries that might theoretically depend on
your library.

Users will either use require, or autoload, but not both. Require means
"load it now" (ie the whole file is executed, probably at emacs
startup), and autoload means "load it the first time I try to use it",
which probably happens as a relevant file is being opened. Practically
speaking that's the only difference.

HTH,
Eric




      parent reply	other threads:[~2014-04-29  0:45 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-26 17:25 Adding and running a major mode hook Jacob Gerlach
2014-04-27  3:12 ` Eric Abrahamsen
2014-04-27  5:15 ` Eric Abrahamsen
     [not found] ` <mailman.107.1398568207.1147.help-gnu-emacs@gnu.org>
2014-04-27 11:56   ` Jacob Gerlach
2014-04-27 12:10     ` Jacob Gerlach
2014-04-29  0:45     ` Eric Abrahamsen [this message]

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=87vbttq7y4.fsf@ericabrahamsen.net \
    --to=eric@ericabrahamsen.net \
    --cc=help-gnu-emacs@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.
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).