Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Validation of PDF output fails #39

Closed
ascherer opened this issue Jun 15, 2022 · 6 comments
Closed

Validation of PDF output fails #39

ascherer opened this issue Jun 15, 2022 · 6 comments
Labels

Comments

@ascherer
Copy link
Owner

ascherer commented Jun 15, 2022

I have noticed on macOS with Adobe Acrobat Reader that output from CWEB 5.0 with make all PDF=pdf fullmanual fails to display any “starred sections”. Now I try to find the cause of the error with CWEB 4.8 (current) and CWEB 3.64c (reference).

Use 3-Heights PDF Validator Online Tool on the CWEB sources (plus user manual).

CWEB 3.64c

Screenshot_20220615_192325
Screenshot_20220615_192408
Screenshot_20220615_192433
Screenshot_20220615_192508

CWEB 4.8

Screenshot_20220615_193139
Screenshot_20220615_193159
Screenshot_20220615_193219
Screenshot_20220615_193243

CWEB 5.0

Screenshot_20220615_191857
Screenshot_20220615_191946
Screenshot_20220615_192035
Screenshot_20220615_192109

@ascherer
Copy link
Owner Author

Another online validator produces long error reports

@ascherer
Copy link
Owner Author

ascherer commented Jun 16, 2022

The “unexpected token” came from an \ifpdf block outside of \ifpdftex. In the olden days, \pdftextrue did not also mean \pdftrue, but in CWEB 5.0, iftex.sty sets \pdftrue for the PDFTeX engine if \pdfoutput=1 (the default).

@ascherer
Copy link
Owner Author

Note that this bug was introduced in version 3.64 almost twenty years ago!

@ascherer
Copy link
Owner Author

Here's the fix in CWEB 4.9/5.0.

@ascherer ascherer added the bug label Jun 16, 2022
@ascherer
Copy link
Owner Author

ascherer commented Jun 17, 2022

And here's the fix in CWEB 4.9/5.0 for the incorrect Last keys.

@ascherer
Copy link
Owner Author

And here's the fix for CTWILL 5.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant