diff options
author | Samuel Huang <huangs@chromium.org> | 2018-03-21 18:54:03 +0000 |
---|---|---|
committer | Edward Lesmes <ehmaldonado@google.com> | 2021-07-23 21:58:36 +0000 |
commit | 1bb3b651b14c9cec5cf0104a1dec1550b9c22983 (patch) | |
tree | 6c50b80ae3ef0e87d09a70a233e9c37f3e0ff4e1 /main_utils.cc | |
parent | 6d057299c73d1c69ddd3084eb7cd1d25d8c70002 (diff) | |
download | zucchini-1bb3b651b14c9cec5cf0104a1dec1550b9c22983.tar.gz |
[Zucchini] Add "-keep" flag; remove redundnat CRC32 check.
By defaulti, Zucchini deletes the output (patch or patched "new") file
upon failure. The "-keep" debug flag overrides this behavior, so the
bad output file can be examined. This CL adds the flag for Zucchini-gen
and Zucchini-apply.
This CL also removes a duplicated call to Ensemble::CheckOldFile(),
which checks the CRC32 of the "old" file. This removal prevents
redundant work and slightly speed up Zucchini-apply (by ~5%).
Bug: 729154
Change-Id: I9ba8e9b00b7783661de06b39fae010761bedc6e6
Reviewed-on: https://chromium-review.googlesource.com/973746
Reviewed-by: agrieve <agrieve@chromium.org>
Commit-Queue: Samuel Huang <huangs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#544786}
NOKEYCHECK=True
GitOrigin-RevId: 21879c3adbfbbf676e5ee28b9f0a19e60d104d80
Diffstat (limited to 'main_utils.cc')
-rw-r--r-- | main_utils.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/main_utils.cc b/main_utils.cc index 3a7055a..7a28388 100644 --- a/main_utils.cc +++ b/main_utils.cc @@ -63,8 +63,10 @@ struct Command { /******** List of Zucchini commands ********/ constexpr Command kCommands[] = { - {"gen", "-gen <old_file> <new_file> <patch_file> [-raw]", 3, &MainGen}, - {"apply", "-apply <old_file> <patch_file> <new_file>", 3, &MainApply}, + {"gen", "-gen <old_file> <new_file> <patch_file> [-raw] [-keep]", 3, + &MainGen}, + {"apply", "-apply <old_file> <patch_file> <new_file> [-keep]", 3, + &MainApply}, {"read", "-read <exe> [-dump]", 1, &MainRead}, {"detect", "-detect <archive_file> [-dd=format#]", 1, &MainDetect}, {"match", "-match <old_file> <new_file>", 2, &MainMatch}, |