re PR c++/55908 (Problem binding a const member function to a const object)

	PR libstdc++/55908
	* include/std/functional (_Mem_fn::operator()): Fix constraints to
	avoid ambiguity.
	* testsuite/20_util/function_objects/mem_fn/55908.cc: New.
	* testsuite/20_util/bind/ref_neg.cc: Adjust dg-error line numbers.

From-SVN: r195035

diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog
index 9d0a6d7574ab180732f17c55b30eaa65ed0cee7c..2b701cb772c30b8d7da59c03278b8e604f20ace4 100644 (file)
--- a/libstdc++-v3/ChangeLog
+++ b/libstdc++-v3/ChangeLog
@@ -1,3 +1,11 @@
+2013-01-08  Jonathan Wakely  <jwakely.gcc@gmail.com>
+
+       PR libstdc++/55908
+       * include/std/functional (_Mem_fn::operator()): Fix constraints to
+       avoid ambiguity.
+       * testsuite/20_util/function_objects/mem_fn/55908.cc: New.
+       * testsuite/20_util/bind/ref_neg.cc: Adjust dg-error line numbers.
+
 2013-01-08  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>
 
        PR libstdc++/55594

diff --git a/libstdc++-v3/include/std/functional b/libstdc++-v3/include/std/functional
index 0b5d47507cad3837928b4ec89f41602e8dca3dc5..714b3ed9ecf4219aa82b2ba05ed233c5357f5d11 100644 (file)
--- a/libstdc++-v3/include/std/functional
+++ b/libstdc++-v3/include/std/functional
@@ -648,7 +648,7 @@ _GLIBCXX_HAS_NESTED_TYPE(result_type)
 
       template<typename _Tp, typename... _Args>
        using _RequireValidArgs2
-         = _Require<_NotSame<_Class, _Tp>, _NotSame<_Class*, _Tp>,
+         = _Require<_NotSame<_Class, _Tp>, _NotSame<const _Class*, _Tp>,
                     _AllConvertible<_Pack<_Args...>, _Pack<_ArgTypes...>>>;
 
       template<typename _Tp, typename... _Args>
@@ -727,7 +727,7 @@ _GLIBCXX_HAS_NESTED_TYPE(result_type)
 
       template<typename _Tp, typename... _Args>
        using _RequireValidArgs2
-         = _Require<_NotSame<_Class, _Tp>, _NotSame<_Class*, _Tp>,
+         = _Require<_NotSame<_Class, _Tp>, _NotSame<volatile _Class*, _Tp>,
                     _AllConvertible<_Pack<_Args...>, _Pack<_ArgTypes...>>>;
 
       template<typename _Tp, typename... _Args>
@@ -807,7 +807,8 @@ _GLIBCXX_HAS_NESTED_TYPE(result_type)
 
       template<typename _Tp, typename... _Args>
        using _RequireValidArgs2
-         = _Require<_NotSame<_Class, _Tp>, _NotSame<_Class*, _Tp>,
+         = _Require<_NotSame<_Class, _Tp>,
+                    _NotSame<const volatile _Class*, _Tp>,
                     _AllConvertible<_Pack<_Args...>, _Pack<_ArgTypes...>>>;
 
       template<typename _Tp, typename... _Args>

diff --git a/libstdc++-v3/testsuite/20_util/bind/ref_neg.cc b/libstdc++-v3/testsuite/20_util/bind/ref_neg.cc
index 9854176547949ca12266fbd83e809948ac42fd42..f65baa0ca5e319131c52fdaca9334c74845ccaee 100644 (file)
--- a/libstdc++-v3/testsuite/20_util/bind/ref_neg.cc
+++ b/libstdc++-v3/testsuite/20_util/bind/ref_neg.cc
@@ -30,10 +30,10 @@ void test01()
 {
   const int dummy = 0;
   std::bind(&inc, _1)(0);               // { dg-error  "no match" }
-  // { dg-error "rvalue|const" "" { target *-*-* } 1346 }
-  // { dg-error "rvalue|const" "" { target *-*-* } 1360 }
-  // { dg-error "rvalue|const" "" { target *-*-* } 1374 }
-  // { dg-error "rvalue|const" "" { target *-*-* } 1388 }
+  // { dg-error "rvalue|const" "" { target *-*-* } 1347 }
+  // { dg-error "rvalue|const" "" { target *-*-* } 1361 }
+  // { dg-error "rvalue|const" "" { target *-*-* } 1375 }
+  // { dg-error "rvalue|const" "" { target *-*-* } 1389 }
   std::bind(&inc, std::ref(dummy))();  // { dg-error  "no match" }
 }
 

diff --git a/libstdc++-v3/testsuite/20_util/function_objects/mem_fn/55908.cc b/libstdc++-v3/testsuite/20_util/function_objects/mem_fn/55908.cc
new file mode 100644 (file)
index 0000000..7bbdb34
--- /dev/null
+++ b/libstdc++-v3/testsuite/20_util/function_objects/mem_fn/55908.cc
@@ -0,0 +1,34 @@
+// { dg-options "-std=gnu++11" }
+// { dg-do compile }
+
+// Copyright (C) 2013 Free Software Foundation, Inc.
+//
+// This file is part of the GNU ISO C++ Library.  This library is free
+// software; you can redistribute it and/or modify it under the
+// terms of the GNU General Public License as published by the
+// Free Software Foundation; either version 3, or (at your option)
+// any later version.
+
+// This library is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+
+// You should have received a copy of the GNU General Public License along
+// with this library; see the file COPYING3.  If not see
+// <http://www.gnu.org/licenses/>.
+
+// PR libstdc++/55908 calling const member function via const pointer
+
+#include <functional>
+
+struct foo
+{
+  void f(int) const { }
+
+  void g() const
+  {
+    auto mf = std::mem_fn(&foo::f);
+    mf(this, 1);
+  }
+};