diff options
| author | Kefu Chai <kchai@redhat.com> | 2021-08-17 02:46:08 +0200 |
|---|---|---|
| committer | Kefu Chai <kchai@redhat.com> | 2021-08-17 02:47:33 +0200 |
| commit | 4152cef1e14aa468aed7e32671bb4544f7f7ce17 (patch) | |
| tree | d2748f359b8f5ebc2d04f65acddeb489bde7318c /src/erasure-code | |
| parent | erasure-code/lrc: early return if to_string() fails (diff) | |
| download | ceph-4152cef1e14aa468aed7e32671bb4544f7f7ce17.tar.xz ceph-4152cef1e14aa468aed7e32671bb4544f7f7ce17.zip | |
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>
Diffstat (limited to 'src/erasure-code')
| -rw-r--r-- | src/erasure-code/clay/ErasureCodeClay.cc | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/erasure-code/clay/ErasureCodeClay.cc b/src/erasure-code/clay/ErasureCodeClay.cc index 26854bce82b..1e5a517dee8 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); |