Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add missing @autoreleasepool to objective-c at_keywords #1424

Merged
merged 2 commits into from Feb 11, 2020

Conversation

n-b
Copy link
Contributor

@n-b n-b commented Feb 2, 2020

reference:
Using Autorelease Pool Blocks https://developer.apple.com/library/archive/documentation/Cocoa/Conceptual/MemoryMgmt/Articles/mmAutoreleasePools.html

--

Hi! this is my first PR on this project. Thanks for building it 馃槂.

@pyrmont pyrmont added the needs-review The PR needs to be reviewed label Feb 5, 2020
@pyrmont pyrmont self-assigned this Feb 5, 2020
@pyrmont
Copy link
Contributor

pyrmont commented Feb 5, 2020

@n-b Thanks for the PR! Could you add an example of the use of @autoreleasepool to the visual sample in spec/visual/samples/objective_c and check that it works in the visual demo? (You can run this by running rackup and visiting localhost:9292.

@pyrmont pyrmont added author-action The PR has been reviewed but action by the author is needed and removed needs-review The PR needs to be reviewed labels Feb 5, 2020
@n-b
Copy link
Contributor Author

n-b commented Feb 6, 2020

@pyrmont done! It looks ok.

@pyrmont pyrmont merged commit 9a0f735 into rouge-ruby:master Feb 11, 2020
@pyrmont
Copy link
Contributor

pyrmont commented Feb 11, 2020

@n-b Thanks for this! We have a release of Rouge going out in the next hour or so and this will be part of that 馃帀

@pyrmont pyrmont removed the author-action The PR has been reviewed but action by the author is needed label Feb 11, 2020
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.

None yet

2 participants