Skip to content

test: let's not dance on the edge of the JIT stack limit#9211

Merged
kubawerlos merged 2 commits intoPHP-CS-Fixer:masterfrom
6b7562617765726c6f73:lets-not-dance-on-the-edge-of-the-JIT-stack-limit
Nov 22, 2025
Merged

test: let's not dance on the edge of the JIT stack limit#9211
kubawerlos merged 2 commits intoPHP-CS-Fixer:masterfrom
6b7562617765726c6f73:lets-not-dance-on-the-edge-of-the-JIT-stack-limit

Conversation

@kubawerlos
Copy link
Copy Markdown
Member

When the value is 117, it results in an error "JIT stack limit exhausted".

Covering in tests 32-dimensional arrays is enough.

@kubawerlos kubawerlos enabled auto-merge (squash) November 22, 2025 06:37
@coveralls
Copy link
Copy Markdown

coveralls commented Nov 22, 2025

Coverage Status

coverage: 93.77%. remained the same
when pulling 1e6940f on 6b7562617765726c6f73:lets-not-dance-on-the-edge-of-the-JIT-stack-limit
into bf8548d on PHP-CS-Fixer:master.

@kubawerlos kubawerlos merged commit b92892b into PHP-CS-Fixer:master Nov 22, 2025
31 of 32 checks passed
@kubawerlos kubawerlos deleted the lets-not-dance-on-the-edge-of-the-JIT-stack-limit branch November 22, 2025 06:57
@mlocati
Copy link
Copy Markdown
Contributor

mlocati commented Nov 25, 2025

I hoped that this PR fixed issues with the two IDEs I use (VSCode and Eclipse PDT), but it's not the case... I still have

It seems the offending file is tests/Fixtures/Integration/misc/issue_8828_c.test-out.php

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants