summaryrefslogtreecommitdiff
path: root/cpp/src/IceBox
diff options
context:
space:
mode:
authorBenoit Foucher <benoit@zeroc.com>2017-01-25 18:42:10 +0100
committerBenoit Foucher <benoit@zeroc.com>2017-01-25 18:42:10 +0100
commit524c89c8c5ea8be6d989c855095cf4c58097e654 (patch)
tree697a384847d23d3d84345b84339f42d63d45687a /cpp/src/IceBox
parentFixed ICE-7281 - skipEmptyEncapsulation now checks for the encoding version (diff)
downloadice-524c89c8c5ea8be6d989c855095cf4c58097e654.tar.bz2
ice-524c89c8c5ea8be6d989c855095cf4c58097e654.tar.xz
ice-524c89c8c5ea8be6d989c855095cf4c58097e654.zip
First cut of ICE-6920 - Remove try/catch block around communicator destroy
Diffstat (limited to 'cpp/src/IceBox')
-rw-r--r--cpp/src/IceBox/ServiceManagerI.cpp27
1 files changed, 4 insertions, 23 deletions
diff --git a/cpp/src/IceBox/ServiceManagerI.cpp b/cpp/src/IceBox/ServiceManagerI.cpp
index fdae86be07f..6226b7c0a4d 100644
--- a/cpp/src/IceBox/ServiceManagerI.cpp
+++ b/cpp/src/IceBox/ServiceManagerI.cpp
@@ -676,7 +676,7 @@ IceBox::ServiceManagerI::start(const string& service, const string& entryPoint,
// xlC warns when casting a void* to function pointer
# pragma report(disable, "1540-0216")
#endif
-
+
SERVICE_FACTORY factory = reinterpret_cast<SERVICE_FACTORY>(sym);
try
{
@@ -860,17 +860,8 @@ IceBox::ServiceManagerI::stopAll()
{
removeAdminFacets("IceBox.Service." + info.name + ".");
- try
- {
- info.communicator->destroy();
- info.communicator = 0;
- }
- catch(const Exception& ex)
- {
- Warning out(_logger);
- out << "ServiceManager: exception while stopping service " << info.name << ":\n";
- out << ex;
- }
+ info.communicator->destroy();
+ info.communicator = 0;
}
try
@@ -1126,17 +1117,7 @@ IceBox::ServiceManagerI::destroyServiceCommunicator(const string& service, const
}
removeAdminFacets("IceBox.Service." + service + ".");
-
- try
- {
- communicator->destroy();
- }
- catch(const Exception& ex)
- {
- Warning out(_logger);
- out << "ServiceManager: exception in shutting down communicator for service " << service << ":\n";
- out << ex;
- }
+ communicator->destroy();
}
bool