fuzz: Two scripted diff renames #21264

pull MarcoFalke wants to merge 2 commits into bitcoin:master from MarcoFalke:2102-fuzzingRenames changing 36 files +51 −51

MarcoFalke commented at 9:30 AM on February 22, 2021: member
Split out two renames from #21003:
- PROVIDE_FUZZ_MAIN_FUNCTION. Reason: This in only used by fuzzing, so the name should indicate that.
- MakeNoLogFileContext. Reason: Better reflects what the helper does. Also, prepares it to be used in non-fuzz tests in the future.

scripted-diff: Rename PROVIDE_MAIN_FUNCTION -> PROVIDE_FUZZ_MAIN_FUNCTION

-BEGIN VERIFY SCRIPT-
sed -i -e 's/PROVIDE_MAIN_FUNCTION/PROVIDE_FUZZ_MAIN_FUNCTION/g' $(git grep -l PROVIDE_MAIN_FUNCTION)
-END VERIFY SCRIPT-

fa4fbec03e

scripted-diff: Rename MakeFuzzingContext to MakeNoLogFileContext

-BEGIN VERIFY SCRIPT-
 # Rename
 sed -i -e 's/MakeFuzzingContext/MakeNoLogFileContext/g' $(git grep -l MakeFuzzingContext)
 # Bump the copyright of touched files in this scripted diff to avoid touching them again later
 ./contrib/devtools/copyright_header.py update ./src/test/fuzz/
-END VERIFY SCRIPT-

fae216a73d

fanquake added the label Tests on Feb 22, 2021
DrahtBot commented at 1:06 PM on February 22, 2021: member


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



Conflicts

Reviewers, this pull request conflicts with the following ones:
- #21003 (test: Move MakeNoLogFileContext to libtest_util, and use it in bench by MarcoFalke)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
practicalswift commented at 4:59 PM on February 24, 2021: contributor

cr ACK fae216a73d71441432cf07212313d18771ce7deb: scripted-diff looks correct
MarcoFalke merged this on Feb 25, 2021
MarcoFalke closed this on Feb 25, 2021
MarcoFalke deleted the branch on Feb 25, 2021
sidhujag referenced this in commit ea890fd13a on Feb 25, 2021
DrahtBot locked this on Aug 16, 2022

Contributors

Labels

Linked (view graph)

#21003 test: Move MakeNoLogFileContext to libtest_util, and use it in bench

fuzz: Two scripted diff renames #21264

Conflicts