* bug#31574: 27.0; Doc string for `with-help-window'
[not found] ` <<83k1rtauc7.fsf@gnu.org>
@ 2018-05-24 17:17 ` Drew Adams
2018-05-24 18:09 ` Eli Zaretskii
0 siblings, 1 reply; 8+ messages in thread
From: Drew Adams @ 2018-05-24 17:17 UTC (permalink / raw
To: Eli Zaretskii, Drew Adams; +Cc: 31574-done
> > The doc string ends with this sentence:
> >
> > See 'help-window-setup' for more options.
> >
> > But that link takes you to this text, which tells you nothing at all
> > about "more options":
>
> "Options" was not the best word here, so I rephrased the last sentence
> of the doc string.
I see that the bug was closed. Not sure what the fix was,
but if it was just to change "options" then it might not
really be fixed.
The real problem is that after you follow the link there
is NOTHING about `with-help-window', AFAICT.
And I should have been clear that the doc string of
`help-window-setup' is even more problematic. It doesn't
say anything helpful at all, AFAICT. What does setting
up that window even mean? The only thing this doc string
tells you, that you might not guess, is that VALUE is
returned from the function.
Did you fix the doc string of `help-window-setup', or can
you please do so as part of this bug? Or do you need a
separate bug filed for that?
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#31574: 27.0; Doc string for `with-help-window'
2018-05-24 17:17 ` bug#31574: 27.0; Doc string for `with-help-window' Drew Adams
@ 2018-05-24 18:09 ` Eli Zaretskii
0 siblings, 0 replies; 8+ messages in thread
From: Eli Zaretskii @ 2018-05-24 18:09 UTC (permalink / raw
To: Drew Adams; +Cc: 31574
> Date: Thu, 24 May 2018 10:17:49 -0700 (PDT)
> From: Drew Adams <drew.adams@oracle.com>
> Cc: 31574-done@debbugs.gnu.org
>
> > "Options" was not the best word here, so I rephrased the last sentence
> > of the doc string.
>
> I see that the bug was closed. Not sure what the fix was,
> but if it was just to change "options" then it might not
> really be fixed.
I told yesterday in response to another message of yours how can you
see the changes in the repository using Web browser. Please take a
look, then we can discuss the fix.
> The real problem is that after you follow the link there
> is NOTHING about `with-help-window', AFAICT.
No, the real problem is that the reference to help-window-setup gave
you wrong expectations regarding what information that function should
add. My change tried to make those expectations match the original
intent.
> And I should have been clear that the doc string of
> `help-window-setup' is even more problematic. It doesn't
> say anything helpful at all, AFAICT.
That's a subroutine, so I don't necessarily see its doc string as
something we need to invest an effort in.
^ permalink raw reply [flat|nested] 8+ messages in thread
[parent not found: <<<eb176951-ffc9-4303-893f-9c146782136d@default>]
* bug#31574: 27.0; Doc string for `with-help-window'
@ 2018-05-23 19:45 Drew Adams
2018-05-24 7:20 ` martin rudalics
2018-05-24 16:31 ` Eli Zaretskii
0 siblings, 2 replies; 8+ messages in thread
From: Drew Adams @ 2018-05-23 19:45 UTC (permalink / raw
To: 31574
The doc string ends with this sentence:
See 'help-window-setup' for more options.
But that link takes you to this text, which tells you nothing at all
about "more options":
Set up help window WINDOW for 'with-help-window'.
WINDOW is the window used for displaying the help buffer.
Return VALUE.
In GNU Emacs 27.0.50 (build 3, x86_64-w64-mingw32)
of 2018-03-21
Repository revision: e70d0c9e66d7a8609450b2889869d16aeb0363b5
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
`configure --without-dbus --host=x86_64-w64-mingw32
--without-compress-install -C 'CFLAGS=-O2 -static -g3''
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#31574: 27.0; Doc string for `with-help-window'
2018-05-23 19:45 Drew Adams
@ 2018-05-24 7:20 ` martin rudalics
2018-05-24 16:31 ` Eli Zaretskii
1 sibling, 0 replies; 8+ messages in thread
From: martin rudalics @ 2018-05-24 7:20 UTC (permalink / raw
To: Drew Adams, 31574, Nick Helm
> The doc string ends with this sentence:
>
> See 'help-window-setup' for more options.
>
> But that link takes you to this text, which tells you nothing at all
> about "more options":
Indeed. This was part of the fix for Bug#30792. Nick, could you
please have a look ?
martin
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#31574: 27.0; Doc string for `with-help-window'
2018-05-23 19:45 Drew Adams
2018-05-24 7:20 ` martin rudalics
@ 2018-05-24 16:31 ` Eli Zaretskii
1 sibling, 0 replies; 8+ messages in thread
From: Eli Zaretskii @ 2018-05-24 16:31 UTC (permalink / raw
To: Drew Adams; +Cc: 31574-done
> Date: Wed, 23 May 2018 12:45:12 -0700 (PDT)
> From: Drew Adams <drew.adams@oracle.com>
>
> The doc string ends with this sentence:
>
> See 'help-window-setup' for more options.
>
> But that link takes you to this text, which tells you nothing at all
> about "more options":
"Options" was not the best word here, so I rephrased the last sentence
of the doc string.
Thanks.
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2018-05-24 19:26 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <<eb176951-ffc9-4303-893f-9c146782136d@default>
[not found] ` <<83k1rtauc7.fsf@gnu.org>
2018-05-24 17:17 ` bug#31574: 27.0; Doc string for `with-help-window' Drew Adams
2018-05-24 18:09 ` Eli Zaretskii
[not found] <<<eb176951-ffc9-4303-893f-9c146782136d@default>
[not found] ` <<<83k1rtauc7.fsf@gnu.org>
[not found] ` <<73e710f8-8373-46e5-913a-b3cba178a384@default>
[not found] ` <<83zi0p9b8l.fsf@gnu.org>
2018-05-24 18:26 ` Drew Adams
2018-05-24 19:09 ` Eli Zaretskii
2018-05-24 19:26 ` Drew Adams
2018-05-23 19:45 Drew Adams
2018-05-24 7:20 ` martin rudalics
2018-05-24 16:31 ` Eli Zaretskii
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.