diff options
author | Matthew Newhook <matthew@zeroc.com> | 2008-03-18 01:37:51 -0800 |
---|---|---|
committer | Matthew Newhook <matthew@zeroc.com> | 2008-03-18 01:37:51 -0800 |
commit | aa0fcb15afd952144967d873bfc7b86d93973192 (patch) | |
tree | e657bb26128e8b1ba27353504bdbe7d2b4e3da98 /cpp/src/IceGrid/ServerI.cpp | |
parent | Got rid of redundant #ifdef. (Bug 2429.) (diff) | |
download | ice-aa0fcb15afd952144967d873bfc7b86d93973192.tar.bz2 ice-aa0fcb15afd952144967d873bfc7b86d93973192.tar.xz ice-aa0fcb15afd952144967d873bfc7b86d93973192.zip |
http://bugzilla.zeroc.com/bugzilla/show_bug.cgi?id=2719
Squashed commit of the following:
commit 5ff1fac3e73ef4fb3ac96a692812a849ff585ebc
Author: Matthew Newhook <matthew@zeroc.com>
Date: Tue Mar 18 01:27:20 2008 -0800
Fixed the various && || warnings from gcc.
Diffstat (limited to 'cpp/src/IceGrid/ServerI.cpp')
-rw-r--r-- | cpp/src/IceGrid/ServerI.cpp | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/cpp/src/IceGrid/ServerI.cpp b/cpp/src/IceGrid/ServerI.cpp index f0a6772379d..4dc1d241882 100644 --- a/cpp/src/IceGrid/ServerI.cpp +++ b/cpp/src/IceGrid/ServerI.cpp @@ -133,7 +133,7 @@ descriptorWithoutRevisionEqual(const InternalServerDescriptorPtr& lhs, const Int return false; } - if(!lhs->distrib && rhs->distrib || lhs->distrib && !rhs->distrib) + if((!lhs->distrib && rhs->distrib) || (lhs->distrib && !rhs->distrib)) { return false; } @@ -877,7 +877,7 @@ ServerI::isAdapterActivatable(const string& id) const } if(_desc->activation == "manual" || - _desc->activation == "session" && _desc->sessionId.empty()) + (_desc->activation == "session" && _desc->sessionId.empty())) { return false; } @@ -1030,7 +1030,8 @@ ServerI::load(const AMD_Node_loadServerPtr& amdCB, const InternalServerDescripto // if(_desc && (replicaName != "Master" || _desc->sessionId == desc->sessionId) && - (_desc->uuid == desc->uuid && _desc->revision == desc->revision || descriptorWithoutRevisionEqual(_desc, desc))) + ((_desc->uuid == desc->uuid && _desc->revision == desc->revision) || + descriptorWithoutRevisionEqual(_desc, desc))) { if(_desc->revision != desc->revision) { @@ -1241,7 +1242,7 @@ ServerI::disableOnFailure() // is always and the server wasn't active at the time of the // failure we disable the server. // - if(_disableOnFailure != 0 || _activation == Always && (_state == Activating || _state == WaitForActivation)) + if(_disableOnFailure != 0 || (_activation == Always && (_state == Activating || _state == WaitForActivation))) { _previousActivation = _activation; _activation = Disabled; @@ -1258,8 +1259,8 @@ ServerI::enableAfterFailure(bool force) } if(force || - _disableOnFailure > 0 && - (_failureTime + IceUtil::Time::seconds(_disableOnFailure) < IceUtil::Time::now(IceUtil::Time::Monotonic))) + (_disableOnFailure > 0 && + (_failureTime + IceUtil::Time::seconds(_disableOnFailure) < IceUtil::Time::now(IceUtil::Time::Monotonic)))) { _activation = _previousActivation; _failureTime = IceUtil::Time(); |