bug#11711: 24.0.95; [PATCH] added imenu and which-func support to rst.el

[Stefan Monnier <monnier@iro.umontreal.ca>]

> This is not a bug report. This is a request to merge a patch
> to the offcial source tree.

> 2012-06-13  Masatake YAMATO  <yamato@redhat.com>

> 	* textmodes/rst.el (rst-mode, rst-imenu-index): Added
> 	imenu and which-func support.

I'll let the other Stefan decide what to do with it.  It looks
pretty good for me.  Just a few nitpicks below:

> +  ;; Imenu
> +  (set (make-local-variable 'imenu-create-index-function) 'rst-imenu-index)
> +
> +  ;;  Which func
> +  (when (and (boundp 'which-func-modes) (listp which-func-modes))
> +    (add-to-list 'which-func-modes 'rst-mode))
> +
>    ;; Font lock.

Try to punctuate your comments.

> +(defun rst-imenu-find-adornments-for-position (adornments p)
> +  "Find adornments cell for position P in ADORNMENTS"

And do punctuate your docstring (C-u M-x checkdoc-current-buffer RET
might help you catch such things).


        Stefan