future (packaged_task::operator bool): Rename to...

2011-02-09  Jonathan Wakely  <jwakely.gcc@gmail.com>

	* include/std/future (packaged_task::operator bool): Rename to...
	(packaged_task::valid): ...this.
	* testsuite/30_threads/packaged_task/cons/1.cc: Adjust.
	* testsuite/30_threads/packaged_task/cons/2.cc: Adjust.
	* testsuite/30_threads/packaged_task/cons/move.cc: Adjust.
	* testsuite/30_threads/packaged_task/cons/move_assign.cc: Adjust.
	* testsuite/30_threads/packaged_task/cons/alloc.cc: Adjust.
	* testsuite/30_threads/packaged_task/members/invoke.cc: Adjust.
	* testsuite/30_threads/packaged_task/members/reset.cc: Adjust.
	* testsuite/30_threads/packaged_task/members/reset2.cc: Adjust.
	* testsuite/30_threads/packaged_task/members/swap.cc: Adjust.
	* testsuite/30_threads/packaged_task/members/boolconv.cc: Remove.
	* testsuite/30_threads/packaged_task/members/valid.cc: Add.

From-SVN: r169988

12 files changed, 54 insertions, 36 deletions

diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog
index eec7703..243659f 100644
--- a/libstdc++-v3/ChangeLog
+++ b/libstdc++-v3/ChangeLog
@@ -1,3 +1,19 @@
+2011-02-09  Jonathan Wakely  <jwakely.gcc@gmail.com>
+
+	* include/std/future (packaged_task::operator bool): Rename to...
+	(packaged_task::valid): ...this.
+	* testsuite/30_threads/packaged_task/cons/1.cc: Adjust.
+	* testsuite/30_threads/packaged_task/cons/2.cc: Adjust.
+	* testsuite/30_threads/packaged_task/cons/move.cc: Adjust.
+	* testsuite/30_threads/packaged_task/cons/move_assign.cc: Adjust.
+	* testsuite/30_threads/packaged_task/cons/alloc.cc: Adjust.
+	* testsuite/30_threads/packaged_task/members/invoke.cc: Adjust.
+	* testsuite/30_threads/packaged_task/members/reset.cc: Adjust.
+	* testsuite/30_threads/packaged_task/members/reset2.cc: Adjust.
+	* testsuite/30_threads/packaged_task/members/swap.cc: Adjust.
+	* testsuite/30_threads/packaged_task/members/boolconv.cc: Remove.
+	* testsuite/30_threads/packaged_task/members/valid.cc: Add.
+
 2011-02-09  Paolo Carlini  <paolo.carlini@oracle.com>
 
 	* doc/xml/manual/io.xml: Fix typo.
diff --git a/libstdc++-v3/include/std/future b/libstdc++-v3/include/std/future
index f268d4ab..e32b2a0 100644
--- a/libstdc++-v3/include/std/future
+++ b/libstdc++-v3/include/std/future
@@ -1,6 +1,6 @@
 // <future> -*- C++ -*-
 
-// Copyright (C) 2009, 2010 Free Software Foundation, Inc.
+// Copyright (C) 2009, 2010, 2011 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
@@ -1250,7 +1250,9 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
       swap(packaged_task& __other)
       { _M_state.swap(__other._M_state); }
 
-      explicit operator bool() const { return static_cast<bool>(_M_state); }
+      bool
+      valid() const
+      { return static_cast<bool>(_M_state); }
 
       // Result retrieval
       future<_Res>
diff --git a/libstdc++-v3/testsuite/30_threads/packaged_task/cons/1.cc b/libstdc++-v3/testsuite/30_threads/packaged_task/cons/1.cc
index a241433..b6ea492 100644
--- a/libstdc++-v3/testsuite/30_threads/packaged_task/cons/1.cc
+++ b/libstdc++-v3/testsuite/30_threads/packaged_task/cons/1.cc
@@ -6,7 +6,7 @@
 // { dg-require-gthreads "" }
 // { dg-require-atomic-builtins "" }
 
-// Copyright (C) 2009 Free Software Foundation, Inc.
+// Copyright (C) 2009, 2010, 2011 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
@@ -35,15 +35,15 @@ void test01()
   using namespace __gnu_test;
 
   packaged_task<int ()> p1;
-  VERIFY( !static_cast<bool>(p1) );
+  VERIFY( !p1.valid() );
   packaged_task<int& ()> p2;
-  VERIFY( !static_cast<bool>(p2) );
+  VERIFY( !p2.valid() );
   packaged_task<void ()> p3;
-  VERIFY( !static_cast<bool>(p3) );
+  VERIFY( !p3.valid() );
   packaged_task<ClassType ()> p4;
-  VERIFY( !static_cast<bool>(p4) );
+  VERIFY( !p4.valid() );
   packaged_task<AbstractClass& (int)> p5;
-  VERIFY( !static_cast<bool>(p5) );
+  VERIFY( !p5.valid() );
 }
 
 int main()
diff --git a/libstdc++-v3/testsuite/30_threads/packaged_task/cons/2.cc b/libstdc++-v3/testsuite/30_threads/packaged_task/cons/2.cc
index 98f5de7..9dc9757 100644
--- a/libstdc++-v3/testsuite/30_threads/packaged_task/cons/2.cc
+++ b/libstdc++-v3/testsuite/30_threads/packaged_task/cons/2.cc
@@ -6,7 +6,7 @@
 // { dg-require-gthreads "" }
 // { dg-require-atomic-builtins "" }
 
-// Copyright (C) 2009 Free Software Foundation, Inc.
+// Copyright (C) 2009, 2010, 2011 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
@@ -46,15 +46,15 @@ void test01()
   using std::packaged_task;
 
   packaged_task<int ()> p1(f1);
-  VERIFY( static_cast<bool>(p1) );
+  VERIFY( p1.valid() );
   packaged_task<int& ()> p2(f2);
-  VERIFY( static_cast<bool>(p2) );
+  VERIFY( p2.valid() );
   packaged_task<void ()> p3(f3);
-  VERIFY( static_cast<bool>(p3) );
+  VERIFY( p3.valid() );
   packaged_task<ClassType ()> p4(f4);
-  VERIFY( static_cast<bool>(p4) );
+  VERIFY( p4.valid() );
   packaged_task<AbstractClass& (int)> p5(f5);
-  VERIFY( static_cast<bool>(p5) );
+  VERIFY( p5.valid() );
 }
 
 int main()
diff --git a/libstdc++-v3/testsuite/30_threads/packaged_task/cons/alloc.cc b/libstdc++-v3/testsuite/30_threads/packaged_task/cons/alloc.cc
index 99253cd..5318714 100644
--- a/libstdc++-v3/testsuite/30_threads/packaged_task/cons/alloc.cc
+++ b/libstdc++-v3/testsuite/30_threads/packaged_task/cons/alloc.cc
@@ -6,7 +6,7 @@
 // { dg-require-gthreads "" }
 // { dg-require-atomic-builtins "" }
 
-// Copyright (C) 2010 Free Software Foundation, Inc.
+// Copyright (C) 2010, 2011 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
@@ -40,7 +40,7 @@ void test01()
   uneq_allocator<char> alloc(99);
 
   packaged_task<int ()> p1(allocator_arg, alloc, f);
-  VERIFY( static_cast<bool>(p1) );
+  VERIFY( p1.valid() );
   p1();
   VERIFY( p1.get_future().get() == 5 );
 }
diff --git a/libstdc++-v3/testsuite/30_threads/packaged_task/cons/move.cc b/libstdc++-v3/testsuite/30_threads/packaged_task/cons/move.cc
index 5335db3..599caf9 100644
--- a/libstdc++-v3/testsuite/30_threads/packaged_task/cons/move.cc
+++ b/libstdc++-v3/testsuite/30_threads/packaged_task/cons/move.cc
@@ -6,7 +6,7 @@
 // { dg-require-gthreads "" }
 // { dg-require-atomic-builtins "" }
 
-// Copyright (C) 2009 Free Software Foundation, Inc.
+// Copyright (C) 2009, 2010, 2011 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
@@ -37,8 +37,8 @@ void test01()
   // move
   packaged_task<int()> p1(f1);
   packaged_task<int()> p2(std::move(p1));
-  VERIFY( !static_cast<bool>(p1) );
-  VERIFY( static_cast<bool>(p2) );
+  VERIFY( !p1.valid() );
+  VERIFY( p2.valid() );
 }
 
 int main()
diff --git a/libstdc++-v3/testsuite/30_threads/packaged_task/cons/move_assign.cc b/libstdc++-v3/testsuite/30_threads/packaged_task/cons/move_assign.cc
index c23e5e3..1c5dfc3 100644
--- a/libstdc++-v3/testsuite/30_threads/packaged_task/cons/move_assign.cc
+++ b/libstdc++-v3/testsuite/30_threads/packaged_task/cons/move_assign.cc
@@ -6,7 +6,7 @@
 // { dg-require-gthreads "" }
 // { dg-require-atomic-builtins "" }
 
-// Copyright (C) 2009 Free Software Foundation, Inc.
+// Copyright (C) 2009, 2011 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
@@ -37,8 +37,8 @@ void test01()
   std::packaged_task<int()> p1;
   std::packaged_task<int()> p2(gen);
   p1 = std::move(p2);
-  VERIFY( static_cast<bool>(p1) );
-  VERIFY( !static_cast<bool>(p2) );
+  VERIFY( p1.valid() );
+  VERIFY( !p2.valid() );
 }
 
 int main()
diff --git a/libstdc++-v3/testsuite/30_threads/packaged_task/members/invoke.cc b/libstdc++-v3/testsuite/30_threads/packaged_task/members/invoke.cc
index cce7b54..238568f 100644
--- a/libstdc++-v3/testsuite/30_threads/packaged_task/members/invoke.cc
+++ b/libstdc++-v3/testsuite/30_threads/packaged_task/members/invoke.cc
@@ -6,7 +6,7 @@
 // { dg-require-gthreads "" }
 // { dg-require-atomic-builtins "" }
 
-// Copyright (C) 2009 Free Software Foundation, Inc.
+// Copyright (C) 2009, 2010, 2011 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
@@ -38,7 +38,7 @@ void test01()
 
   p1();
 
-  VERIFY( static_cast<bool>(p1) );
+  VERIFY( p1.valid() );
   VERIFY( f1.get() == 0 );
 }
 
diff --git a/libstdc++-v3/testsuite/30_threads/packaged_task/members/reset.cc b/libstdc++-v3/testsuite/30_threads/packaged_task/members/reset.cc
index a126c3a..01a06d6 100644
--- a/libstdc++-v3/testsuite/30_threads/packaged_task/members/reset.cc
+++ b/libstdc++-v3/testsuite/30_threads/packaged_task/members/reset.cc
@@ -6,7 +6,7 @@
 // { dg-require-gthreads "" }
 // { dg-require-atomic-builtins "" }
 
-// Copyright (C) 2009 Free Software Foundation, Inc.
+// Copyright (C) 2009. 2010, 2011 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
@@ -39,7 +39,7 @@ void test01()
   future<int> f1 = p1.get_future();
 
   p1.reset();
-  VERIFY( static_cast<bool>(p1) );
+  VERIFY( p1.valid() );
 
   future<int> f2 = p1.get_future();
 
diff --git a/libstdc++-v3/testsuite/30_threads/packaged_task/members/reset2.cc b/libstdc++-v3/testsuite/30_threads/packaged_task/members/reset2.cc
index e38d047..2fac366 100644
--- a/libstdc++-v3/testsuite/30_threads/packaged_task/members/reset2.cc
+++ b/libstdc++-v3/testsuite/30_threads/packaged_task/members/reset2.cc
@@ -6,7 +6,7 @@
 // { dg-require-gthreads "" }
 // { dg-require-atomic-builtins "" }
 
-// Copyright (C) 2009 Free Software Foundation, Inc.
+// Copyright (C) 2009, 2010, 2011 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
@@ -39,7 +39,7 @@ void test01()
   p1();
   p1.reset();
 
-  VERIFY( static_cast<bool>(p1) );
+  VERIFY( p1.valid() );
   VERIFY( f1.get() == 0 );
 
   std::future<int> f2 = p1.get_future();
diff --git a/libstdc++-v3/testsuite/30_threads/packaged_task/members/swap.cc b/libstdc++-v3/testsuite/30_threads/packaged_task/members/swap.cc
index ced2a00..9055d98 100644
--- a/libstdc++-v3/testsuite/30_threads/packaged_task/members/swap.cc
+++ b/libstdc++-v3/testsuite/30_threads/packaged_task/members/swap.cc
@@ -6,7 +6,7 @@
 // { dg-require-gthreads "" }
 // { dg-require-atomic-builtins "" }
 
-// Copyright (C) 2009 Free Software Foundation, Inc.
+// Copyright (C) 2009, 2010, 2011 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
@@ -35,11 +35,11 @@ void test01()
 
   std::packaged_task<int()> p1(zero);
   std::packaged_task<int()> p2;
-  VERIFY( static_cast<bool>(p1) );
-  VERIFY( !static_cast<bool>(p2) );
+  VERIFY( p1.valid() );
+  VERIFY( !p2.valid() );
   p1.swap(p2);
-  VERIFY( !static_cast<bool>(p1) );
-  VERIFY( static_cast<bool>(p2) );
+  VERIFY( !p1.valid() );
+  VERIFY( p2.valid() );
 }
 
 int main()
diff --git a/libstdc++-v3/testsuite/30_threads/packaged_task/members/boolconv.cc b/libstdc++-v3/testsuite/30_threads/packaged_task/members/valid.cc
index c332b85..d7f1ed1 100644
--- a/libstdc++-v3/testsuite/30_threads/packaged_task/members/boolconv.cc
+++ b/libstdc++-v3/testsuite/30_threads/packaged_task/members/valid.cc
@@ -6,7 +6,7 @@
 // { dg-require-gthreads "" }
 // { dg-require-atomic-builtins "" }
 
-// Copyright (C) 2009 Free Software Foundation, Inc.
+// Copyright (C) 2011 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
@@ -34,10 +34,10 @@ void test01()
   bool test __attribute__((unused)) = true;
 
   std::packaged_task<int()> p1;
-  VERIFY( !static_cast<bool>(p1) );
+  VERIFY( !p1.valid() );
 
   std::packaged_task<int()> p2(zero);
-  VERIFY( static_cast<bool>(p2) );
+  VERIFY( p2.valid() );
 }
 
 int main()