commit f299342e38fd34b589604ef5b730623da13ee235
author Daniel Thornburgh <dthorn@google.com> Tue Apr 19 21:39:36 2022 +0000
committer Boringssl LUCI CQ <boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Apr 21 22:33:05 2022 +0000
tree 2821bfc9d3eaa5b86a33454b541a9631f1463753
parent 6686352e492b67cb4d57915fc9bca45cdc7cef16

[build] Fix build with HEAD clang.

The HEAD version of Clang fails to build boringssl due to the following
warning triggered at -Werror:

    /usr/local/google/home/dthorn/curl/boringssl/tool/digest.cc:229:12:
    error: variable 'bad_hash_lines' set but not used

This change removes the variable.

Change-Id: I25009925d9a2dfc5b1783accab19e4b861db4ec2
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/52265
Reviewed-by: Adam Langley <agl@google.com>
Commit-Queue: Adam Langley <agl@google.com>

tool/digest.cc

diff --git a/tool/digest.cc b/tool/digest.cc
index 3c8fd5a..d810928 100644
--- a/tool/digest.cc
+++ b/tool/digest.cc
@@ -226,7 +226,6 @@
   unsigned bad_lines = 0;
   unsigned parsed_lines = 0;
   unsigned error_lines = 0;
-  unsigned bad_hash_lines = 0;
   unsigned line_no = 0;
   bool ok = true;
   bool draining_overlong_line = false;
@@ -297,7 +296,6 @@
     }
 
     if (calculated_hex_digest != std::string(line, hex_size)) {
-      bad_hash_lines++;
       if (!args.status) {
         printf("%s: FAILED\n", target_filename.c_str());
       }