From: Jakub Narebski <jnareb@gmail.com>
Date: Fri, 09 Feb 2007 09:39:45 GMT
Subject: Re: [PATCH 6/7] git-blame.el: Doc fixes and cleanup
Message-ID: <eqhfdv$8uq$1@sea.gmane.org>
In-Reply-To: <87irebpwps.fsf@morpheus.local>

[Cc: git@vger.kernel.org]

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