refactor: Drop unused CBufferedFile::Seek() #19593

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:200726-seek changing 2 files +4 −20

hebasto commented at 7:50 PM on July 26, 2020: member

No description provided.
refactor: Drop unused CBufferedFile::Seek() 7b3851e947
DrahtBot added the label Refactoring on Jul 26, 2020
DrahtBot added the label Tests on Jul 26, 2020
practicalswift commented at 10:52 PM on July 26, 2020: contributor

ACK 7b3851e9473e74043342d414c056c2ef87c2f261 -- deleted code is better than unused untested code:)
promag commented at 10:55 PM on July 26, 2020: member

Code review ACK 7b3851e9473e74043342d414c056c2ef87c2f261. @practicalswift agree, see #19391.
MarcoFalke removed the label Tests on Jul 27, 2020
MarcoFalke commented at 5:37 AM on July 27, 2020: member

ACK 7b3851e9473e74043342d414c056c2ef87c2f261, assuming that removing this should either be correct or result in a compile failure
jonasschnelli commented at 7:39 AM on July 27, 2020: contributor

utACK 7b3851e9473e74043342d414c056c2ef87c2f261
laanwj merged this on Jul 27, 2020
laanwj closed this on Jul 27, 2020
hebasto deleted the branch on Jul 27, 2020
LarryRuane cross-referenced this on Dec 7, 2020 from issue util: improve FindByte() performance by LarryRuane
Fabcien referenced this in commit 29403dd84c on Feb 3, 2021
furszy cross-referenced this on Jun 7, 2021 from issue Road to Tor v3 support (BIP155) by furszy
random-zebra referenced this in commit b4751e10ce on Aug 11, 2021
bitcoin locked this on Feb 15, 2022

Contributors

Labels