std::atomic is used in the file, so to avoid compile issues, add the missing include.
blockstorage: Add missing atomic include #22309
pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2106-include changing 1 files +4 −3-
MarcoFalke commented at 9:08 AM on June 22, 2021: member
-
add missing atomic include fa2d21fec8
- MarcoFalke added the label Refactoring on Jun 22, 2021
- MarcoFalke added the label Block storage on Jun 22, 2021
- MarcoFalke added this to the milestone 22.0 on Jun 22, 2021
- hebasto approved
-
hebasto commented at 11:02 AM on June 22, 2021: member
ACK fa2d21fec8e532aa0781211e198ee6098f89350e, I have reviewed the code and it looks OK, I agree it can be merged.
-
practicalswift commented at 12:45 PM on June 22, 2021: contributor
cr ACK fa2d21fec8e532aa0781211e198ee6098f89350e
- unknown approved
-
unknown commented at 1:39 PM on June 22, 2021: none
crACK https://github.com/bitcoin/bitcoin/pull/22309/commits/fa2d21fec8e532aa0781211e198ee6098f89350e
Confirmed
std::atomic_boolis used at Line 38 and 39 which requires#include <atomic>https://www.cplusplus.com/reference/atomic/atomic/atomic/ https://en.cppreference.com/w/cpp/atomic/atomic
-
jamesob commented at 1:44 PM on June 22, 2021: member
- MarcoFalke merged this on Jun 22, 2021
- MarcoFalke closed this on Jun 22, 2021
- MarcoFalke deleted the branch on Jun 22, 2021
- sidhujag referenced this in commit 8852f3b720 on Jun 24, 2021
- gwillen referenced this in commit c86aff975c on Jun 1, 2022
- DrahtBot locked this on Aug 18, 2022
