EricWF updated the summary for this revision.
EricWF updated this revision to Diff 31107.
EricWF added a comment.

David Blaikie pointed out that a better way to solve this warning is to make 
`__time_get_c_storage`'s destructor protected.


http://reviews.llvm.org/D11670

Files:
  include/locale

Index: include/locale
===================================================================
--- include/locale
+++ include/locale
@@ -1888,6 +1888,9 @@
     virtual const string_type& __r() const;
     virtual const string_type& __x() const;
     virtual const string_type& __X() const;
+
+    _LIBCPP_ALWAYS_INLINE
+    ~__time_get_c_storage() {}
 };
 
 template <class _CharT, class _InputIterator = istreambuf_iterator<_CharT> >


Index: include/locale
===================================================================
--- include/locale
+++ include/locale
@@ -1888,6 +1888,9 @@
     virtual const string_type& __r() const;
     virtual const string_type& __x() const;
     virtual const string_type& __X() const;
+
+    _LIBCPP_ALWAYS_INLINE
+    ~__time_get_c_storage() {}
 };
 
 template <class _CharT, class _InputIterator = istreambuf_iterator<_CharT> >
_______________________________________________
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to