-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/7179/
-----------------------------------------------------------

Review request for qpid, Andrew Stitcher, Chug Rolke, and Steve Huston.


Description
-------

This patch introduces a primitive qpid::sys::SystemInfo::threadSafeShutdown().  
It always returns true on Linux and Solaris.  On Windows, it returns true for 
FreeLibrary and false if exit() has been called, main() returns, or the Qpid 
libraries have been statically linked.

It is used to fix the noted static destructor problems in the main Jira.

Most of the work is plumbing related to obtain knowledge of how the module is 
terminated in time to be useful to the consumer of the threadSafeShutdown call.


This addresses bug qpid-4330.
    https://issues.apache.org/jira/browse/qpid-4330


Diffs
-----

  
http://svn.apache.org/repos/asf/qpid/trunk/qpid/cpp/include/qpid/sys/SystemInfo.h
 1387463 
  http://svn.apache.org/repos/asf/qpid/trunk/qpid/cpp/src/CMakeLists.txt 
1387463 
  http://svn.apache.org/repos/asf/qpid/trunk/qpid/cpp/src/Makefile.am 1387463 
  
http://svn.apache.org/repos/asf/qpid/trunk/qpid/cpp/src/qpid/client/ConnectionImpl.cpp
 1387463 
  
http://svn.apache.org/repos/asf/qpid/trunk/qpid/cpp/src/qpid/client/windows/ClientDllMain.cpp
 PRE-CREATION 
  
http://svn.apache.org/repos/asf/qpid/trunk/qpid/cpp/src/qpid/sys/posix/SystemInfo.cpp
 1387463 
  
http://svn.apache.org/repos/asf/qpid/trunk/qpid/cpp/src/qpid/sys/solaris/SystemInfo.cpp
 1387463 
  
http://svn.apache.org/repos/asf/qpid/trunk/qpid/cpp/src/qpid/sys/windows/QpidDllMain.h
 PRE-CREATION 
  
http://svn.apache.org/repos/asf/qpid/trunk/qpid/cpp/src/qpid/sys/windows/Socket.cpp
 1387463 
  
http://svn.apache.org/repos/asf/qpid/trunk/qpid/cpp/src/qpid/sys/windows/SystemInfo.cpp
 1387463 
  
http://svn.apache.org/repos/asf/qpid/trunk/qpid/cpp/src/qpid/sys/windows/Thread.cpp
 1387463 

Diff: https://reviews.apache.org/r/7179/diff/


Testing
-------


Thanks,

Cliff Jansen

Reply via email to