Re: LPEG on Solaris

[Roberto Ierusalimschy <roberto@...>]

> here is a PR for lpeg that fixes some issues we have had with deploying the
> it on Solaris / OmniOS.
> Please consider merging this upstream.
> 
> https://github.com/LuaDist/lpeg/pull/1
> 
> """
> On operating systems like Solaris and Illumos where the stack grows down,
> lightuserdata and user data cannot be assigned from the stack. This commit
> patch fixes this behaviour by moving the allocations to the heap (using
> calloc).

Sorry, but this does not seem to make sense. Of course we can allocate
and use an automatic array in C no matter the direction the stack
grows. Your fix probably only hides the bug, instead of fixing it.

Do you have a simple example that exercises the bug?

-- Roberto