Fix some benign races in timestamp logging #9230

pull TheBlueMatt wants to merge 2 commits into bitcoin:master from TheBlueMatt:2016-11-loglocks changing 2 files +4 −3

TheBlueMatt commented at 11:37 PM on November 27, 2016: member

No description provided.
Fix race when accessing std::locale::classic()
See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78552
507145d785

Make fStartedNewLine an std::atomic_bool

While this doesnt really fix the race of adding timestamps
mid-logical-line, it avoids the undefined behavior of using a
bool in multiple threads.

8b22efb6f7

dcousens approved
fanquake added the label Refactoring on Nov 27, 2016
sipa commented at 7:22 AM on November 28, 2016: member

utACK 8b22efb6f7c406951f33a04e84377fd16f02121c
jonasschnelli commented at 7:37 AM on November 28, 2016: contributor

utACK 8b22efb6f7c406951f33a04e84377fd16f02121c
gmaxwell commented at 8:03 AM on November 28, 2016: contributor

utACK.
paveljanik commented at 8:11 AM on November 28, 2016: contributor

utACK 8b22efb
laanwj merged this on Dec 1, 2016
laanwj closed this on Dec 1, 2016
laanwj referenced this in commit c79e52ad30 on Dec 1, 2016
codablock referenced this in commit d60a0704d4 on Jan 16, 2018
codablock referenced this in commit 4f002e428a on Jan 16, 2018
codablock referenced this in commit 3b2fd85b59 on Jan 17, 2018
andvgal referenced this in commit d7b7ca9f1d on Jan 6, 2019
CryptoCentric referenced this in commit 1d0226fe90 on Feb 25, 2019
LarryRuane referenced this in commit 74b484e112 on Feb 24, 2021
LarryRuane referenced this in commit cafc3f845d on Apr 1, 2021
zkbot referenced this in commit 1b5f17c900 on Apr 1, 2021
zkbot referenced this in commit 80e66e7daa on Apr 2, 2021
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels