[bug #45034] [PATCH] mdoc(7): add support for the mdocmx(7) reference extension

2021-05-25 Thread anonymous
Follow-up Comment #16, bug #45034 (project groff): In the meantime less got support for OSC 8 "hyperlinks in terminal", and i changed the three lines in mdocmx to create \X'tty: osc8 ...' for the TTY output device instead. Please find attached a patch for grotty that implements \X'tty: osc8' sequ

[bug #60666] [grotty] add OSC 8 support for hyperlinks

2021-05-25 Thread anonymous
Follow-up Comment #1, bug #60666 (project groff): Great! Let me attach my final patches to grotty and less (your linked one does not yet securely quote content). It would be great to see OSC 8 addition to grotty, not unlikely converters like docbook etc. would start using it. (I am not subscribed.

[bug #60672] troff: No warning about ".el \{"

2021-05-25 Thread Bjarni Ingi Gislason
URL: Summary: troff: No warning about ".el \{" Project: GNU troff Submitted by: bjarniig Submitted on: Tue 25 May 2021 07:23:28 PM UTC Category: Core Severity: 3 - Nor

[bug #60389] [PATCH] contrib/mm/m.tmac: omit a comma, if macro "let@sg_SP" is called with one argument

2021-05-25 Thread Bjarni Ingi Gislason
Follow-up Comment #2, bug #60389 (project groff): A backslash (reverse solidus) is missing after '\{' .ie '\\$2'' \{\ . misc@toupper "\\$1" .\} -.el \{ +.el \{\ . misc@toupper "\\$1, \\$2" .\} .if \\$4 .sp ( ___ Reply to this it

[bug #60672] troff: No warning about ".el \{"

2021-05-25 Thread Dave
Update of bug #60672 (project groff): Item Group:None => Warning/Suspicious behaviour ___ Follow-up Comment #1: [comment #0 original submission:] > Related reports are #59434, #45502, and #60260. Making

[bug #60665] doc/groff.texi: clarify nroff adjustment algorithm differences

2021-05-25 Thread Dave
Follow-up Comment #2, bug #60665 (project groff): [comment #1 comment #1:] > Maybe troff devices always have a horizontal resolution greater than > one-twelfth of an em, and that is the smallest amount of distributable > inter-word space we can manipulate with the .ss request. That's slightly off

[bug #60665] groff.texi, groff_diff.7.man: clarify nroff adjustment algorithm differences

2021-05-25 Thread Dave
Update of bug #60665 (project groff): Summary: doc/groff.texi: clarify nroff adjustment algorithm differences => groff.texi, groff_diff.7.man: clarify nroff adjustment algorithm differences ___ Follow-up Comment #3: And I

[bug #60673] adjustment algorithm should disregard nonadjusted lines in its alternation pattern

2021-05-25 Thread Dave
URL: Summary: adjustment algorithm should disregard nonadjusted lines in its alternation pattern Project: GNU troff Submitted by: barx Submitted on: Tue 25 May 2021 10:47:19 PM CDT Ca

[bug #57836] grotty's adjustment algorithm is not diversion-aware

2021-05-25 Thread Dave
Follow-up Comment #2, bug #57836 (project groff): [comment #1 comment #1:] > But if that turns out to be wrong, I can open a new bug report > for this manifestation. I believe it has in fact turned out to be wrong, based on the code snippet Branden posted in bug #60665; accordingly, I've opened t

[bug #60672] troff: No warning about ".el \{"

2021-05-25 Thread G. Branden Robinson
Update of bug #60672 (project groff): Status:None => Invalid Assigned to:None => gbranden Open/Closed:Open => Closed

[bug #60672] troff: No warning about ".el \{"

2021-05-25 Thread G. Branden Robinson
Follow-up Comment #3, bug #60672 (project groff): Quoting verbatim stuff angered Savannah. Let's try that again. Hi Bjarni, Thanks for the report, but I'm afraid it's invalid. [comment #0 original submission:] > Subject: troff: No warning about ".el \{" > > Example: > printf '.pl 20v > .nr

[bug #60666] [grotty] add OSC 8 support for hyperlinks

2021-05-25 Thread G. Branden Robinson
Update of bug #60666 (project groff): Status:None => In Progress ___ Follow-up Comment #2: Thanks, Steffen! I'm inlining your groff patch; I have comments on it. I'd like to share my thin

[bug #60673] adjustment algorithm should disregard nonadjusted lines in its alternation pattern

2021-05-25 Thread Dave
Follow-up Comment #1, bug #60673 (project groff): Based on aforementioned code snippet in bug #60665, it looks like fixing this may be as simple as three lines of code: diff --git a/src/roff/troff/env.cpp b/src/roff/troff/env.cpp index 1be86470..312c0f7d 100644 --- a/src/roff/troff/env.cpp +++ b

[bug #60666] [grotty] add OSC 8 support for hyperlinks

2021-05-25 Thread G. Branden Robinson
Follow-up Comment #3, bug #60666 (project groff): My design and implementation thoughts. 1. As suggested earlier, I think we want two device control keywords. One, "osc8" to enable/disable this feature. The other, to actual direct the output of the link, I think we should call something more ob

[bug #60666] [grotty] add OSC 8 support for hyperlinks

2021-05-25 Thread G. Branden Robinson
Follow-up Comment #4, bug #60666 (project groff): Sorry, forgot to preview--that thing was riddled with typos. Second try. My design and implementation thoughts. 1. As suggested earlier, I think we want two device control keywords. One, "osc8" to enable/disable this feature. The other, to act