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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Get Swifty - Update everything to use ObjC++ or Swift contexts explicitly #121

Merged
merged 5 commits into from
Nov 15, 2015

Conversation

ultramiraculous
Copy link

Adding a po-like shortcuts to run expression in either a Swift (pswift) or ObjC++ (pobjc) context, rather than use the language of the current frame. Currently if you're trying do do something like po [UIViewController _printHierarchy] and you're not in an ObjC frame, the expression won't parse/run. It's potentially useful if you're trying to re-use commands and you're potentially going to change between languages in the meantime.

Mainly trying to fix commands like pvc that end up on the fast track to Failtown when run from a breakpoint in Swift code.

@ultramiraculous ultramiraculous changed the title Commands for running expressions in ObjC++ or Swift explicitly Commands for running print expressions in ObjC++ or Swift explicitly Oct 15, 2015
@ultramiraculous
Copy link
Author

Also probably resolves #80.

@tomaskraina
Copy link

👍

@ultramiraculous ultramiraculous changed the title Commands for running print expressions in ObjC++ or Swift explicitly Get Swifty - Update everything to use ObjC++ or Swift explicitly Oct 23, 2015
@ultramiraculous ultramiraculous changed the title Get Swifty - Update everything to use ObjC++ or Swift explicitly Get Swifty - Update everything to use ObjC++ or Swift contexts explicitly Oct 23, 2015
@ignat980
Copy link

How long does it take to merge a fix? This is a very useful plugin and I'd like for it to work within swift contexts.

arigrant added a commit that referenced this pull request Nov 15, 2015
Get Swifty - Update everything to use ObjC++ or Swift contexts explicitly
@arigrant arigrant merged commit 9e25da6 into facebook:master Nov 15, 2015
@arigrant
Copy link
Contributor

This is awesome. Thanks!

@ultramiraculous
Copy link
Author

(Y)

@tomaskraina
Copy link

Don't you guys want to make a release (1.4.0?) so we can install/update this using brew intall chisel? :) Let me know if I can help with that.

@kastiglione
Copy link
Contributor

Thanks @tomaskraina – I'll get right on that.

@tomaskraina
Copy link

Thank you!

On 10 Feb 2016, at 05:25, Dave Lee notifications@github.com wrote:

Thanks @tomaskraina – I'll get right on that.


Reply to this email directly or view it on GitHub.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants