]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
rgw: kill a compile warning for rgw_sync 10425/head
authorAbhishek Lekshmanan <abhishek@suse.com>
Mon, 25 Jul 2016 09:21:11 +0000 (11:21 +0200)
committerAbhishek Lekshmanan <abhishek@suse.com>
Wed, 10 Aug 2016 16:33:27 +0000 (18:33 +0200)
killing the compile warning for

```
/ceph/src/rgw/rgw_sync.cc:1462:12:
warning: suggest explicit braces to avoid ambiguous ‘else’ [-Wparentheses]
         if (can_adjust_marker) yield {
            ^
```

Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
Signed-off-by: Casey Bodley <cbodley@redhat.com>
src/rgw/rgw_sync.cc

index f7b419be77f9b04a7cff9cc603158b19b8d4a545..e0506650f77c7ab82dde545614f076aee445dccc 100644 (file)
@@ -1459,18 +1459,18 @@ public:
 
       if (!lost_lock) {
         /* update marker to reflect we're done with full sync */
-        if (can_adjust_marker) yield {
-          sync_marker.state = rgw_meta_sync_marker::IncrementalSync;
-          sync_marker.marker = sync_marker.next_step_marker;
-          sync_marker.next_step_marker.clear();
-
-          RGWRados *store = sync_env->store;
-          ldout(sync_env->cct, 0) << *this << ": saving marker pos=" << sync_marker.marker << dendl;
-          using WriteMarkerCR = RGWSimpleRadosWriteCR<rgw_meta_sync_marker>;
-          call(new WriteMarkerCR(sync_env->async_rados, store, pool,
-                                 sync_env->shard_obj_name(shard_id),
-                                 sync_marker));
+        if (can_adjust_marker) {
+         sync_marker.state = rgw_meta_sync_marker::IncrementalSync;
+         sync_marker.marker = sync_marker.next_step_marker;
+         sync_marker.next_step_marker.clear();
+         ldout(sync_env->cct, 0) << *this << ": saving marker pos=" << sync_marker.marker << dendl;
+
+         using WriteMarkerCR = RGWSimpleRadosWriteCR<rgw_meta_sync_marker>;
+         yield call(new WriteMarkerCR(sync_env->async_rados, sync_env->store,
+                                      pool, sync_env->shard_obj_name(shard_id),
+                                      sync_marker));
         }
+
         if (retcode < 0) {
           ldout(sync_env->cct, 0) << "ERROR: failed to set sync marker: retcode=" << retcode << dendl;
           return retcode;