On Tue, 29 Oct 2024 at 06:13, user202729--- via luatex <luatex@tug.org>
wrote:
> I notice that the `run_put_next` function inserts the token list using
>
> begin_token_list(h,0);
>
> however, this has the problem that in `end_token_list`, the token list is
> not freed when the type is `0` (i.e. `parameter`).
>
> I think the `0` should be changed to e.g. `inserted` (4), so that it is
> correctly freed.
>
> For a test case, consider the following program.
>
> ```
> lualatex '\directlua{i=1; t=token.get_next()}~\def~#1A{\directlua{i=i+1;if
> i&1023==0 then print(i) end;for i=1,1000 do token.put_next(t); end}A}~A'
> </dev/null
> ```
>
> Before the change, the memory usage grows steadily. After the change, the
> memory usage remains constant.
>
> Please take a look. Thank you.
>
>
commit cd4888ec08e37cf0beae04be0a904d1df17eac41
Author: Luigi Scarso <luigi.sca...@gmail.com>
Date: Sun Nov 3 13:17:40 2024 +0100
Fixed a memory leak in token.put_next() and token.set_macro() (thanks
to user202...@protonmail.com)
(also in texlive)
--
luigi
