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

Removing the "Unused Entity" warning in WEPopoverController.m #31

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jfradj
Copy link

@jfradj jfradj commented Jan 4, 2012

Adding inline compiler instructions to avoid the following warning:

"WEPopoverController.m: warning: Unused Entity Issue: Property access result unused - getters should not be used for side effects"

"WEPopoverController.m: warning: Unused Entity Issue: Property access result unused - getters should not be used for side effects"
@Nektarios-zz
Copy link

Instead of patching this using pragmas, a cleaner alternative is to just change:

contentViewController.view

to

[contentViewController view];

@jfradj
Copy link
Author

jfradj commented Jan 27, 2012

Indeed, but why make things simple when they can be complicated... :-x
I think that was because I had just learned how to use compiler pragma ;-)

@Nektarios-zz
Copy link

They're both valid don't feel bad or anything, we're all learning and I appreciate you offering a patch

On Jan 27, 2012, at 5:32 AM, Johann [email protected] wrote:

Indeed, but why make things simple when they can be complicated... :-x
I think that was because I had just learned how to use compiler pragma ;-)


Reply to this email directly or view it on GitHub:
#31 (comment)

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.

2 participants