Skip to content

Move the size check to be the first thing checked in CFAllocator functions #4641

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 30, 2023

Conversation

AZero13
Copy link
Contributor

@AZero13 AZero13 commented Oct 2, 2022

This saves us a lot of work is size is 0

@AZero13 AZero13 force-pushed the CFBase branch 2 times, most recently from 64f41f1 to e744ada Compare October 2, 2022 17:45
@parkera
Copy link
Contributor

parkera commented Oct 13, 2022

Does this really save us 'a lot of work'?

@AZero13
Copy link
Contributor Author

AZero13 commented Jan 28, 2023

It saves some work @parkera

@AZero13 AZero13 force-pushed the CFBase branch 2 times, most recently from ec41427 to 7182370 Compare January 29, 2023 01:33
@AZero13
Copy link
Contributor Author

AZero13 commented Mar 1, 2023

Any update on this @parkera

@parkera
Copy link
Contributor

parkera commented Mar 31, 2023

@swift-ci test

@AZero13 AZero13 changed the title Move the size check to be the first thing checked in CFAllocatorCreate Move the size check to be the first thing checked in CFAllocator functions May 12, 2023
@AZero13
Copy link
Contributor Author

AZero13 commented May 12, 2023

@parkera Can we try this one more time?

@parkera
Copy link
Contributor

parkera commented May 17, 2023

@swift-ci test

@AZero13
Copy link
Contributor Author

AZero13 commented May 20, 2023

@parkera I apologize. I did not account for the malloc_zone tests. That has been fixed. Can we try one more time?

…tions

This saves us a lot of work is size is 0
@parkera
Copy link
Contributor

parkera commented May 22, 2023

@swift-ci test

@AZero13
Copy link
Contributor Author

AZero13 commented May 23, 2023

@parkera this passed!

@parkera parkera merged commit b70418c into swiftlang:main May 30, 2023
@AZero13 AZero13 deleted the CFBase branch May 30, 2023 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants