Skip to content

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