]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
erasure-code/clay: do not set unreferenced variable
authorKefu Chai <kchai@redhat.com>
Tue, 17 Aug 2021 00:46:08 +0000 (08:46 +0800)
committerKefu Chai <kchai@redhat.com>
Tue, 17 Aug 2021 00:47:33 +0000 (08:47 +0800)
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>
src/erasure-code/clay/ErasureCodeClay.cc

index 26854bce82b822d95cb37d1f954f06aa219c0b34..1e5a517dee8e9cff6942b9e4e7a842b26e284c66 100644 (file)
@@ -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);