all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: van@scratch.space
Cc: 36927@debbugs.gnu.org
Subject: bug#36927: 26.2.90; include stdarg.h is without hyperlinking on vfprintf's manpage
Date: Fri, 09 Aug 2019 16:58:00 +0300	[thread overview]
Message-ID: <83ftmaa1av.fsf@gnu.org> (raw)
In-Reply-To: <83imrbd24r.fsf@gnu.org> (message from Eli Zaretskii on Mon, 05 Aug 2019 19:10:28 +0300)

> Date: Mon, 05 Aug 2019 19:10:28 +0300
> From: Eli Zaretskii <eliz@gnu.org>
> Cc: 36927@debbugs.gnu.org
> 
> > From: VanL <van@scratch.space>
> > Date: Mon, 5 Aug 2019 11:51:56 +1000
> > 
> > See manpage for vfprintf().
> > 
> > Unlike the first instance of the #include preprocessing directive
> > 
> >   #include <stdio.h>
> > 
> > at approximately line 12 of printf(3), which is underlined and
> > hyperlinked, the second instance at approximately line 32, is not
> > underlined and not hyperlinked.
> > 
> > 1. M-x man RET vfprintf
> > 2. on line 32 see #include <stdarg.h> which is without hyperlinking
> 
> For some reason, we don't put a button on the second #include.

The logic in Man-highlight-references0 was flawed: it assumed that a
section ("SYNOPSIS", "FILES", "SEE ALSO", etc.) is always received
entirely in a single chunk of text that is read from the subprocess,
which is, of course, not guaranteed, and depends on factors such as
the size of each section, buffering, how busy is the system, etc.
(Those reasons also made this a bitch to debug, because a bug comes
and goes unpredictably, and never shows when one steps with Edebug.)

I think I made it much better on master, although I still see a rare
case where one or the other "include" is not buttonized.  But I've run
out of time, so someone else will have to find and fix what I missed.

I'm leaving this bug open because I don't think it's 100% fixed.





  parent reply	other threads:[~2019-08-09 13:58 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-08-05  1:51 bug#36927: 26.2.90; include stdarg.h is without hyperlinking on vfprintf's manpage VanL
2019-08-05 16:10 ` Eli Zaretskii
2019-08-06  2:36   ` VanL
2019-08-06  3:56     ` Eli Zaretskii
2019-08-06  7:22       ` VanL
2019-08-06 14:36         ` Eli Zaretskii
2019-08-07  8:57           ` VanL
2019-08-09 13:58   ` Eli Zaretskii [this message]
2019-08-10  5:05     ` VanL
2019-08-10  7:48       ` Eli Zaretskii
2019-08-11  5:26         ` VanL
2019-08-11 14:09           ` Eli Zaretskii
2019-08-18 22:37     ` Noam Postavsky

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=83ftmaa1av.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=36927@debbugs.gnu.org \
    --cc=van@scratch.space \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.