Skip to content

Tidy some usage of py.from_borrowed_ptr and py.from_borrowed_ptr_or_opt#3877

Merged
davidhewitt merged 2 commits intoPyO3:mainfrom
LilyFirefly:tidy-from-borrowed-ptr
Feb 23, 2024
Merged

Tidy some usage of py.from_borrowed_ptr and py.from_borrowed_ptr_or_opt#3877
davidhewitt merged 2 commits intoPyO3:mainfrom
LilyFirefly:tidy-from-borrowed-ptr

Conversation

@LilyFirefly
Copy link
Contributor

No description provided.

@codspeed-hq
Copy link

codspeed-hq bot commented Feb 20, 2024

CodSpeed Performance Report

Merging #3877 will not alter performance

Comparing LilyFoote:tidy-from-borrowed-ptr (4de8b1a) with main (61bc02d)

Summary

✅ 67 untouched benchmarks

Copy link
Member

@davidhewitt davidhewitt left a comment

Choose a reason for hiding this comment

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

Thanks! I have a few suggestions here mostly related to skipping unnecessary reference counting :)

@davidhewitt davidhewitt added the CI-skip-changelog Skip checking changelog entry label Feb 20, 2024
@LilyFirefly LilyFirefly force-pushed the tidy-from-borrowed-ptr branch from c21d9b5 to c602d8d Compare February 20, 2024 10:10
Copy link
Member

@davidhewitt davidhewitt left a comment

Choose a reason for hiding this comment

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

LGTM! 👍

@davidhewitt davidhewitt added this pull request to the merge queue Feb 22, 2024
Merged via the queue into PyO3:main with commit 22a23ff Feb 23, 2024
@LilyFirefly LilyFirefly deleted the tidy-from-borrowed-ptr branch February 23, 2024 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI-skip-changelog Skip checking changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants