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

Change category code of hash sign (#) to other in the \markdownInput command #181

Merged
merged 1 commit into from
Aug 29, 2022

Conversation

Witiko
Copy link
Owner

@Witiko Witiko commented Aug 29, 2022

Previously, plain TeX output produced by the \markdownInput command was placed to a temporary file and then input, which has caused TeX to tokenize it lazily. With the lt3luabridge library, all plain TeX output produced by the \markdownInput command is first tokenized by TeX and then placed to TeX's input stream. This is desirable, because it avoids temporary files and helps with #129. However, it also breaks all outputs that contain hash signs (#), because TeX will incorrectly parse them as parameters. This pull request makes the category code of hash signs (#) to other (12) in the plain TeX output produced by the \markdownInput command.

@Witiko Witiko added bug plaintex Related to the plain TeX interface and implementation lt3luabridge Related to the lt3luabridge library automerge This pull request will be automatically merged after continuous integration has succeeded labels Aug 29, 2022
@Witiko Witiko force-pushed the fix/hash-signs branch 2 times, most recently from 626a283 to 1cce9cb Compare August 29, 2022 14:17
@Witiko Witiko merged commit 31f1d9c into main Aug 29, 2022
@Witiko Witiko deleted the fix/hash-signs branch August 29, 2022 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge This pull request will be automatically merged after continuous integration has succeeded bug lt3luabridge Related to the lt3luabridge library plaintex Related to the plain TeX interface and implementation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant