Subject: bug#20545: New minor mode Electric Quote

From: Eli Zaretskii <[email-masked]>
Date: Wed, 27 May 2015 17:24:23 +0300

> Date: Tue, 26 May 2015 04:24:12 -0700
> From: Paul Eggert <[email-masked]>
> CC: Eli Zaretskii <[email-masked]>, 
>  Stefan Monnier <[email-masked]>,
>  Artur Malabarba <[email-masked]>
> 
> Attached is a revised patch that should address the comments raised:

Thanks.  Please allow me a comment about the documentation:

> --- a/doc/emacs/modes.texi
> +++ b/doc/emacs/modes.texi
> @@ -200,6 +200,15 @@ Auto Save mode saves the buffer contents periodically to reduce the
>  amount of work you can lose in case of a crash.  @xref{Auto Save}.
>  
>  @item
> +@cindex Electric Quote mode
> +@cindex mode, Electric Quote
> +@findex electric-quote-mode
> +Electric Quote mode automatically converts quotation marks.  For
> +example, it requotes text typed @t{`like this'} to text @t{‘like
> +this’}.  You can control what kind of text it operates in, and you can
> +disable it entirely in individual buffers.  @xref{Quotation Marks}.
> [...]
> --- a/doc/emacs/text.texi
> +++ b/doc/emacs/text.texi
> @@ -404,6 +405,43 @@ that separates pages (@pxref{Regexps}).  The normal value of this
>  variable is @code{"^\f"}, which matches a formfeed character at the
>  beginning of a line.
>  
> +@node Quotation Marks
> +@section Quotation Marks
> +@cindex Quotation marks
> +@cindex Electric Quote mode
> +@cindex mode, Electric Quote
> +  One common way to quote is the typewriter convention, which quotes

As you see, you have identical index entries in 2 different places.
That is generally not useful, because they get converted to something
like "foo" and "foo <1>", and the reader will have no way of knowing
which one does she want.

In this case, I would simply drop the first set of entries (and move
the "electric-quote-mode" one to the second set), to where the actual
description of this mode lives.  There's no need to index a place that
just mentions the mode in passing and sends the reader to the real
description.