]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
test/fio: remove assign-only variable 61031/head
authorJohn Mulligan <jmulligan@redhat.com>
Mon, 26 Aug 2024 15:03:08 +0000 (11:03 -0400)
committerJohn Mulligan <jmulligan@redhat.com>
Tue, 10 Dec 2024 21:25:53 +0000 (16:25 -0500)
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>
src/test/fio/fio_librgw.cc

index bac4ff2daac671e974d04c5c6f3bf5aba86d66bc..89c09647b615816327873239d51e8ec9affe4d6d 100644 (file)
@@ -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;