Skip to content

Defuse the linebreaker in semantic pagination mode - #245

Merged
pkgw merged 7 commits into
tectonic-typesetting:masterfrom
pkgw:html-work-2
Oct 13, 2018
Merged

Defuse the linebreaker in semantic pagination mode#245
pkgw merged 7 commits into
tectonic-typesetting:masterfrom
pkgw:html-work-2

Conversation

@pkgw

@pkgw pkgw commented Oct 13, 2018

Copy link
Copy Markdown
Collaborator

Finally figured out the linebreaking algorithm well enough to think that I know the correct way to do this. The new functionality isn't testable as yet without hacking the code a bit — but if you hack the code to activate the semantic_pagination_enabled codepaths in xetex-linebreak.c without turning on the other ones, you can get an output PDF with zany paragraphs shooting off the edge of the page, as intended by this feature.

pkgw added 7 commits October 10, 2018 23:20
After much, much time spent trying to understand the linebreaking algorithm, I
think I am confident in the right way to modify it to activate semantic
pagination mode. Here it is! A quick test on xenia has visual results exactly
as I was hoping: a very "short" document with normal-looking paragraph lines
flying off the edge of the page. Huzzah!
@codecov

codecov Bot commented Oct 13, 2018

Copy link
Copy Markdown

Codecov Report

Merging #245 into master will increase coverage by 0.01%.
The diff coverage is 67.11%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #245      +/-   ##
==========================================
+ Coverage   39.87%   39.89%   +0.01%     
==========================================
  Files         133      133              
  Lines       63742    63785      +43     
==========================================
+ Hits        25419    25448      +29     
- Misses      38323    38337      +14
Impacted Files Coverage Δ
tectonic/xetex-ini.c 86.96% <ø> (ø) ⬆️
tectonic/xetex-xetexd.h 57.14% <ø> (ø) ⬆️
tectonic/xetex-xetex0.c 77.93% <ø> (+0.62%) ⬆️
tectonic/xetex-linebreak.c 88.05% <67.11%> (-5.37%) ⬇️
tectonic/dpx-pdfobj.c 32.07% <0%> (+0.1%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e469ff3...0312038. Read the comment docs.

@pkgw
pkgw merged commit 31b6aaa into tectonic-typesetting:master Oct 13, 2018
@pkgw
pkgw deleted the html-work-2 branch October 27, 2018 19:42
Mrmaxmeier pushed a commit to Mrmaxmeier/tectonic that referenced this pull request Jul 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant