Made cds::OS children namespaces inline, removed OS-specific syserror.h headers
authorkhizmax <libcds.dev@gmail.com>
Fri, 28 Nov 2014 20:54:08 +0000 (23:54 +0300)
committerkhizmax <libcds.dev@gmail.com>
Fri, 28 Nov 2014 20:54:08 +0000 (23:54 +0300)
13 files changed:
cds/os/libc/alloc_aligned.h
cds/os/posix/alloc_aligned.h
cds/os/posix/fake_topology.h
cds/os/posix/syserror.h [deleted file]
cds/os/posix/thread.h
cds/os/posix/timer.h
cds/os/win/alloc_aligned.h
cds/os/win/syserror.h [deleted file]
cds/os/win/thread.h
projects/Win/vc12/cds.vcxproj
projects/Win/vc12/cds.vcxproj.filters
projects/Win/vc12/unit-prerequisites.vcxproj
readme

index 4079dae..bf55b65 100644 (file)
@@ -7,7 +7,7 @@
 
 //@cond none
 namespace cds { namespace OS {
-    namespace libc {
+    CDS_CXX11_INLINE_NAMESPACE namespace libc {
         /// Allocates memory on a specified alignment boundary
         static inline void * aligned_malloc(
             size_t nSize,       ///< Size of the requested memory allocation
@@ -26,8 +26,10 @@ namespace cds { namespace OS {
         }
     }   // namespace libc
 
+#ifndef CDS_CXX11_INLINE_NAMESPACE_SUPPORT
     using libc::aligned_malloc;
     using libc::aligned_free;
+#endif
 
 }} // namespace cds::OS
 //@endcond
index 664063f..b123a64 100644 (file)
@@ -14,7 +14,7 @@
 
 //@cond none
 namespace cds { namespace OS {
-    namespace posix {
+    CDS_CXX11_INLINE_NAMESPACE namespace posix {
         /// Allocates memory on a specified alignment boundary
         static inline void * aligned_malloc(
             size_t nSize,       ///< Size of the requested memory allocation
@@ -34,8 +34,10 @@ namespace cds { namespace OS {
         }
     }   // namespace posix
 
+#ifndef CDS_CXX11_INLINE_NAMESPACE_SUPPORT
     using posix::aligned_malloc;
     using posix::aligned_free;
+#endif
 
 }} // namespace cds::OS
 //@endcond
index c5f91c5..4fca62f 100644 (file)
@@ -11,7 +11,7 @@
 #include <unistd.h>
 
 namespace cds { namespace OS {
-    namespace posix {
+    CDS_CXX11_INLINE_NAMESPACE namespace posix {
         /// Fake system topology
         struct topology {
             /// Logical processor count for the system. Always returns 1
@@ -44,7 +44,10 @@ namespace cds { namespace OS {
         };
     }   // namespace posix
 
+#ifndef CDS_CXX11_INLINE_NAMESPACE_SUPPORT
     using posix::topology;
+#endif
+
 }}  // namespace cds::OS
 
 #endif  // #ifndef __CDS_OS_POSIX_FAKE_TOPOLOGY_H
diff --git a/cds/os/posix/syserror.h b/cds/os/posix/syserror.h
deleted file mode 100644 (file)
index 2929e01..0000000
+++ /dev/null
@@ -1,40 +0,0 @@
-//$$CDS-header$$
-
-#ifndef __CDS_OS_POSIX_SYSERROR_H
-#define __CDS_OS_POSIX_SYSERROR_H
-
-#include <stdlib.h>     // system
-#include <pthread.h>
-#include <sched.h>
-#include <cerrno>
-#include <cstdlib>
-#include <string>
-#include <string.h>
-
-namespace cds { namespace OS {
-    namespace posix {
-        /// Posix error code type
-        typedef int                error_code;
-
-        /// Get text for system error \p nCode
-        static inline std::string getSystemErrorText( error_code nCode )
-        {
-            char *msg = strerror( nCode );
-            return std::string( msg );
-        }
-
-        /// Get curent error code (returns \p errno)
-        static inline error_code  getErrorCode()
-        {
-            return errno;
-        }
-
-    }    // namespace posix
-
-    using posix::error_code;
-    using posix::getErrorCode;
-    using posix::getSystemErrorText;
-}} // namespace cds::OS
-
-
-#endif // #ifndef __CDS_OS_POSIX_SYSERROR_H
index 5b8ddf9..adaf022 100644 (file)
@@ -8,7 +8,7 @@
 
 namespace cds { namespace OS {
     /// posix-related wrappers
-    namespace posix {
+    CDS_CXX11_INLINE_NAMESPACE namespace posix {
 
         /// Posix thread id type
         typedef std::thread::native_handle_type ThreadId;
@@ -29,13 +29,14 @@ namespace cds { namespace OS {
         }
     }    // namespace posix
 
-    //@cond
+    constexpr const posix::ThreadId c_NullThreadId = 0;
+
+#ifndef CDS_CXX11_INLINE_NAMESPACE_SUPPORT
     using posix::ThreadId;
-    constexpr const ThreadId c_NullThreadId = 0;
 
     using posix::getCurrentThreadId;
     using posix::isThreadAlive;
-    //@endcond
+#endif
 
 }} // namespace cds::OS
 
index 9275a24..1220a5b 100644 (file)
@@ -11,7 +11,7 @@
 
 //@cond none
 namespace cds { namespace OS {
-    namespace posix {
+    CDS_CXX11_INLINE_NAMESPACE namespace posix {
 
         // High resolution timer
         // From Linux as an example
@@ -79,7 +79,9 @@ namespace cds { namespace OS {
         };
     }    // namespace posix
 
+#ifndef CDS_CXX11_INLINE_NAMESPACE_SUPPORT
     using posix::Timer;
+#endif
 
 }}    // namespace cds::OS
 //@endcond
index 061afc4..5a6bf1d 100644 (file)
@@ -7,7 +7,7 @@
 
 //@cond none
 namespace cds { namespace OS {
-    namespace Win32 {
+    CDS_CXX11_INLINE_NAMESPACE namespace Win32 {
         /// Allocates memory on a specified alignment boundary
         static inline void * aligned_malloc(
             size_t nSize,       ///< Size of the requested memory allocation
@@ -26,8 +26,10 @@ namespace cds { namespace OS {
         }
     }   // namespace Win32
 
+#ifndef CDS_CXX11_INLINE_NAMESPACE_SUPPORT
     using Win32::aligned_malloc;
     using Win32::aligned_free;
+#endif
 
 }} // namespace cds::OS
 //@endcond
diff --git a/cds/os/win/syserror.h b/cds/os/win/syserror.h
deleted file mode 100644 (file)
index 26491e0..0000000
+++ /dev/null
@@ -1,48 +0,0 @@
-//$$CDS-header$$
-
-#ifndef __CDS_OS_WIN_SYSERROR_H
-#define __CDS_OS_WIN_SYSERROR_H
-
-#include <windows.h>
-#include <string>
-
-namespace cds { namespace OS {
-    namespace Win32 {
-
-        /// OS-specific type of error code
-        typedef DWORD            error_code;
-
-        /// Get system error code
-        static inline error_code  getErrorCode()
-        {
-            return ::GetLastError();
-        }
-
-        /// Get system error text
-        static inline std::string getSystemErrorText( error_code nCode )
-        {
-            char *ptmp = 0;
-            if ( !FormatMessage( FORMAT_MESSAGE_ALLOCATE_BUFFER | FORMAT_MESSAGE_FROM_SYSTEM | FORMAT_MESSAGE_IGNORE_INSERTS,
-                nullptr,
-                nCode,
-                MAKELANGID(LANG_NEUTRAL, SUBLANG_DEFAULT),
-                (LPTSTR) &ptmp, 0, 0  )
-                )
-            {
-                if ( ptmp )
-                    LocalFree( ptmp );
-                return std::string();
-            }
-            std::string str( ptmp );
-            LocalFree( ptmp );
-            return str;
-        }
-    }    // namespace Win32
-
-    using Win32::error_code;
-    using Win32::getErrorCode;
-    using Win32::getSystemErrorText;
-}} // namespace cds::OS
-
-#endif // #ifndef __CDS_OS_WIN_SYSERROR_H
-
index 2185d7a..0e3d664 100644 (file)
@@ -7,7 +7,7 @@
 
 namespace cds { namespace OS {
     /// Windows-specific functions
-    namespace Win32 {
+    CDS_CXX11_INLINE_NAMESPACE namespace Win32 {
 
         /// OS-specific type of thread identifier
         typedef DWORD           ThreadId;
@@ -30,11 +30,13 @@ namespace cds { namespace OS {
     }    // namespace Win32
 
     //@cond
-    using Win32::ThreadId;
-    CDS_CONSTEXPR const ThreadId c_NullThreadId = 0;
+    CDS_CONSTEXPR const Win32::ThreadId c_NullThreadId = 0;
 
+#ifndef CDS_CXX11_INLINE_NAMESPACE_SUPPORT
+    using Win32::ThreadId;
     using Win32::getCurrentThreadId;
     using Win32::isThreadAlive;
+#endif
     //@endcond
 
 }} // namespace cds::OS
index 2f97461..011f15f 100644 (file)
     <ClInclude Include="..\..\..\cds\os\linux\timer.h" />\r
     <ClInclude Include="..\..\..\cds\os\linux\topology.h" />\r
     <ClInclude Include="..\..\..\cds\os\posix\alloc_aligned.h" />\r
-    <ClInclude Include="..\..\..\cds\os\posix\syserror.h" />\r
     <ClInclude Include="..\..\..\cds\os\posix\thread.h" />\r
     <ClInclude Include="..\..\..\cds\os\sunos\alloc_aligned.h" />\r
     <ClInclude Include="..\..\..\cds\os\sunos\timer.h" />\r
     <ClInclude Include="..\..\..\cds\os\sunos\topology.h" />\r
     <ClInclude Include="..\..\..\cds\os\win\alloc_aligned.h" />\r
-    <ClInclude Include="..\..\..\cds\os\win\syserror.h" />\r
     <ClInclude Include="..\..\..\cds\os\win\thread.h" />\r
     <ClInclude Include="..\..\..\cds\os\win\timer.h" />\r
     <ClInclude Include="..\..\..\cds\os\win\topology.h" />\r
index 0fcc08e..732a2ae 100644 (file)
     <ClInclude Include="..\..\..\cds\os\posix\alloc_aligned.h">\r
       <Filter>Header Files\cds\OS\posix</Filter>\r
     </ClInclude>\r
-    <ClInclude Include="..\..\..\cds\os\posix\syserror.h">\r
-      <Filter>Header Files\cds\OS\posix</Filter>\r
-    </ClInclude>\r
     <ClInclude Include="..\..\..\cds\os\posix\thread.h">\r
       <Filter>Header Files\cds\OS\posix</Filter>\r
     </ClInclude>\r
     <ClInclude Include="..\..\..\cds\os\win\alloc_aligned.h">\r
       <Filter>Header Files\cds\OS\win</Filter>\r
     </ClInclude>\r
-    <ClInclude Include="..\..\..\cds\os\win\syserror.h">\r
-      <Filter>Header Files\cds\OS\win</Filter>\r
-    </ClInclude>\r
     <ClInclude Include="..\..\..\cds\os\win\thread.h">\r
       <Filter>Header Files\cds\OS\win</Filter>\r
     </ClInclude>\r
index 7d7b988..3890ae7 100644 (file)
   <ItemGroup>\r
     <ClCompile Include="..\..\..\tests\unit\ellen_bintree_update_desc_pool.cpp" />\r
     <ClCompile Include="..\..\..\tests\unit\michael_alloc.cpp" />\r
-    <ClCompile Include="..\..\..\tests\cppunit\test_main.cpp" />\r
+    <ClCompile Include="..\..\..\tests\cppunit\test_main.cpp">\r
+      <PreprocessorDefinitions Condition="'$(Configuration)|$(Platform)'=='Debug|Win32'">_CRT_SECURE_NO_WARNINGS;%(PreprocessorDefinitions)</PreprocessorDefinitions>\r
+      <PreprocessorDefinitions Condition="'$(Configuration)|$(Platform)'=='DebugVLD|Win32'">_CRT_SECURE_NO_WARNINGS;%(PreprocessorDefinitions)</PreprocessorDefinitions>\r
+      <PreprocessorDefinitions Condition="'$(Configuration)|$(Platform)'=='ICL-Debug|Win32'">_CRT_SECURE_NO_WARNINGS;%(PreprocessorDefinitions)</PreprocessorDefinitions>\r
+      <PreprocessorDefinitions Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">_CRT_SECURE_NO_WARNINGS;%(PreprocessorDefinitions)</PreprocessorDefinitions>\r
+      <PreprocessorDefinitions Condition="'$(Configuration)|$(Platform)'=='DebugVLD|x64'">_CRT_SECURE_NO_WARNINGS;%(PreprocessorDefinitions)</PreprocessorDefinitions>\r
+      <PreprocessorDefinitions Condition="'$(Configuration)|$(Platform)'=='ICL-Debug|x64'">_CRT_SECURE_NO_WARNINGS;%(PreprocessorDefinitions)</PreprocessorDefinitions>\r
+    </ClCompile>\r
     <ClCompile Include="..\..\..\tests\cppunit\thread.cpp" />\r
   </ItemGroup>\r
   <ItemGroup>\r
diff --git a/readme b/readme
index c5ec7b5..eeab1c5 100644 (file)
--- a/readme
+++ b/readme
@@ -93,7 +93,7 @@ Supported compilers
 ---------------------------------\r
 \r
 GCC: 4.8 and above\r
-MS Visual C++: 12 (2013) and above\r
+MS Visual C++: 12 (2013) Update 4 and above\r
 Clang: 3.3 and above\r
 \r
 How to build\r