* bug#29400: 26.0; Add Elisp manual index entry for `defvar' to node `Compiler Errors'
[not found] ` <<83wp2f8uwr.fsf@gnu.org>
@ 2017-11-24 17:39 ` Drew Adams
0 siblings, 0 replies; 5+ messages in thread
From: Drew Adams @ 2017-11-24 17:39 UTC (permalink / raw)
To: Eli Zaretskii, Drew Adams; +Cc: 29400
> > > but I don't understand why you wanted an
> > > index entry with "defvar" in it. A reader who will look for "defvar"
> > > when they want to find ways of suppressing compiler warnings already
> > > knows that defvar is used for that purpose, so why would they use
> > > such a topic at Info-index's prompt?
> >
> > To find exactly what the manual says about it.
>
> But defvar is described elsewhere, and index entries about it should
> lead there. And after my changes, the description there mentions its
> use for suppressing warnings, so I don't think an index entry leading
> elsewhere is needed.
That was the point of the bug report. You have added
mention of supressing warnings to the node that is
indexed by `defvar', which responds to the report.
(And hopefully you will also have added there a link
to the more complete coverage of compiler warnings.)
> > Looking in the index for `warning' is not the only
> > reasonable way to use an index for this - see above.
>
> Feel free to suggest other topics, but "defvar" is
> definitely not one of them.
You just got through admitting that it is one of them.
You corrected, you say, the node that index-entry
`defvar' points to, by adding info there about
suppressing warnings (and presumably linking to the
node that deals more generally with such warnings and
their suppression).
^ permalink raw reply [flat|nested] 5+ messages in thread
[parent not found: <<f03d7e3c-b121-4aad-bc5c-1f4a8fba506f@default>]
* bug#29400: 26.0; Add Elisp manual index entry for `defvar' to node `Compiler Errors'
@ 2017-11-22 16:50 Drew Adams
2017-11-24 14:42 ` Eli Zaretskii
0 siblings, 1 reply; 5+ messages in thread
From: Drew Adams @ 2017-11-22 16:50 UTC (permalink / raw)
To: 29400
Node `Compiler Errors' seems to be the only place in the Elisp manual
where we tell users that you can use a vacuous `defvar' (no value) to
suppress a byte-compiler warning about it not being defined.
But this node has no index entry - at least none that has the word
`defvar' in it. Please add such an entry.
And please either add the same info (about this use of a vacuous
`defvar') to node `Defining Variables' or add a cross-reference from
that node to node `Compiler Errors'.
As it is now, it is not so easy to find this info.
In GNU Emacs 26.0.90 (build 3, x86_64-w64-mingw32)
of 2017-10-13
Repository revision: 906224eba147bdfc0514090064e8e8f53160f1d4
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
`configure --without-dbus --host=x86_64-w64-mingw32
--without-compress-install 'CFLAGS=-O2 -static -g3''
^ permalink raw reply [flat|nested] 5+ messages in thread
* bug#29400: 26.0; Add Elisp manual index entry for `defvar' to node `Compiler Errors'
2017-11-22 16:50 Drew Adams
@ 2017-11-24 14:42 ` Eli Zaretskii
0 siblings, 0 replies; 5+ messages in thread
From: Eli Zaretskii @ 2017-11-24 14:42 UTC (permalink / raw)
To: Drew Adams; +Cc: 29400-done
> Date: Wed, 22 Nov 2017 08:50:33 -0800 (PST)
> From: Drew Adams <drew.adams@oracle.com>
>
> Node `Compiler Errors' seems to be the only place in the Elisp manual
> where we tell users that you can use a vacuous `defvar' (no value) to
> suppress a byte-compiler warning about it not being defined.
>
> But this node has no index entry - at least none that has the word
> `defvar' in it. Please add such an entry.
I added some index entries, but I don't understand why you wanted an
index entry with "defvar" in it. A reader who will look for "defvar"
when they want to find ways of suppressing compiler warnings already
knows that defvar is used for that purpose, so why would they use such
a topic at Info-index's prompt? That's the opposite of what good
index entries should provide -- they should _lead_ to defvar's
description as the way to suppress warnings when the reader thinks of
"warnings" or some such.
> And please either add the same info (about this use of a vacuous
> `defvar') to node `Defining Variables' or add a cross-reference from
> that node to node `Compiler Errors'.
Done.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2017-11-24 17:39 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <<<f03d7e3c-b121-4aad-bc5c-1f4a8fba506f@default>
[not found] ` <<<83a7zbagbk.fsf@gnu.org>
[not found] ` <<f6b9ee9f-e49a-4d7d-9af8-12fb6f3af11d@default>
[not found] ` <<83wp2f8uwr.fsf@gnu.org>
2017-11-24 17:39 ` bug#29400: 26.0; Add Elisp manual index entry for `defvar' to node `Compiler Errors' Drew Adams
[not found] <<f03d7e3c-b121-4aad-bc5c-1f4a8fba506f@default>
[not found] ` <<83a7zbagbk.fsf@gnu.org>
2017-11-24 16:54 ` Drew Adams
2017-11-24 17:10 ` Eli Zaretskii
2017-11-22 16:50 Drew Adams
2017-11-24 14:42 ` Eli Zaretskii
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).