[Date Prev][Date Next][Thread Prev][Thread Next]
[Date Index]
[Thread Index]
- Subject: Re: LPEG on Solaris
- From: Roberto Ierusalimschy <roberto@...>
- Date: Wed, 1 Jun 2016 11:14:43 -0300
> 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