From: John Mulligan <jmulligan@redhat.com>
Date: Mon, 26 Aug 2024 15:03:08 +0000 (-0400)
Subject: test/fio: remove assign-only variable
X-Git-Tag: testing/wip-vshankar-testing-20241219.063429-debug~30^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=4b4473f3bd63a2e5b52ea1a7bbb84af92072c9c6;p=ceph-ci.git

test/fio: remove assign-only variable

This variable in the unit test gets flagged by -Werror which is enabled
by default when compiling the unit tests in a container based
environment. Examining the code the warning appears correct and it
ought to be OK to simply remove this variable.

Signed-off-by: John Mulligan <jmulligan@redhat.com>
---

diff --git a/src/test/fio/fio_librgw.cc b/src/test/fio/fio_librgw.cc
index bac4ff2daac..89c09647b61 100644
--- a/src/test/fio/fio_librgw.cc
+++ b/src/test/fio/fio_librgw.cc
@@ -300,8 +300,6 @@ namespace {
  */
   static void fio_librgw_cleanup(struct thread_data *td)
   {
-    int r = 0;
-
     dprint(FD_IO, "fio_librgw_cleanup\n");
 
     /* cleanup specific data */
@@ -312,9 +310,9 @@ namespace {
       data->release_handles();
 
       if (data->bucket_fh) {
-	r = rgw_fh_rele(data->fs, data->bucket_fh, 0 /* flags */);
+	rgw_fh_rele(data->fs, data->bucket_fh, 0 /* flags */);
       }
-      r = rgw_umount(data->fs, RGW_UMOUNT_FLAG_NONE);
+      rgw_umount(data->fs, RGW_UMOUNT_FLAG_NONE);
       librgw_shutdown(data->rgw_h);
       td->io_ops_data = nullptr;
       delete data;