diff options
author | Calder Kitagawa <ckitagawa@google.com> | 2018-03-14 19:28:36 +0000 |
---|---|---|
committer | Edward Lesmes <ehmaldonado@google.com> | 2021-07-23 21:53:02 +0000 |
commit | 63974a2cc4de484d94537ef1360a801a52a7c37d (patch) | |
tree | c10323b4734f7cf8d2fb70f26b41f85354c79da7 | |
parent | 56a6ff47bca7087db1916b2beeedee283cd1caf2 (diff) | |
download | zucchini-63974a2cc4de484d94537ef1360a801a52a7c37d.tar.gz |
[Zucchini]: Cleanup some issues found when merging to trunk.
A few simple changes/typos that require correction which were detected
when merging changes into trunk.
Change-Id: Ib088ca4a1741e642e3d912d010e4e77ba5fb72b0
Reviewed-on: https://chromium-review.googlesource.com/962357
Commit-Queue: Calder Kitagawa <ckitagawa@google.com>
Reviewed-by: Samuel Huang <huangs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543155}
NOKEYCHECK=True
GitOrigin-RevId: 3135d7d1cb10ef7d7f94da864643da26ed5a9c84
-rw-r--r-- | zucchini_commands.cc | 2 | ||||
-rw-r--r-- | zucchini_integration.cc | 13 |
2 files changed, 8 insertions, 7 deletions
diff --git a/zucchini_commands.cc b/zucchini_commands.cc index 90208a0..53b5bd7 100644 --- a/zucchini_commands.cc +++ b/zucchini_commands.cc @@ -144,7 +144,7 @@ zucchini::status::Code MainMatch(MainParams params) { } File new_file(params.file_paths[1], File::FLAG_OPEN | File::FLAG_READ); zucchini::MappedFileReader new_image(std::move(new_file)); - if (old_image.HasError()) { + if (new_image.HasError()) { LOG(ERROR) << "Error with file " << params.file_paths[1].value() << ": " << new_image.error(); return zucchini::status::kStatusFileReadError; diff --git a/zucchini_integration.cc b/zucchini_integration.cc index 3ca4601..2d78fdd 100644 --- a/zucchini_integration.cc +++ b/zucchini_integration.cc @@ -17,8 +17,7 @@ namespace { struct FileNames { FileNames() : is_dummy(true) { - // Use fake names. If |is_dummy| is true these files are only used for error - // output. + // Use fake names. old_name = old_name.AppendASCII("old_name"); patch_name = patch_name.AppendASCII("patch_name"); new_name = new_name.AppendASCII("new_name"); @@ -35,6 +34,8 @@ struct FileNames { base::FilePath old_name; base::FilePath patch_name; base::FilePath new_name; + + // A flag to decide whether the filenames are only for error output. const bool is_dummy; }; @@ -97,10 +98,10 @@ status::Code ApplyCommon(base::File&& old_file_handle, } // namespace -status::Code Apply(base::File old_file_handle, - base::File patch_file_handle, - base::File new_file_handle) { - const FileNames file_names = FileNames(); +status::Code Apply(base::File&& old_file_handle, + base::File&& patch_file_handle, + base::File&& new_file_handle) { + const FileNames file_names; return ApplyCommon(std::move(old_file_handle), std::move(patch_file_handle), std::move(new_file_handle), file_names); } |