Skip to content

Conversation

@aisk
Copy link
Contributor

@aisk aisk commented Oct 31, 2025

Copy link
Member

@serhiy-storchaka serhiy-storchaka left a comment

Choose a reason for hiding this comment

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

LGTM. 👍

@serhiy-storchaka
Copy link
Member

Please rebase this on 3.14.

Comment on lines -774 to -776
The object also support equality comparison semantics, so handle objects will
compare equal if they both reference the same underlying Windows handle value.
Closed handle objects (those with a handle value of zero) always compare equal.
Copy link
Member

Choose a reason for hiding this comment

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

This should only be removed in 3.14, isn't?

Please rebase the branch on 3.14 (in Git and in the GitHub interface) or create a new PR for 3.14.

@aisk
Copy link
Contributor Author

aisk commented Nov 4, 2025

Sorry, I just messed things up. I was trying to rebase this change onto the 3.14 branch, but there were too many conflicts.
So I ran git reset --hard 3.14, then cherry-picked the commit in this PR, and did a git push -f to this branch.

GitHub may have thought I reverted all the changes between this commit (around the main branch) and 3.14, and automatically requested all the code owners whose files were affected as reviewers 😂

Really sorry for making such a noise. Will close this PR and make a new PR based on 3.14 branch.

@aisk aisk closed this Nov 4, 2025
@aisk aisk deleted the hkey-compare-doc branch November 4, 2025 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting merge docs Documentation in the Doc dir needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes skip news

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants