Merge pull request #12 from meyerj/master-merged-with-upstream

Merge with upstream version 2017-07-13 08:48

Author: meyerj

CMakeLists.txt

@@ -30,11 +30,7 @@ ELSE (WIN32)
   ENDIF (APPLE)
 ENDIF (WIN32)
 
-#IF ( CMAKE_BUILD_TYPE MATCHES "Debug" )
-#  SET ( LOG4CPP_LIBRARY_NAME "log4cppD" )
-#ELSE ( CMAKE_BUILD_TYPE MATCHES "Debug" )
-  SET ( LOG4CPP_LIBRARY_NAME "log4cpp" )
-#ENDIF ( CMAKE_BUILD_TYPE MATCHES "Debug" )
+SET ( LOG4CPP_LIBRARY_NAME "orocos-log4cpp" )
 
 ADD_LIBRARY ( ${LOG4CPP_LIBRARY_NAME} SHARED
   src/Appender.cpp
@@ -92,12 +88,28 @@ IF (WIN32)
 #  SET_TARGET_PROPERTIES(${LOG4CPP_LIBRARY_NAME} PROPERTIES LINK_FLAGS /NODEFAULTLIB:msvcrt)
 ENDIF (WIN32)
 
-# Note: this so-version is required by OCL, such that it can distinguish 'regular' log4cpp
-# from 'rtt' log4cpp.
-SET(VERSION "6.0.0")
-SET(SOVERSION "6.0")
+SET(VERSION "2.9.0")
+SET(SOVERSION "2.9")
 SET_TARGET_PROPERTIES(${LOG4CPP_LIBRARY_NAME} PROPERTIES VERSION ${VERSION} SOVERSION ${SOVERSION})
 
+# Ensure that the full path+lib name is used in dynamic library dependencies
+# in dependent libraries/executables. Without this, CMake drops the path and
+# the dependency becomes just the lib name (which requires working DYLD_xxx)
+SET_TARGET_PROPERTIES(${LOG4CPP_LIBRARY_NAME} PROPERTIES
+                      INSTALL_RPATH_USE_LINK_PATH ON)
+# Set INSTALL_NAME_DIR for MacOS X to tell users of this library how to find it:
+if(APPLE)
+    if (CMAKE_VERSION VERSION_LESS "3.0.0")
+        SET_TARGET_PROPERTIES( ${LOG4CPP_LIBRARY_NAME} PROPERTIES
+            INSTALL_NAME_DIR "@rpath"
+        )
+    else()
+        # cope with CMake 3.x
+        SET_TARGET_PROPERTIES( ${LOG4CPP_LIBRARY_NAME} PROPERTIES
+            MACOSX_RPATH ON)
+    endif()
+endif()
+
 ADD_DEFINITIONS(${LOG4CPP_CFLAGS})
 
 ###########################################################
@@ -120,18 +132,18 @@ SET(prefix "${CMAKE_INSTALL_PREFIX}")
 SET(exec_prefix "\${prefix}")
 SET(log4cpp_cflags "")
 SET(log4cpp_libs "${LOG4CPP_LIBS}")
-SET(includedir "\${prefix}/include")
+SET(includedir "\${prefix}/include/orocos")
 SET(libdir "\${prefix}/lib")
-CONFIGURE_FILE(log4cpp.pc.in ${CMAKE_CURRENT_BINARY_DIR}/log4cpp.pc @ONLY)
+CONFIGURE_FILE(log4cpp.pc.in ${CMAKE_CURRENT_BINARY_DIR}/orocos-log4cpp.pc @ONLY)
 
 INSTALL(
-  FILES ${CMAKE_CURRENT_BINARY_DIR}/log4cpp.pc
+  FILES ${CMAKE_CURRENT_BINARY_DIR}/orocos-log4cpp.pc
   DESTINATION lib/pkgconfig
   )
 
 INSTALL (
   DIRECTORY include/log4cpp
-  DESTINATION include
+  DESTINATION include/orocos
   PATTERN "config.h.in" EXCLUDE
   PATTERN ".svn" EXCLUDE
   PATTERN "*.am" EXCLUDE
@@ -141,7 +153,7 @@ INSTALL (
 
 INSTALL (
   FILES ${CMAKE_CURRENT_BINARY_DIR}/include/log4cpp/config.h
-  DESTINATION include/log4cpp
+  DESTINATION include/orocos/log4cpp
   )
 
 INSTALL(TARGETS ${LOG4CPP_LIBRARY_NAME}

ChangeLog

@@ -1,3 +1,13 @@
+2017-07-13 08:48  sanchouss_
+	* bug 146; 
+	* bug 147 - Being prepared for Removing Deprecated Exception Specifications from C++17
+2016-11-11 18:48  sanchouss_
+
+	* Numerous fixes;
+	* Added .target property for the ConsoleAppender to allow printing to STDERR 
+	* Added method Category::shutdownForced(). It releases more memory than shutdown() by deleting all appenders. 
+	Category::shutdownForced() may be invoked several times, not only at the end of program.
+
 2015-03-28 18:48  sanchouss_
 
 	* Added DailyRollingFileAppender.cpp

doc/html/index.html

@@ -45,7 +45,7 @@ <h5>How to use</h5>
 </ul>
 <h5>Development</h5>
 <ul>
-<li><a title="CVS Repository" href="#cvs">CVS
+<li><a title="Git Repository" href="#git">Git
 Repository</a> </li>
 <li><a title="Releases" href="#releases">Releases</a>
 </li>
@@ -561,14 +561,18 @@ <h2><a id="propfile">Properties file example</a></h2>
 </div>
 
 <div class="chapter">
-<h2><a id="cvs">CVS Repository</a></h2>
-<p>Log4cpp is also available directly through CVS, see the <a href="http://sourceforge.net/cvs/?group_id=15190">SourceForge CVS page</a> for instructions.
-CVS currently has two branches:
+<h2><a id="git">Git Repository</a></h2>
+<p>
+log4cpp is moved to git <a href="https://sourceforge.net/p/log4cpp/codegit/">SourceForge Git page</a> since version 1.1.1, cvs is stale now.
 </p>
-<p></p><table>
+<p>Log4cpp of older versions is also available through CVS, see the <a href="https://sourceforge.net/p/log4cpp/code/">SourceForge CVS page</a> for instructions.</p>
+<small>
+CVS had two branches:
+<table>
 <tbody><tr><td>MAIN</td><td>for log4cpp development</td></tr>
 <tr><td>BRANCH_MAINT_0_2</td><td>for maintainance of log4cpp-stable (0.2.x)</td></tr>
 </tbody></table><br>
+</small>
 Each release will receive a tag named REL_x_y_z.
 <p></p>
 <p>To start working with a freshly checked out log4cpp revision, run <b>./autogen.sh</b>
@@ -584,6 +588,14 @@ <h2><a id="releases">Releases</a></h2>
 WARNING: releases from the development branch are a 'work in progress' and may fail to build, crash or redecorate your desktop.
 </p>
 <small>
+
+<dl>
+<dt>1.1.2 - master branch (18 April 2017)</dt>
+<dd>Fixed memory leak after Category::shutdown() called: method Category::shutdownForced() releases memory allocated for appenders.</dd>
+<dd>PropertyConfiguratorImpl: add target property to allow printing to STDERR.</dd>
+<dd>Some fixes.</dd>
+</dl>
+
 <dl>
 <dt>1.1.2rc1 - master branch (04 April 2015)</dt>
 <dd>Small fixes, DailyRollingFile appender, tests, options to disable support for smtp and syslog</dd>
@@ -711,7 +723,7 @@ <h2><a id="releases">Releases</a></h2>
 
 <div class="chapter">
 <h2><a id="status">Status</a></h2>
-<p>The latest stable release is 1.1.1</p>
+<p>The latest stable release is 1.1.2</p>
 <small>
 <p>log4cpp is moved to git since version 1.1.1, cvs is stale since then</p>
 <p>As of version 0.3.0 log4cpp has a separate 'stable' and 'development' branches.
@@ -783,4 +795,5 @@ <h2><a id="id">Chapter</a></h2>
 </div>
 </div>
 </div>
-</body></html>
+</body></html>
+

include/log4cpp/Appender.hh

@@ -14,6 +14,7 @@
 #include <string>
 #include <map>
 #include <set>
+#include <vector>
 #include <stdarg.h>
 #include <sys/types.h>
 #include <sys/stat.h>
@@ -31,6 +32,7 @@ namespace log4cpp {
      *  statements.
      **/
     class LOG4CPP_EXPORT Appender {
+		friend class Category;
         public:
 
         /**
@@ -130,21 +132,38 @@ namespace log4cpp {
          **/
         virtual Filter* getFilter() = 0;
 
-        private:
+		private:
         typedef std::map<std::string, Appender*> AppenderMap;
-        
-        static AppenderMap* _allAppenders;
-        static threading::Mutex _appenderMapMutex;
 
         static AppenderMap& _getAllAppenders();
         static void _deleteAllAppenders();
+		static void _deleteAllAppendersWOLock(std::vector<Appender*> &appenders);
         static void _addAppender(Appender* appender);
         static void _removeAppender(Appender* appender);
 
         const std::string _name;
+
+		public:
+		class AppenderMapStorage {
+		public:
+			Appender::AppenderMap* _allAppenders;	// single shared instance, nifty-counter defensed
+	        threading::Mutex _appenderMapMutex;	// mutex protecting map from multiple thread access 
+
+			AppenderMapStorage();
+			~AppenderMapStorage();
+		};
+		class LOG4CPP_EXPORT AppenderMapStorageInitializer {
+		public:
+			AppenderMapStorageInitializer();
+			~AppenderMapStorageInitializer();
+		};
+		private:
+        static AppenderMapStorage &_appenderMapStorageInstance;
     };
 
+	static Appender::AppenderMapStorageInitializer appenderMapStorageInitializer; // static initializer for every translation unit
     typedef std::set<Appender *> AppenderSet;
+
 }
 
 #endif // _LOG4CPP_APPENDER_HH

include/log4cpp/Category.hh

@@ -95,6 +95,12 @@ namespace log4cpp {
         static void shutdown();
 
         /**
+         * This method will remove all Appenders from Categories.XXX and delete all appenders.
+		 * Releases more memory than shutdown() by deleting appenders.
+         **/
+        static void shutdownForced();
+
+		/**
          * Destructor for Category.
          **/
         virtual ~Category();
@@ -112,8 +118,7 @@ namespace log4cpp {
          * @exception std::invalid_argument if the caller tries to set
          * Priority::NOTSET on the Root Category.
          **/
-        virtual void setPriority(Priority::Value priority) 
-        throw(std::invalid_argument);
+        virtual void setPriority(Priority::Value priority);
 
         /**
          * Returns the assigned Priority, if any, for this Category.
@@ -146,8 +151,7 @@ namespace log4cpp {
          * @param appender The Appender to wich this category has to log.
          * @exception std::invalid_argument if the appender is NULL.
          **/
-        virtual void addAppender(Appender* appender) 
-        throw(std::invalid_argument);
+        virtual void addAppender(Appender* appender);
 
         /**
          * Adds an Appender for this Category.

include/log4cpp/PatternLayout.hh

@@ -84,8 +84,7 @@ namespace log4cpp {
          * @param conversionPattern the conversion pattern
          * @exception ConfigureFailure if the pattern is invalid
          **/
-        virtual void setConversionPattern(const std::string& conversionPattern)
-            throw(ConfigureFailure);
+        virtual void setConversionPattern(const std::string& conversionPattern);
 
         virtual std::string getConversionPattern() const;
 

include/log4cpp/Portability.hh

@@ -16,6 +16,21 @@
 #    else
 #        include <log4cpp/config-win32.h>
 #    endif
+
+#ifdef MSVC_MEMORY_LEAK_CHECK
+#define _CRTDBG_MAP_ALLOC
+
+#include <stdlib.h>
+#include <crtdbg.h>
+
+#ifdef _DEBUG
+   #ifndef DBG_NEW
+      #define DBG_NEW new ( _NORMAL_BLOCK , __FILE__ , __LINE__ )
+      #define new DBG_NEW
+   #endif
+#endif  // _DEBUG
+#endif // MSVC_MEMORY_LEAK_CHECK
+
 #else
 #if defined(__OPENVMS__)
 #    include <log4cpp/config-openvms.h>
@@ -50,4 +65,5 @@
 #  endif // LOG4CPP_HAVE_SSTREAM
 #endif // _APPLE_
 
+
 #endif

include/log4cpp/Priority.hh

@@ -103,8 +103,7 @@ namespace log4cpp {
 	 * @throw std::invalid_argument if the priorityName does not 
 	 * correspond with a known Priority name or a number
 	 **/
-        static Value getPriorityValue(const std::string& priorityName)
-	throw(std::invalid_argument);
+        static Value getPriorityValue(const std::string& priorityName);
     };
 }
 

include/log4cpp/PropertyConfigurator.hh

@@ -45,7 +45,13 @@ namespace log4cpp {
     **/
     class LOG4CPP_EXPORT PropertyConfigurator {
         public:
-        static void configure(const std::string& initFileName) throw (ConfigureFailure);
+        /**
+         *
+         * @param initFileName
+         * @exception ConfigureFailure if the method encountered a read or
+         * syntax error.
+         */
+        static void configure(const std::string& initFileName);
     };
 }
 

include/log4cpp/SimpleConfigurator.hh

@@ -34,7 +34,7 @@ namespace log4cpp {
          * @exception ConfigureFailure if the method encountered a read or 
          * syntax error.
          **/
-        static void configure(const std::string& initFileName) throw (ConfigureFailure);
+        static void configure(const std::string& initFileName);
 
         /**
          * Configure log4cpp with the configuration in the given file.
@@ -45,7 +45,8 @@ namespace log4cpp {
          * @exception ConfigureFailure if the method encountered a read or 
          * syntax error.
          **/
-        static void configure(std::istream& initFile) throw (ConfigureFailure);    };
+        static void configure(std::istream& initFile);
+    };
 }
 
 #endif

log4cpp.pc.in

@@ -6,6 +6,6 @@ includedir=@includedir@
 Name: @PACKAGE@
 Description: C++ library for flexible logging, modeled after Log4j
 Version: @VERSION@
-Libs: -L${libdir} -llog4cpp
+Libs: -L${libdir} -l@LOG4CPP_LIBRARY_NAME@
 Cflags: -I${includedir} @log4cpp_cflags@
 Libs.private: @log4cpp_libs@

package.xml

@@ -1,6 +1,6 @@
 <package>
   <name>log4cpp</name>
-  <version>2.8.3</version>
+  <version>2.9.0</version>
   <description >
     Log4cpp maintained by Orocos developers
     This version of log4cpp deviates from the official release