fixes

Author: salvoilmiosi

include/boost/locale/numpunct.hpp

@@ -9,22 +9,19 @@
 #ifndef BOOST_LOCALE_NUMPUNCT_HPP_INCLUDED
 #define BOOST_LOCALE_NUMPUNCT_HPP_INCLUDED
 #include <boost/locale/config.hpp>
-#ifdef BOOST_MSVC
-#  pragma warning(push)
-#  pragma warning(disable : 4275 4251 4231 4660)
-#endif
 #include <locale>
 #include <string>
 
 namespace boost {
     namespace locale {
 
         template<typename CharType>
-        class BOOST_LOCALE_DECL numpunct_base : public std::numpunct<CharType>
+        class numpunct_base : public std::numpunct<CharType>
         {
             typedef std::basic_string<CharType> string_type;
         public:
             numpunct_base(size_t refs = 0) : std::numpunct<CharType>(refs) {}
+            virtual ~numpunct_base();
 
             string_type decimal_point_str() const {
                 return do_decimal_point_str();
@@ -35,7 +32,7 @@ namespace boost {
             }
 
         protected:
-            virtual CharType do_decimal_point() const {
+            CharType do_decimal_point() const BOOST_OVERRIDE {
                 string_type dec = do_decimal_point_str();
                 if (dec.size() > 1) {
                     return '.';
@@ -49,7 +46,7 @@ namespace boost {
                 return string_type(t, t + sizeof(t) - 1);
             }
 
-            virtual CharType do_thousands_sep() const {
+            CharType do_thousands_sep() const BOOST_OVERRIDE {
                 string_type thous = do_thousands_sep_str();
                 if (thous.size() > 1) {
                     return ',';
@@ -63,18 +60,18 @@ namespace boost {
                 return string_type(t, t + sizeof(t) - 1);
             }
 
-            virtual string_type do_truename() const {
+            virtual string_type do_truename() const BOOST_OVERRIDE {
                 static const char t[] = "true";
                 return string_type(t, t + sizeof(t) - 1);
             }
 
-            virtual string_type do_falsename() const {
+            virtual string_type do_falsename() const BOOST_OVERRIDE {
                 static const char t[] = "false";
                 return string_type(t, t + sizeof(t) - 1);
             }
         };
 
-        template<typename CharType> struct numpunct {};
+        template<typename CharType> struct numpunct;
 
         template<> struct numpunct<char> : numpunct_base<char> {
             numpunct (size_t refs = 0) : numpunct_base<char>(refs) {}

src/boost/locale/icu/numeric.cpp

@@ -381,13 +381,13 @@ struct icu_numpunct : public numpunct<CharType> {
         }
     }
 protected:
-    virtual string_type do_decimal_point_str() const {
+    string_type do_decimal_point_str() const BOOST_OVERRIDE {
         return decimal_point_;
     }
-    virtual string_type do_thousands_sep_str() const {
+    string_type do_thousands_sep_str() const BOOST_OVERRIDE {
         return thousands_sep_;
     }
-    virtual std::string do_grouping() const {
+    std::string do_grouping() const BOOST_OVERRIDE {
         return grouping_;
     }
 

src/boost/locale/posix/numeric.cpp

@@ -423,28 +423,18 @@ class num_punct_posix : public numpunct<CharType> {
     {
         s2=conv::to_utf<wchar_t>(s1,nl_langinfo_l(CODESET,lc));
     }
-    virtual string_type do_decimal_point_str() const
+    string_type do_decimal_point_str() const BOOST_OVERRIDE
     {
         return decimal_point_;
     }
-    virtual string_type do_thousands_sep_str() const
+    string_type do_thousands_sep_str() const BOOST_OVERRIDE
     {
         return thousands_sep_;
     }
     std::string do_grouping() const BOOST_OVERRIDE
     {
         return grouping_;
     }
-    string_type do_truename() const BOOST_OVERRIDE
-    {
-        static const char t[]="true";
-        return string_type(t,t+sizeof(t)-1);
-    }
-    string_type do_falsename() const BOOST_OVERRIDE
-    {
-        static const char t[]="false";
-        return string_type(t,t+sizeof(t)-1);
-    }
 private:
     string_type decimal_point_;
     string_type thousands_sep_;

src/boost/locale/shared/ids.cpp

@@ -11,6 +11,7 @@
 #include <boost/locale/date_time_facet.hpp>
 #include <boost/locale/info.hpp>
 #include <boost/locale/message.hpp>
+#include <boost/locale/numpunct.hpp>
 
 #include <boost/core/ignore_unused.hpp>
 
@@ -25,16 +26,19 @@ namespace boost {
         std::locale::id converter<char>::id;
         converter<char>::~converter() {}
         std::locale::id base_message_format<char>::id;
+        numpunct_base<char>::~numpunct_base() {}
 
         std::locale::id converter<wchar_t>::id;
         converter<wchar_t>::~converter() {}
         std::locale::id base_message_format<wchar_t>::id;
+        numpunct_base<wchar_t>::~numpunct_base() {}
 
         #ifdef BOOST_LOCALE_ENABLE_CHAR16_T
 
         std::locale::id converter<char16_t>::id;
         converter<char16_t>::~converter() {}
         std::locale::id base_message_format<char16_t>::id;
+        numpunct_base<char16_t>::~numpunct_base() {}
 
         #endif
 
@@ -43,6 +47,7 @@ namespace boost {
         std::locale::id converter<char32_t>::id;
         converter<char32_t>::~converter() {}
         std::locale::id base_message_format<char32_t>::id;
+        numpunct_base<char32_t>::~numpunct_base() {}
 
         #endif
 

src/boost/locale/win32/numeric.cpp

@@ -141,11 +141,11 @@ BOOST_LOCALE_END_CONST_CONDITION
     {
         s2=conv::from_utf(s1,"UTF-8");
     }
-    virtual string_type do_decimal_point_str() const
+    string_type do_decimal_point_str() const BOOST_OVERRIDE
     {
         return decimal_point_;
     }
-    virtual string_type do_thousands_sep_str() const
+    string_type do_thousands_sep_str() const BOOST_OVERRIDE
     {
         return thousands_sep_;
     }