revert: stl_algobase.h (max, min): Use conditional operator.
authorPaolo Carlini <paolo.carlini@oracle.com>
Sun, 23 Sep 2012 19:58:16 +0000 (19:58 +0000)
committerPaolo Carlini <paolo@gcc.gnu.org>
Sun, 23 Sep 2012 19:58:16 +0000 (19:58 +0000)
2012-09-23  Paolo Carlini  <paolo.carlini@oracle.com>

Revert:
2012-09-21  Paolo Carlini  <paolo.carlini@oracle.com>

* include/bits/stl_algobase.h (max, min): Use conditional operator.

From-SVN: r191652

libstdc++-v3/ChangeLog
libstdc++-v3/include/bits/stl_algobase.h

index 606f5658786f7280fdd8ad6a3388d21a93955c6c..70ee2a88e1beff9f6323922d122cceb6e3c0cc11 100644 (file)
@@ -1,3 +1,10 @@
+2012-09-23  Paolo Carlini  <paolo.carlini@oracle.com>
+
+       Revert:
+       2012-09-21  Paolo Carlini  <paolo.carlini@oracle.com>
+
+       * include/bits/stl_algobase.h (max, min): Use conditional operator.
+
 2012-09-23  Uros Bizjak  <ubizjak@gmail.com>
 
        PR libstdc++/54675
index 3099d0d10f6c1a2272583b67d253878943b353e7..fe30f6ce9f5e7c2931cfb424257f2e27ca4a7650 100644 (file)
@@ -195,8 +195,10 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
     {
       // concept requirements
       __glibcxx_function_requires(_LessThanComparableConcept<_Tp>)
-
-      return __b < __a ? __b : __a;
+      //return __b < __a ? __b : __a;
+      if (__b < __a)
+       return __b;
+      return __a;
     }
 
   /**
@@ -216,8 +218,10 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
     {
       // concept requirements
       __glibcxx_function_requires(_LessThanComparableConcept<_Tp>)
-
-      return  __a < __b ? __b : __a;
+      //return  __a < __b ? __b : __a;
+      if (__a < __b)
+       return __b;
+      return __a;
     }
 
   /**
@@ -234,7 +238,12 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
   template<typename _Tp, typename _Compare>
     inline const _Tp&
     min(const _Tp& __a, const _Tp& __b, _Compare __comp)
-    { return __comp(__b, __a) ? __b : __a; }
+    {
+      //return __comp(__b, __a) ? __b : __a;
+      if (__comp(__b, __a))
+       return __b;
+      return __a;
+    }
 
   /**
    *  @brief This does what you think it does.
@@ -250,7 +259,12 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
   template<typename _Tp, typename _Compare>
     inline const _Tp&
     max(const _Tp& __a, const _Tp& __b, _Compare __comp)
-    { return __comp(__a, __b) ? __b : __a; }
+    {
+      //return __comp(__a, __b) ? __b : __a;
+      if (__comp(__a, __b))
+       return __b;
+      return __a;
+    }
 
   // If _Iterator is a __normal_iterator return its base (a plain pointer,
   // normally) otherwise return it untouched.  See copy, fill, ...