diff options
| author | Kevin Rocard <kevinx.rocard@intel.com> | 2014-12-19 15:15:50 +0100 | 
|---|---|---|
| committer | Eric Laurent <elaurent@google.com> | 2015-04-24 13:39:13 -0700 | 
| commit | 09af813fef22c2c3cbb34795e290ebab3db65e35 (patch) | |
| tree | 32d50d5c19355fc9bf202c5747285d7baa919452 | |
| parent | 6bd9fcf524e77a1ca7469d6d91c58afd242d388a (diff) | |
| download | external_parameter-framework-09af813fef22c2c3cbb34795e290ebab3db65e35.zip external_parameter-framework-09af813fef22c2c3cbb34795e290ebab3db65e35.tar.gz external_parameter-framework-09af813fef22c2c3cbb34795e290ebab3db65e35.tar.bz2  | |
Remove explicit server stop
Stopping the remote-processor in the stop command has two side effect:
 - stopping the remote-processor thread will fail as it is not possible
   to join a thread to itself. This leads to the leak of all the thread
   object (destructor not called).
 - the server will not return the exit command status as it has been
   stopped during it's execution.
Remove the remote-processor stop as it will be called during it's
destruction.
Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com>
| -rw-r--r-- | test/test-platform/TestPlatform.cpp | 3 | 
1 files changed, 0 insertions, 3 deletions
diff --git a/test/test-platform/TestPlatform.cpp b/test/test-platform/TestPlatform.cpp index a6613e9..9cb45f7 100644 --- a/test/test-platform/TestPlatform.cpp +++ b/test/test-platform/TestPlatform.cpp @@ -154,9 +154,6 @@ CTestPlatform::CommandReturn CTestPlatform::exit(  {      (void)remoteCommand; -    // Stop local server -    _pRemoteProcessorServer->stop(); -      // Release the main blocking semaphore to quit application      sem_post(&_exitSemaphore);  | 
