]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
rgw: clean up some compiler warnings 39585/head
authorJ. Eric Ivancich <ivancich@redhat.com>
Fri, 19 Feb 2021 21:28:58 +0000 (16:28 -0500)
committerJ. Eric Ivancich <ivancich@redhat.com>
Fri, 19 Feb 2021 21:28:58 +0000 (16:28 -0500)
Minimal changes to clean up a few compiler warnings.

Signed-off-by: J. Eric Ivancich <ivancich@redhat.com>
src/rgw/rgw_cr_tools.cc
src/rgw/rgw_rados.cc
src/rgw/rgw_reshard.cc

index e3b8d42aeee3423d90eb2357c6b0f044b0ebe238..76428a0d3908b0609fb2b8a2ce8a1e5dff85cb98 100644 (file)
@@ -278,8 +278,6 @@ int RGWBucketLifecycleConfigCR::Request::_send_request()
 template<>
 int RGWBucketGetSyncPolicyHandlerCR::Request::_send_request()
 {
-  CephContext *cct = store->ctx();
-
   int r = store->ctl()->bucket->get_sync_policy_handler(params.zone,
                                                         params.bucket,
                                                         &result->policy_handler,
index 9c66c3dc67251d261002589dad54267d77b8291c..97c968ea793ce4193d11d865582411f0424d1aed 100644 (file)
@@ -4727,10 +4727,11 @@ int RGWRados::set_buckets_enabled(vector<rgw_bucket>& buckets, bool enabled, con
 
   for (iter = buckets.begin(); iter != buckets.end(); ++iter) {
     rgw_bucket& bucket = *iter;
-    if (enabled)
+    if (enabled) {
       ldpp_dout(dpp, 20) << "enabling bucket name=" << bucket.name << dendl;
-    else
+    } else {
       ldpp_dout(dpp, 20) << "disabling bucket name=" << bucket.name << dendl;
+    }
 
     RGWBucketInfo info;
     map<string, bufferlist> attrs;
@@ -5455,10 +5456,11 @@ int RGWRados::get_obj_state_impl(const DoutPrefixProvider *dpp, RGWObjectCtx *rc
       }
     }
   }
-  if (s->obj_tag.length())
+  if (s->obj_tag.length()) {
     ldpp_dout(dpp, 20) << "get_obj_state: setting s->obj_tag to " << s->obj_tag.c_str() << dendl;
-  else
+  } else {
     ldpp_dout(dpp, 20) << "get_obj_state: s->obj_tag was set empty" << dendl;
+  }
 
   /* an object might not be olh yet, but could have olh id tag, so we should set it anyway if
    * it exist, and not only if is_olh() returns true
index af7037fb3da48e7c5cfeb28b284327749bf3abbb..9f85814a296579f96cdd8ade1910f4ae616b1508 100644 (file)
@@ -990,7 +990,6 @@ int RGWReshard::process_single_logshard(int logshard_num, const DoutPrefixProvid
   string marker;
   bool truncated = true;
 
-  CephContext *cct = store->ctx();
   constexpr uint32_t max_entries = 1000;
 
   string logshard_oid;