loccur.el - loccur function should be autoloaded

loccur.el - loccur function should be autoloaded

[pwr]

Since the command loccur will nearly always be called before loccur-mode, this
should be autoloaded to avoid needing to explicitly require the mode (in this
modern age of package-initialize).


---
/Users/Paul/.emacs.d/backups/!Users!Paul!.emacs.d!packages!loccur-1.2.3!loccur.el~	2019-08-05
14:06:10.000000000 +1000
+++ /Users/Paul/.emacs.d/packages/loccur-1.2.3/loccur.el	2019-08-05
14:08:58.000000000 +1000
@@ -166,6 +166,7 @@
       (when (overlay-get ovl loccur-overlay-visible-property-name)
         (overlay-put ovl 'face (if loccur-highlight-matching-regexp
'loccur-face nil))))))

+;;;###autoload
 (defun loccur (regex)
   "Perform a simple grep in current buffer.

Re: loccur.el - loccur function should be autoloaded

[Stefan Monnier]

> Since the command loccur will nearly always be called before loccur-mode, this
> should be autoloaded to avoid needing to explicitly require the mode (in this
> modern age of package-initialize).

Indeed, thanks.  Autloading loccur-mode is not very useful, in contrast.
I also added an autoload cookie for loccur-current, since the doc
suggests it as another entry point (I didn't add loccur-previous-match,
OTOH, since this one only works after another loccur use).


        Stefan

Re: loccur.el - loccur function should be autoloaded

[pwr]

> Indeed, thanks.  Autloading loccur-mode is not very useful, in contrast.
> I also added an autoload cookie for loccur-current, since the doc
> suggests it as another entry point (I didn't add loccur-previous-match,
> OTOH, since this one only works after another loccur use).

I was just cloning the ELPA repo but then saw your commit. Thanks :)