Remove usage of std::binary_function

std::binary_function is removed in C++17 and the latest version of libc++, and its presence is preventing Chrome from updating libc++.
google · Jun 27, 2022 · e2012f1 · e2012f1
1 parent a56be6f
commit e2012f1
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/cpp/src/preload_supplier.cc b/cpp/src/preload_supplier.cc
@@ -22,7 +22,6 @@
 #include <algorithm>
 #include <cassert>
 #include <cstddef>
-#include <functional>
 #include <map>
 #include <memory>
 #include <set>
@@ -47,10 +46,9 @@ namespace {
 // reader would consider to be "the same". The default implementation just does
 // case insensitive string comparison, but StringCompare can be overridden with
 // more sophisticated implementations.
-class IndexLess : public std::binary_function<std::string, std::string, bool> {
+class IndexLess {
  public:
-  result_type operator()(const first_argument_type& a,
-                         const second_argument_type& b) const {
+  bool operator()(const std::string& a, const std::string& b) const {
     static const StringCompare kStringCompare;
     return kStringCompare.NaturalLess(a, b);
   }