Jakub Narebski <jnareb@xxxxxxxxx> writes: > [Cc: git@xxxxxxxxxxxxxxx] > > David K?gedal wrote: > >> @@ -123,6 +123,7 @@ >> >> ;;;###autoload >> (defun git-blame-mode (&optional arg) >> + "Minor mode for displaying Git blame" >> (interactive "P") >> (if arg >> (setq git-blame-mode (eq arg 1)) > > We should probably also change file description in header, and autoload > instruction in Installation: section of commentary: Agreed. > ;;; git-blame.el --- Minor mode for displaying Git blame -*- coding: utf-8 -*- > ;; > > [...] > > ;; If you do not want to load this package before it is necessary, you > ;; can make use of the `autoload' feature, e.g. by adding to your .emacs > ;; the following lines > ;; > ;; (autoload 'git-blame-mode "git-blame" > ;; "Minor mode for displaying Git blame" t) > > (docstring is needed because INTERACTIVE is _fourth_ parameter). Yes? Of course you need the doc string here, since the file isn't loaded yet. -- David Kågedal - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html