File: foreach.patch

package info (click to toggle)
chromium 142.0.7444.175-1
  • links: PTS, VCS
  • area: main
  • in suites: forky, sid
  • size: 6,295,352 kB
  • sloc: cpp: 35,488,378; ansic: 7,479,680; javascript: 4,259,373; python: 1,466,843; xml: 757,444; asm: 710,716; pascal: 187,980; sh: 89,247; perl: 88,690; objc: 79,984; sql: 56,984; cs: 42,192; fortran: 24,137; makefile: 22,913; tcl: 15,277; php: 14,018; yacc: 9,005; ruby: 7,553; awk: 3,720; lisp: 3,096; lex: 1,330; ada: 727; jsp: 228; sed: 36
file content (31 lines) | stat: -rw-r--r-- 1,413 bytes parent folder | download | duplicates (5)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
author: Andres Salomon <dilinger@debian.org>

work around an issue with libstdc++-12, where clang gets confused and can't
find base::Value::Dict's Append when inside a lambda inside of for_each:

In file included from ../../components/services/app_service/public/cpp/permission.cc:9:
../../base/containers/to_value_list.h:39:46: error: no matching member function for call to 'Append'
   39 |       [&]<typename T>(T&& value) { container.Append(std::forward<T>(value)); },
      |                                    ~~~~~~~~~~^~~~~~


Rather than call for_each, we iterate manually (but still call the lambda).

--- a/base/containers/to_value_list.h
+++ b/base/containers/to_value_list.h
@@ -34,10 +34,11 @@ template <typename Range, typename Proj
                std::indirect_result_t<Proj, std::ranges::iterator_t<Range>>>
 Value::List ToValueList(Range&& range, Proj proj = {}) {
   auto container = Value::List::with_capacity(std::ranges::size(range));
-  std::ranges::for_each(
-      std::forward<Range>(range),
-      [&]<typename T>(T&& value) { container.Append(std::forward<T>(value)); },
-      std::move(proj));
+  auto first = std::ranges::begin(range);
+  auto last = std::ranges::end(range);
+  auto f = [&]<typename T>(T&& value) { container.Append(std::forward<T>(value)); };
+  for (; first != last; ++first)
+    std::invoke(f, std::invoke(std::move(proj), *first));
   return container;
 }