-
Notifications
You must be signed in to change notification settings - Fork 801
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 show{view,layer,image,imageref} into visualize #25
Conversation
Awesome! I like this. Can you add updating the README to the diff please? @arigrant What do you think? |
@mattjgalloway Thanks for reminding me about it. |
if fb.evaluateBooleanExpression('(BOOL)((unsigned long)CFGetTypeID((CFTypeRef)' + object + ') == (unsigned long)CGImageGetTypeID())'): | ||
_showImage('(id)[UIImage imageWithCGImage:' + object + ']') | ||
else: | ||
comarisonStr = '(BOOL)[' + object + 'isKindOfClass:[{} class]]' |
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.
The (BOOL)
cast is unnecessary with the use of evaluateBooleanExpression
.
Thanks @antons 🔨 I have a couple comments, after that I'm 👍 |
@kastiglione Good idea about the error message. Nice to know |
if fb.evaluateBooleanExpression('(unsigned long)CFGetTypeID((CFTypeRef)' + object + ') == (unsigned long)CGImageGetTypeID()'): | ||
_showImage('(id)[UIImage imageWithCGImage:' + object + ']') | ||
else: | ||
isKindOfClassStr = '[' + object + 'isKindOfClass:[{} class]]' |
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.
Do you mind adding an NSObjectHelpers
file and making an isKindOfClass
method? Other than that I am so happy for this and totally psyched. Thanks for doing this!
Yeah! |
Looks good to me. Let's take it. Thanks! |
Merged show{view,layer,image,imageref} into visualize
As suggested in #11 and #17, I merged
show{view,layer,image,image ref}
intovisualize
.