google / marl

A hybrid thread / fiber task scheduler written in C++ 11
Apache License 2.0
1.88k stars 194 forks source link

Explicitly #include <utility> in finally.h #242

Closed alanzhao1 closed 1 year ago

alanzhao1 commented 1 year ago

finally.h uses std::move(...), which is declared in the header. This currently compiles because std::move is provided by transitive libc++ modules, but this transitive include is not guaranteed to exist, as was found when compiling chrome with Clang modules (https://crbug.com/543704).