Skip to content

Commit

Permalink
Delete line-drawing fixtures and remove ligatures
Browse files Browse the repository at this point in the history
The C/A/T didn't support line-drawing, and \D'...' sequences are treated
literally by ancient versions of troff(1), sans the leading backslash.

Ligatures are problematic because grops(1) uses absolute motions to draw
them, whereas dedicated glyphs were used by the C/A/T. There's no way of
fixing this, short of patching grops(1) to recognise "C ffi" as a single
character. Using multiple character sequences to "fake" ligatures proved
impossible because C/A/T commands are emitted boustrophedonically (where
movement/drawing sequences are reversed for every second line).

Rather than using crude hacks to "fix" this, we'll document it as a bug.
Luckily, Roff.js's canvas renderer isn't affected by this, as "C foobar"
is treated as "t foobar" without position changing (assuming "foobar" is
not a recognised character).
  • Loading branch information
Alhadis committed May 24, 2019
1 parent fb57d1c commit 3279ac7
Show file tree
Hide file tree
Showing 8 changed files with 0 additions and 38 deletions.
Binary file removed test/fixtures/cat/drawing.cat
Binary file not shown.
Binary file removed test/fixtures/cat/drawing.pdf
Binary file not shown.
34 changes: 0 additions & 34 deletions test/fixtures/cat/drawing.roff

This file was deleted.

4 changes: 0 additions & 4 deletions test/fixtures/cat/drawing.txt

This file was deleted.

Binary file modified test/fixtures/cat/v7-adb.cat
Binary file not shown.
Binary file modified test/fixtures/cat/v7-ed.cat
Binary file not shown.
Binary file modified test/fixtures/cat/v7-sh.cat
Binary file not shown.
Binary file modified test/fixtures/cat/v7-troff.cat
Binary file not shown.

0 comments on commit 3279ac7

Please sign in to comment.