GetTimeSeconds() removal followups #25155

pull jonatack wants to merge 1 commits into bitcoin:master from jonatack:GetTimeSeconds-removal-followups changing 3 files +4 −3

jonatack commented at 12:25 PM on May 17, 2022: member

A few cleanup followups to recently merged #25102 (and #24595).

Edit: dropped commit Update GetTime() doxygen doc for removed GetTimeSeconds() as it would be fixed in #25101.
jonatack force-pushed on May 17, 2022
DrahtBot added the label Utils/log/libs on May 17, 2022
in src/util/time.h:51 in 18cd67a2ac outdated
```
  47 | @@ -48,7 +48,7 @@ inline double CountSecondsDouble(SecondsDouble t) { return t.count(); }
  48 |  
  49 |  /**
  50 |   * DEPRECATED
  51 | - * Use either GetTimeSeconds (not mockable) or GetTime<T> (mockable)
  52 | + * Use GetTime<T> (mockable)
```
MarcoFalke commented at 1:35 PM on May 17, 2022:

See https://github.com/bitcoin/bitcoin/pull/25101/files#diff-5e3737e32cc88e35709e8dc031417044edc21e77481865bb51f57a8e24dd1b0e

jonatack commented at 1:49 PM on May 17, 2022:

Dropped that commit.
Update and appease the spelling linter 1f1ed8d30c
jonatack force-pushed on May 17, 2022
jonatack commented at 1:51 PM on May 17, 2022: member

Closing, as the fix is already handled in another open pull. Leaving the remaining small fixes as up for grabs.
jonatack closed this on May 17, 2022
DrahtBot locked this on May 17, 2023

Contributors

Labels