Simplify with C++17 #46

issue ryanofsky opened this issue on March 15, 2021
  1. ryanofsky commented at 6:34 PM on March 15, 2021: collaborator

    Saving note from an email, there are a ton of a simplifications that can be made now that C++17 is required:

    A lot of code needs better comments, and there are also big swathes that are overcomplicated because they were written to use c++11, and can now be simplified with c++17. Examples:

    • AUTO_RETURN macro uses can be replaced with auto return types
    • Priority<0>, Prioriity<1>, Priority<2> overloads used for conditional compilation can all go away and be replaced with if constexpr
    • std::tuple indexing code can go away and be replaced with std::apply

    Basically all of mp/util.h can go away. Also kj::mvCapture uses can replaced with shorter direct captures.

  2. ryanofsky referenced this in commit bc6624a5d3 on Dec 31, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin-core/libmultiprocess. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-18 13:30 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me