Skip to content

Conversation

@stefanor
Copy link
Contributor

Missed in GH-139914, as it was introduced around the same time and is only relevant to pydebug builds.

Python/ceval.c Outdated
assert(base<top);
assert((top-base) >= _PyOS_MIN_STACK_SIZE);
#else
assert(base <= top);
Copy link
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, that's bogus.

We were seeing:
_freeze_module: ../Python/ceval.c:498: tstate_set_stack: Assertion base < top' failed.`
But there must be more to that, because that assert was obviously correct.

Copy link
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed the commit to what I think should be correct.

Let me just try to reproduce the problem on the 3.14 branch...

Missed in pythonGH-139914, as it was introduced around the same time and is only relevant to pydebug builds.
@stefanor
Copy link
ContributorAuthor

Sorry, don't know what I was thinking.

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

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants

@stefanor@StanFromIreland