-
-
Notifications
You must be signed in to change notification settings - Fork 0
R3 stack limits too low #1252
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
Comments
@carls I'm quite puzzled with this one. It is working as expected (throwing the error) when I compile using |
Ok... I found it... the reason is, that the binary stack is overflowed sooner than the one checked in Rebol. The difference is, that my With Setting limit using mentioned Anyway.. In my 64bit builds I'm getting |
1. Using `Trap_Stack` instead of `Trap0(RE_STACK_OVERFLOW)` as the second one displays `near` information, which may not be good to show. 2. Don't try to expand stack over limit. Related to: Oldes/Rebol-issues#1252
Submitted by: pekr
While we were told R3 stack limits are user settable, they are too low by default, and should be imo adjusted to some more usefull level:
REBOL level recursion:
Parse level recursion:
Imported from: CureCode [ Version: alpha 86 Type: Issue Platform: Windows Category: Unspecified Reproduce: Always Fixed-in:alpha 87 ]
Imported from: metaeducation#1252
Comments:
Submitted by: Carl
The stack size needs to be balanced against the memory footprint you want.
Currently, it's set for a small footprint. However, it can be expanded in A87 with this:
The limit here is imposed by the C stack, which in A87 reserves about 4MB.
The PARSE function is now limited only by the C stack, not by a depth counter.
Submitted by: pekr
It is nice we can set the stack higher, but I thought that we will make it so (at least to R2 level) by default. Is there really a reason, why R3 default stack setting is nearly 1/4 of the R2 value? Also disabling entire security in order to raise the stack value is surely not a good idea (more in #1259 )
The text was updated successfully, but these errors were encountered: