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

be LuaJIT-compatible #158

Merged
merged 1 commit into from
Dec 16, 2024
Merged

be LuaJIT-compatible #158

merged 1 commit into from
Dec 16, 2024

Conversation

RadioNoiseE
Copy link
Contributor

The // operator was introduced in Lua 5.3, but it is not available in the current version of LuaJIT. To ensure compatibility with LuaJITTeX, we can use math.floor as an alternative.

@dohyunkim
Copy link
Contributor

Would you please give me a minimal example that shows the issue?
There seems to be no problem in the patch, but I want to check it in practice.

@dohyunkim
Copy link
Contributor

ok. something like this:

\input luamplib.sty
\mplibcode
beginfig(1)
draw mpliboutlinetext.b
  ("\hbox to 5em{\TeX\dotfill}")
  ( withcolor blue)
  (
    withpen pencircle scaled .25
    withcolor red
  )
  scaled 5
  ;
endfig;
\endmplibcode
\end.

But ltluatex.tex should also be patched, and strange seg fault at the end of luajittex running.
Anyway, I'll merge the PR.

@dohyunkim dohyunkim merged commit c7a3848 into lualatex:master Dec 16, 2024
@RadioNoiseE
Copy link
Contributor Author

weird, segfault for me too.

most of my mp graphs work alright though.

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.

2 participants