util: fix compiler warning about deprecated space before _MiB #31691

pull vasild wants to merge 1 commits into bitcoin:master from vasild:deprecated_MiB changing 1 files +1 −1
  1. vasild commented at 1:50 pm on January 20, 2025: contributor
    0src/util/byte_units.h:13:29: error: identifier '_MiB' preceded by whitespace in a literal operator declaration is deprecated [-Werror,-Wdeprecated-literal-operator]
    1   13 | constexpr size_t operator"" _MiB(unsigned long long mebibytes)
    2      |                  ~~~~~~~~~~~^~~~
    3      |                  operator""_MiB
    41 error generated.
    

    Clang 20.0.0

  2. util: fix compiler warning about deprecated space before _MiB
    ```
    src/util/byte_units.h:13:29: error: identifier '_MiB' preceded by whitespace in a literal operator declaration is deprecated [-Werror,-Wdeprecated-literal-operator]
       13 | constexpr size_t operator"" _MiB(unsigned long long mebibytes)
          |                  ~~~~~~~~~~~^~~~
          |                  operator""_MiB
    1 error generated.
    ```
    
    Clang 20.0.0
    d3339a7cd5
  3. DrahtBot commented at 1:50 pm on January 20, 2025: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/31691.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK TheCharlatan, maflcko

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

  4. DrahtBot added the label Utils/log/libs on Jan 20, 2025
  5. TheCharlatan approved
  6. TheCharlatan commented at 1:53 pm on January 20, 2025: contributor
    ACK d3339a7cd5f10e8a6b2a6c64daa66cd71a3e81d3
  7. maflcko commented at 2:06 pm on January 20, 2025: member

    lgtm ACK d3339a7cd5f10e8a6b2a6c64daa66cd71a3e81d3

    There should be an rc1 in two weeks, so the CI could be bumped to that to avoid issues in the future: https://llvm.org/docs/HowToReleaseLLVM.html#annual-release-schedule

  8. fanquake merged this on Jan 20, 2025
  9. fanquake closed this on Jan 20, 2025

  10. vasild deleted the branch on Jan 20, 2025
  11. hebasto commented at 2:38 pm on January 20, 2025: member
    Post-merge ACK d3339a7cd5f10e8a6b2a6c64daa66cd71a3e81d3.
  12. hebasto commented at 3:58 pm on January 20, 2025: member

    There should be an rc1 in two weeks, so the CI could be bumped to that to avoid issues in the future: https://llvm.org/docs/HowToReleaseLLVM.html#annual-release-schedule

    In the meantime, I’ve added a nightly build that uses the LLVM Ubuntu nightly packages.


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2025-01-21 03:12 UTC

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