[ 
https://issues.apache.org/jira/browse/CB-9702?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14978954#comment-14978954
 ] 

Perry Stoll commented on CB-9702:
---------------------------------

Jose,

I suspect this change has not been released yet. It is not fatal, it is just a 
warning. And the stated behavior (those condition evaluations being true) is 
safe in this case. 

But this cleanup was a reminder - never ignore compiler warnings!

Perry



> Fix plugin initialization based on XCode 7.0 compiler warnings
> --------------------------------------------------------------
>
>                 Key: CB-9702
>                 URL: https://issues.apache.org/jira/browse/CB-9702
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: Plugin Network Information
>    Affects Versions: Master
>         Environment: XCode Version 7.0 (7A220)
>            Reporter: Perry Stoll
>
> xcode complains "These will always evaluate to true", which, looking at these 
> lines, makes sense. These things are objects, taking their address will 
> always yield a non-null value. 
> {code}
> cordova-plugin-network-information pstoll$ git diff
> diff --git a/src/ios/CDVConnection.m b/src/ios/CDVConnection.m
> index 3749767..223dd7c 100644
> --- a/src/ios/CDVConnection.m
> +++ b/src/ios/CDVConnection.m
> @@ -118,7 +118,7 @@
>      [self.internetReach startNotifier];
>      [[NSNotificationCenter defaultCenter] addObserver:self 
> selector:@selector(updateConnectionType:)
>                                                   
> name:kReachabilityChangedNotification object:nil];
> -    if (&UIApplicationDidEnterBackgroundNotification && 
> &UIApplicationWillEnterForegroundNotification) {
> +    if (UIApplicationDidEnterBackgroundNotification && 
> UIApplicationWillEnterForegroundNotification) {
>          [[NSNotificationCenter defaultCenter] addObserver:self 
> selector:@selector(onPause) name:UIApplicationDidEnterBackgroundNotification 
> object:nil];
>          [[NSNotificationCenter defaultCenter] addObserver:self 
> selector:@selector(onResume) 
> name:UIApplicationWillEnterForegroundNotification object:nil];
>      }
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to