diff options
author | Tianjie Xu <xunchang@google.com> | 2017-06-26 23:56:49 +0200 |
---|---|---|
committer | android-build-merger <android-build-merger@google.com> | 2017-06-26 23:56:49 +0200 |
commit | 170aa1c92370fb572fb1be35553a4faab97c2a8e (patch) | |
tree | 0d71f0f0e6ae2978d5d25c0758726aea06e80c8d | |
parent | Merge "Restructure vr_ui" am: 20d40ce53a (diff) | |
parent | Merge "avoid assuming build number is a 32-bit integer" (diff) | |
download | android_bootable_recovery-170aa1c92370fb572fb1be35553a4faab97c2a8e.tar android_bootable_recovery-170aa1c92370fb572fb1be35553a4faab97c2a8e.tar.gz android_bootable_recovery-170aa1c92370fb572fb1be35553a4faab97c2a8e.tar.bz2 android_bootable_recovery-170aa1c92370fb572fb1be35553a4faab97c2a8e.tar.lz android_bootable_recovery-170aa1c92370fb572fb1be35553a4faab97c2a8e.tar.xz android_bootable_recovery-170aa1c92370fb572fb1be35553a4faab97c2a8e.tar.zst android_bootable_recovery-170aa1c92370fb572fb1be35553a4faab97c2a8e.zip |
-rw-r--r-- | install.cpp | 22 |
1 files changed, 9 insertions, 13 deletions
diff --git a/install.cpp b/install.cpp index db4ba9373..7ba8f0139 100644 --- a/install.cpp +++ b/install.cpp @@ -66,18 +66,14 @@ static constexpr float VERIFICATION_PROGRESS_FRACTION = 0.25; static std::condition_variable finish_log_temperature; // This function parses and returns the build.version.incremental -static int parse_build_number(const std::string& str) { +static std::string parse_build_number(const std::string& str) { size_t pos = str.find('='); if (pos != std::string::npos) { - std::string num_string = android::base::Trim(str.substr(pos+1)); - int build_number; - if (android::base::ParseInt(num_string.c_str(), &build_number, 0)) { - return build_number; - } + return android::base::Trim(str.substr(pos+1)); } LOG(ERROR) << "Failed to parse build number in " << str; - return -1; + return ""; } bool read_metadata_from_package(ZipArchiveHandle zip, std::string* metadata) { @@ -114,14 +110,14 @@ static void read_source_target_build(ZipArchiveHandle zip, std::vector<std::stri for (const std::string& line : lines) { std::string str = android::base::Trim(line); if (android::base::StartsWith(str, "pre-build-incremental")) { - int source_build = parse_build_number(str); - if (source_build != -1) { - log_buffer->push_back(android::base::StringPrintf("source_build: %d", source_build)); + std::string source_build = parse_build_number(str); + if (!source_build.empty()) { + log_buffer->push_back("source_build: " + source_build); } } else if (android::base::StartsWith(str, "post-build-incremental")) { - int target_build = parse_build_number(str); - if (target_build != -1) { - log_buffer->push_back(android::base::StringPrintf("target_build: %d", target_build)); + std::string target_build = parse_build_number(str); + if (!target_build.empty()) { + log_buffer->push_back("target_build: " + target_build); } } } |