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

MAS_ATTR_FORWARD(attribute) raises a warning if mismatched-return-types is enabled #71

Closed
vicpenap opened this issue May 22, 2014 · 7 comments

Comments

@vicpenap
Copy link

If the flag -Wmismatched-return-types is set, the new property attribute from MASShorthandAdditions introduced in v0.5.0 will raise a warning when forwarding it to the non-shorthand equivalent.

This is the output from the compiler:

conflicting return type in implementation of 'attribute': 'MASViewAttribute *(^)(NSLayoutAttribute)' vs 'MASViewAttribute *' [-Wmismatched-return-types]
MAS_ATTR_FORWARD(attribute);
@lluisgerard
Copy link
Contributor

That's interesting, I once added a C library (https://github.com/JeremyOT/ObjectiveZlib) into my project and started to have this issue, I finally found out going back into my own commits. This library has a ObjectiveZlib-Prefix.pch file and maybe that was the problem. But I'm sorry, because I finally don't have to use it and don't know why it happened. Just saying in case it rings you a bell.

@vicpenap
Copy link
Author

Unfortunately it doesn't. Thanks anyway, @lluisgerard :)

@cloudkite
Copy link
Contributor

@vicpenap Sorry for the late reply! should be fixed in the latest commit 1e08fa3

@vicpenap
Copy link
Author

It is! Thanks a lot, @cloudkite!

@sibljon
Copy link
Contributor

sibljon commented May 29, 2014

@cloudkite Thanks for fixing so fast!

Would it be possible for you to tag 1e08fa3 and do a CocoaPods point release?

@cloudkite
Copy link
Contributor

No problem, v0.5.2 should be up now :)

@sibljon
Copy link
Contributor

sibljon commented May 29, 2014

🆒 !! Thanks so much!

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

No branches or pull requests

4 participants