Skip to content

Conversation

@miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Jul 8, 2020

The running loop holder cache variable was always set to NULL when
calling set_running_loop.

Now set_running_loop saves the newly created running loop holder in the
cache variable for faster access in get_running_loop.

Automerge-Triggered-By: @1st1
(cherry picked from commit 529f426)

Co-authored-by: Tony Solomonik [email protected]

https://bugs.python.org/issue41247

Automerge-Triggered-By: @1st1

…onGH-21401)

The running loop holder cache variable was always set to NULL when
calling set_running_loop.

Now set_running_loop saves the newly created running loop holder in the
cache variable for faster access in get_running_loop.

Automerge-Triggered-By: @1st1
(cherry picked from commit 529f426)

Co-authored-by: Tony Solomonik <[email protected]>
@miss-islington
Copy link
Contributor Author

@tontinton: Status check is done, and it's a failure ❌ .

@tontinton
Copy link
Contributor

tontinton commented Jul 8, 2020

@tontinton: Status check is done, and it's a failure ❌ .

I've created a branch with the same fix only on 3.8, but github has issues right now, so I'll update when the PR is up, I will probably edit this comment with the PR link or something

PR: #21406

@1st1 1st1 closed this Jul 8, 2020
@1st1 1st1 reopened this Jul 8, 2020
@miss-islington
Copy link
Contributor Author

@tontinton: Status check is done, and it's a failure ❌ .

2 similar comments
@miss-islington
Copy link
Contributor Author

@tontinton: Status check is done, and it's a failure ❌ .

@miss-islington
Copy link
Contributor Author

@tontinton: Status check is done, and it's a failure ❌ .

@miss-islington miss-islington deleted the backport-529f426-3.8 branch July 8, 2020 23:01
@miss-islington
Copy link
Contributor Author

@tontinton: Status check is done, and it's a failure ❌ .

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.

5 participants