From 42305044d53c7cb8e3d79032ce4f3abd2c59ef40 Mon Sep 17 00:00:00 2001 From: Yehuda Sadeh Date: Mon, 30 Nov 2015 13:57:56 -0800 Subject: [PATCH] rgw: remove comments Signed-off-by: Yehuda Sadeh --- src/rgw/rgw_acl_s3.cc | 2 +- src/rgw/rgw_bucket.cc | 2 -- 2 files changed, 1 insertion(+), 3 deletions(-) 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); -- 2.47.3