Skip to content

Add luatexbase.new_luacommand (#2191) - #2192

Merged
FrankMittelbach merged 2 commits into
latex3:developfrom
Udi-Fogiel:develop
Oct 17, 2026
Merged

FrankMittelbach merged 2 commits into
latex3:developfrom
Udi-Fogiel:develop

Conversation

@Udi-Fogiel

Copy link
Copy Markdown
Contributor

Wraps new_luafunction and token.set_lua into a single call for binding a Lua function to a control sequence.

READ ME FIRST: Please understand that in most cases we will not be able to merge a pull request because there are a lot of internal activities needed when updating the LaTeX2e sources. If you have a code suggestion please discuss it with the team first.

Internal housekeeping

Status of pull request

  • Feedback wanted

Checklist of required changes before merge will be approved

  • Test file(s) added
  • Version and date string updated in changed source files
  • Relevant \changes entries in source included
  • Relevant changes.txt updated
  • Rollback provided (if necessary)?
  • ltnewsX.tex (and/or latexchanges.tex) updated

Wraps new_luafunction and token.set_lua into a single call for
binding a Lua function to a control sequence.
@davidcarlisle

Copy link
Copy Markdown
Member

I think I'm generally in favour of doing something like this, back when we added new-luafunction in lt;uatex it was rather new in LuaTeX and we didn't really have any examples and I've thought before the interface in ltluatex wasn't quite right having to do more than is ideal with managing the underlying table index.

@FrankMittelbach FrankMittelbach left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no real opinion, that is for others to approve or disaprove, but as far as I can see it looks fine (which name to use may be a question).

@FrankMittelbach
FrankMittelbach merged commit 0c79069 into latex3:develop Oct 17, 2026
55 checks passed
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.

5 participants