Fix (ICE-7762) - Change dispatch bool return value

4 files changed, 26 insertions, 26 deletions

diff --git a/cpp/src/Ice/Object.cpp b/cpp/src/Ice/Object.cpp
index aa053d59d1d..0b4a3295126 100644
--- a/cpp/src/Ice/Object.cpp
+++ b/cpp/src/Ice/Object.cpp
@@ -137,7 +137,7 @@ Ice::Object::_iceD_ice_isA(Incoming& inS, const Current& current)
     OutputStream* ostr = inS.startWriteParams();
     ostr->write(ret);
     inS.endWriteParams();
-    return false;
+    return true;
 }
 
 bool
@@ -146,7 +146,7 @@ Ice::Object::_iceD_ice_ping(Incoming& inS, const Current& current)
     inS.readEmptyParams();
     ice_ping(current);
     inS.writeEmptyParams();
-    return false;
+    return true;
 }
 
 bool
@@ -157,7 +157,7 @@ Ice::Object::_iceD_ice_ids(Incoming& inS, const Current& current)
     OutputStream* ostr = inS.startWriteParams();
     ostr->write(&ret[0], &ret[0] + ret.size(), false);
     inS.endWriteParams();
-    return false;
+    return true;
 }
 
 bool
@@ -168,7 +168,7 @@ Ice::Object::_iceD_ice_id(Incoming& inS, const Current& current)
     OutputStream* ostr = inS.startWriteParams();
     ostr->write(ret, false);
     inS.endWriteParams();
-    return false;
+    return true;
 }
 
 
@@ -356,7 +356,7 @@ Ice::Blobject::_iceDispatch(Incoming& in, const Current& current)
     {
         in.writeParamEncaps(&outEncaps[0], static_cast<Ice::Int>(outEncaps.size()), ok);
     }
-    return false;
+    return true;
 }
 
 bool
@@ -376,7 +376,7 @@ Ice::BlobjectArray::_iceDispatch(Incoming& in, const Current& current)
     {
         in.writeParamEncaps(&outEncaps[0], static_cast<Ice::Int>(outEncaps.size()), ok);
     }
-    return false;
+    return true;
 }
 
 bool
@@ -404,7 +404,7 @@ Ice::BlobjectAsync::_iceDispatch(Incoming& in, const Current& current)
 #else
     ice_invoke_async(new ::IceAsync::Ice::AMD_Object_ice_invoke(in), vector<Byte>(inEncaps, inEncaps + sz), current);
 #endif
-    return true;
+    return false;
 }
 
 bool
@@ -426,5 +426,5 @@ Ice::BlobjectArrayAsync::_iceDispatch(Incoming& in, const Current& current)
 #else
     ice_invoke_async(new ::IceAsync::Ice::AMD_Object_ice_invoke(in), inEncaps, current);
 #endif
-    return true;
+    return false;
 }
diff --git a/cpp/src/slice2cpp/Gen.cpp b/cpp/src/slice2cpp/Gen.cpp
index b09983d0f45..c8bd53530e6 100644
--- a/cpp/src/slice2cpp/Gen.cpp
+++ b/cpp/src/slice2cpp/Gen.cpp
@@ -3081,12 +3081,12 @@ Slice::Gen::ObjectVisitor::visitOperation(const OperationPtr& p)
             {
                 C << nl << "inS.writeEmptyParams();";
             }
-            C << nl << "return false;";
+            C << nl << "return true;";
         }
         else
         {
             C << nl << "this->" << name << "_async" << argsAMD << ';';
-            C << nl << "return true;";
+            C << nl << "return false;";
         }
         C << eb;
     }
@@ -7144,7 +7144,7 @@ Slice::Gen::Cpp11InterfaceVisitor::visitOperation(const OperationPtr& p)
                 C << nl << "inS.writeEmptyParams();";
             }
         }
-        C << nl << "return false;";
+        C << nl << "return true;";
     }
     else
     {
@@ -7164,7 +7164,7 @@ Slice::Gen::Cpp11InterfaceVisitor::visitOperation(const OperationPtr& p)
             C << eb << ';';
         }
         C << nl << "this->" << opName << spar << args << epar << ';';
-        C << nl << "return true;";
+        C << nl << "return false;";
     }
     C << eb;
 }
diff --git a/cpp/src/slice2java/GenCompat.cpp b/cpp/src/slice2java/GenCompat.cpp
index ff344f57bf8..28d724f8646 100644
--- a/cpp/src/slice2java/GenCompat.cpp
+++ b/cpp/src/slice2java/GenCompat.cpp
@@ -1409,7 +1409,7 @@ Slice::JavaCompatVisitor::writeDispatchAndMarshalling(Output& out, const ClassDe
             {
                 out << nl << "inS.writeEmptyParams();";
             }
-            out << nl << "return false;";
+            out << nl << "return true;";
 
             out << eb;
         }
@@ -1501,7 +1501,7 @@ Slice::JavaCompatVisitor::writeDispatchAndMarshalling(Output& out, const ClassDe
                 out << ", ";
             }
             out << "current);";
-            out << nl << "return true;";
+            out << nl << "return false;";
 
             out << eb;
         }
diff --git a/cpp/test/Ice/interceptor/Client.cpp b/cpp/test/Ice/interceptor/Client.cpp
index ec263295058..bb1fe439030 100644
--- a/cpp/test/Ice/interceptor/Client.cpp
+++ b/cpp/test/Ice/interceptor/Client.cpp
@@ -192,21 +192,21 @@ ClientApp::run(const Test::MyObjectPrxPtr& prx, const InterceptorIPtr& intercept
     test(interceptor->getLastOperation().empty());
     prx->ice_ping();
     test(interceptor->getLastOperation() == "ice_ping");
-    test(!interceptor->getLastStatus());
+    test(interceptor->getLastStatus());
     string typeId = prx->ice_id();
     test(interceptor->getLastOperation() == "ice_id");
-    test(!interceptor->getLastStatus());
+    test(interceptor->getLastStatus());
     test(prx->ice_isA(typeId));
     test(interceptor->getLastOperation() == "ice_isA");
-    test(!interceptor->getLastStatus());
+    test(interceptor->getLastStatus());
     test(prx->add(33, 12) == 45);
     test(interceptor->getLastOperation() == "add");
-    test(!interceptor->getLastStatus());
+    test(interceptor->getLastStatus());
     cout << "ok" << endl;
     cout << "testing retry... " << flush;
     test(prx->addWithRetry(33, 12) == 45);
     test(interceptor->getLastOperation() == "addWithRetry");
-    test(!interceptor->getLastStatus());
+    test(interceptor->getLastStatus());
     cout << "ok" << endl;
     cout << "testing user exception... " << flush;
     try
@@ -219,7 +219,7 @@ ClientApp::run(const Test::MyObjectPrxPtr& prx, const InterceptorIPtr& intercept
         // expected
     }
     test(interceptor->getLastOperation() == "badAdd");
-    test(!interceptor->getLastStatus());
+    test(interceptor->getLastStatus());
     cout << "ok" << endl;
     cout << "testing ONE... " << flush;
 
@@ -260,7 +260,7 @@ ClientApp::run(const Test::MyObjectPrxPtr& prx, const InterceptorIPtr& intercept
     cout << "testing simple AMD... " << flush;
     test(prx->amdAdd(33, 12) == 45);
     test(interceptor->getLastOperation() == "amdAdd");
-    test(interceptor->getLastStatus());
+    test(!interceptor->getLastStatus());
     cout << "ok" << endl;
 
     return EXIT_SUCCESS;
@@ -273,12 +273,12 @@ ClientApp::runAmd(const Test::MyObjectPrxPtr& prx, const AMDInterceptorIPtr& int
     test(interceptor->getLastOperation().empty());
     test(prx->amdAdd(33, 12) == 45);
     test(interceptor->getLastOperation() == "amdAdd");
-    test(interceptor->getLastStatus());
+    test(!interceptor->getLastStatus());
     cout << "ok" << endl;
     cout << "testing retry... " << flush;
     test(prx->amdAddWithRetry(33, 12) == 45);
     test(interceptor->getLastOperation() == "amdAddWithRetry");
-    test(interceptor->getLastStatus());
+    test(!interceptor->getLastStatus());
     cout << "ok" << endl;
     cout << "testing user exception... " << flush;
     try
@@ -291,7 +291,7 @@ ClientApp::runAmd(const Test::MyObjectPrxPtr& prx, const AMDInterceptorIPtr& int
         // expected
     }
     test(interceptor->getLastOperation() == "amdBadAdd");
-    test(interceptor->getLastStatus());
+    test(!interceptor->getLastStatus());
     cout << "ok" << endl;
     cout << "testing ONE... " << flush;
     interceptor->clear();
@@ -305,7 +305,7 @@ ClientApp::runAmd(const Test::MyObjectPrxPtr& prx, const AMDInterceptorIPtr& int
         // expected
     }
     test(interceptor->getLastOperation() == "amdNotExistAdd");
-    test(interceptor->getLastStatus());
+    test(!interceptor->getLastStatus());
 
     test(dynamic_cast<Ice::ObjectNotExistException*>(interceptor->getException()) != 0);
 
@@ -326,7 +326,7 @@ ClientApp::runAmd(const Test::MyObjectPrxPtr& prx, const AMDInterceptorIPtr& int
         test(prx->ice_isCollocationOptimized());
     }
     test(interceptor->getLastOperation() == "amdBadSystemAdd");
-    test(interceptor->getLastStatus());
+    test(!interceptor->getLastStatus());
     test(dynamic_cast<MySystemException*>(interceptor->getException()) != 0);
     cout << "ok" << endl;
     return EXIT_SUCCESS;