all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: VanL <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: Sun, 11 Aug 2019 17:09:38 +0300	[thread overview]
Message-ID: <83pnlbu731.fsf@gnu.org> (raw)
In-Reply-To: <D8E164E0-D53C-412B-9DA4-24813A146A52@scratch.space> (message from VanL on Sun, 11 Aug 2019 15:26:50 +1000)

> From: VanL <van@scratch.space>
> Date: Sun, 11 Aug 2019 15:26:50 +1000
> Cc: 36927@debbugs.gnu.org
> 
> >> A possible way is to wait for all the subprocess sections to be done and to post-process on the fully collected buffer having lines like
> > 
> > I considered this possibility, but rejected it: since we display the
> > man page even before the async subprocess exits, it would mean that
> > the displayed portion is not fontified until the very end, which would
> > render the entire idea of formatting in the background useless, and
> > would have use behave on all platforms as we do on MS-DOS.
> 
> Graphics engines use a two buffer mechanism for collecting unfinished pixels backstage in one buffer and displaying finished pixels on stage in the ready buffer and then they alternate in their role.

We already do that in man.el, effectively: a new chunk of the text is
first fontified and buttonized, and only afterwards displayed.

> Not having all the `#include' lines buttonized is like missing a main character on stage.

I never said anything to the contrary.  I've just spent several hours
of my time fixing that.

> Of the 40 `(defun' signatures in man.el the following 6 hint of opportunities in the event model's pipeline to place buttons in at 
> 
>   2) `update' or 6) `cleanup' before 3) `notify-when-ready'
> 
> 1) on line 1075: (defun Man-getpage-in-background (topic)

That's the top-level function which does the job of, among other
things, fontifying and buttonizing the man page.

> 2) on line 1127: (defun Man-update-manpage ()

Not relevant to the issue at hand: this command is for a different use
case, and it invokes 'man' synchronously anyway.

> 3) on line 1154: (defun Man-notify-when-ready (man-buffer)

This one is called when the 'man' program exits, so waiting until that
time will eliminate the advantage of running 'man' in the background.
Irrelevant.

> 4) on line 1262: (defun Man-highlight-references (&optional xref-man-type)
> 5) on line 1289: (defun Man-highlight-references0 (start-section regexp button-pos target type)

These two are the ones where I made the changes (the first one just
calls the second one).

> 6) on line 1331: (defun Man-cleanup-manpage (&optional interactive)     

Isn't called in the scenario discussed here, so is irrelevant.

Not sure where you are trying to take this discussion.





  reply	other threads:[~2019-08-11 14:09 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
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 [this message]
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=83pnlbu731.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.