From: Yehuda Sadeh <yehuda@redhat.com>
Date: Mon, 30 Nov 2015 21:57:56 +0000 (-0800)
Subject: rgw: remove comments
X-Git-Tag: v10.0.2~127^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F6784%2Fhead;p=ceph.git

rgw: remove comments

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
---

diff --git a/src/rgw/rgw_acl_s3.cc b/src/rgw/rgw_acl_s3.cc
index d9130e0d94a3..aecb7a3e66a9 100644
--- a/src/rgw/rgw_acl_s3.cc
+++ b/src/rgw/rgw_acl_s3.cc
@@ -516,7 +516,7 @@ int RGWAccessControlPolicy_S3::rebuild(RGWRados *store, ACLOwner *owner, RGWAcce
           ldout(cct, 0) << "ERROR: src_grant.get_id() failed" << dendl;
           return -EINVAL;
         }
-        email = u.id;  // XXX none of that .to_str() stuff here, but why?
+        email = u.id;
         ldout(cct, 10) << "grant user email=" << email << dendl;
         if (rgw_get_user_info_by_email(store, email, grant_user) < 0) {
           ldout(cct, 10) << "grant user email not found or other error" << dendl;
diff --git a/src/rgw/rgw_bucket.cc b/src/rgw/rgw_bucket.cc
index 456700dfe454..5f83ba45cb41 100644
--- a/src/rgw/rgw_bucket.cc
+++ b/src/rgw/rgw_bucket.cc
@@ -168,7 +168,6 @@ int rgw_bucket_sync_user_stats(RGWRados *store, const string& tenant_name, const
 int rgw_link_bucket(RGWRados *store, const rgw_user& user_id, rgw_bucket& bucket, time_t creation_time, bool update_entrypoint)
 {
   int ret;
-  // XXX Actually, should we use user_id.tenant when creating bucket?
   string& tenant_name = bucket.tenant;
   string& bucket_name = bucket.name;
 
@@ -1770,7 +1769,6 @@ public:
 
     if (!exists || old_bci.info.bucket.bucket_id != bci.info.bucket.bucket_id) {
       /* a new bucket, we need to select a new bucket placement for it */
-      // XXX not sure if this is correct -- stolen from Radoslaw; what about get()?
       string tenant_name;
       string bucket_name;
       parse_bucket(entry, tenant_name, bucket_name);