Subject: Re: [PATCH 6/7] git-blame.el: Doc fixes and cleanup

From: Jakub Narebski <[email-masked]>
Date: Fri, 09 Feb 2007 10:39:45 +0100

[Cc: [email-masked]]

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:

;;; 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).
-- 
Jakub Narebski
Warsaw, Poland
ShadeHawk on #git