From: Kefu Chai <kchai@redhat.com>
Date: Tue, 17 Aug 2021 00:46:08 +0000 (+0800)
Subject: erasure-code/clay: do not set unreferenced variable
X-Git-Tag: v17.1.0~1100^2~1
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=4152cef1e14aa468aed7e32671bb4544f7f7ce17;p=ceph.git

erasure-code/clay: do not set unreferenced variable

plane_count was introduced by 2a3e56b22c5b52e4834a9fd5047b3cdff7e8859f,
and presumably it is only used for debugging. let's drop it as it is not
referenced in production.

this change silences following warning:

../src/erasure-code/clay/ErasureCodeClay.cc:502:7: error: variable 'plane_count' set but not used [-Werror,-Wunused-but-set-variable]
  int plane_count = 0;
      ^

Signed-off-by: Kefu Chai <kchai@redhat.com>
---

diff --git a/src/erasure-code/clay/ErasureCodeClay.cc b/src/erasure-code/clay/ErasureCodeClay.cc
index 26854bce82b8..1e5a517dee8e 100644
--- a/src/erasure-code/clay/ErasureCodeClay.cc
+++ b/src/erasure-code/clay/ErasureCodeClay.cc
@@ -499,7 +499,6 @@ int ErasureCodeClay::repair_one_lost_chunk(map<int, bufferlist> &recovered_data,
   }
   assert((unsigned)plane_ind == repair_subchunks);
 
-  int plane_count = 0;
   for (int i = 0; i < q*t; i++) {
     if (U_buf[i].length() == 0) {
       bufferptr buf(buffer::create_aligned(sub_chunk_no*sub_chunksize, SIMD_ALIGN));
@@ -529,7 +528,6 @@ int ErasureCodeClay::repair_one_lost_chunk(map<int, bufferlist> &recovered_data,
     if (ordered_planes.count(order) == 0) {
       break;
     }
-    plane_count += ordered_planes[order].size();
     for (auto z : ordered_planes[order]) {
       get_plane_vector(z, z_vec);