[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Emacs-diffs] scratch/so-long 7273fb2: Add so-long library
From: |
Stefan Monnier |
Subject: |
Re: [Emacs-diffs] scratch/so-long 7273fb2: Add so-long library |
Date: |
Thu, 10 Jan 2019 13:07:25 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux) |
> +FIXME: +++ once all documentation has been added.
No need for this FIXME: the whole point of this system is that you don't
need to do anything special to mark entries as "not yet handled" (any `**`
header without a preceding `---` or `+++` has not been handled yet), so
it's "fail safe".
> +;; Installation
> +;; ------------
> +;; And add the following to your init file to enable the library:
> +;;
> +;; (when (require 'so-long nil :noerror)
> +;; (so-long-enable))
I think the above should just be
(so-long-auto-mode 1)
and so-long-auto-mode (just my suggestion for a name, feel free to
chose something else) should be an autoloaded global minor mode which
activates so-long-mode.
On the design side, I think you should merge `so-long` and
`so-long-mode` into a single function and make that function
a (buffer-local) minor-mode (i.e. not have any major mode, just use
fundamental-mode instead). Making it a minor mode rather than
a major-mode will also make it easier to remember the previous
major-mode without any need for a change-major-mode-hook hack.
> +(defvar so-long-enabled t
> + "Set to nil to prevent `so-long' from being triggered automatically.")
Loading a file should not change the behavior of Emacs, so the default
should be nil unless/until we decide to preload this package.
Rather than defadvice, please use advice-add.
Other than that, looks great, thanks.
Stefan
- Re: [Emacs-diffs] scratch/so-long 7273fb2: Add so-long library,
Stefan Monnier <=