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
Update code to work for Xcode 10b1 #4729
Conversation
@@ -2537,12 +2537,12 @@ - (NSUInteger)countByEnumeratingWithState:(NSFastEnumerationState *)state | |||
return [_array countByEnumeratingWithState:state objects:buffer count:len]; | |||
} | |||
|
|||
- (void)enumerateObjectsUsingBlock:(void (^)(id obj, NSUInteger idx, BOOL *stop))block { | |||
- (void)enumerateObjectsUsingBlock:(void (NS_NOESCAPE ^)(id obj, NSUInteger idx, BOOL *stop))block { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update objectivec/README.md, i believe it says we just require Xcode 7.0, so this likely needs to be raised.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done, also updated the script
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, don't we also want the NO_ESCAPE in the header also so code calling this is correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And no we don't since these are just the hooked cases. We probably should add NOESCAPE to the GPB ones, but that can happen later.
* Update code to work for Xcode 10b * Update README and test scripts to mention that Xcode 7 is no longer supported
* Update code to work for Xcode 10b * Update README and test scripts to mention that Xcode 7 is no longer supported
…code 10b1 (protocolbuffers#4729) Update code to work for Xcode 10b1 (protocolbuffers#4729)
No description provided.