From bda21f726473ba78bac86db39f52451c227623da Mon Sep 17 00:00:00 2001 From: Danny Al-Gaaf Date: Tue, 2 Apr 2013 16:17:52 +0200 Subject: [PATCH] rgw/rgw_user.cc: use 'true' directly instead of variable Instead of passing 'true' via bool defer_user_update variable in RGWUser::execute_modify() to keys.add() use it directly. Signed-off-by: Danny Al-Gaaf --- src/rgw/rgw_user.cc | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/rgw/rgw_user.cc b/src/rgw/rgw_user.cc index 7ba9ae3c3f04..8166385e28c9 100644 --- a/src/rgw/rgw_user.cc +++ b/src/rgw/rgw_user.cc @@ -1753,7 +1753,6 @@ int RGWUser::remove(RGWUserAdminOpState& op_state, std::string *err_msg) int RGWUser::execute_modify(RGWUserAdminOpState& op_state, std::string *err_msg) { bool populated = op_state.is_populated(); - bool defer_user_update = true; int ret = 0; std::string subprocess_msg; std::string op_email = op_state.get_user_email(); @@ -1850,7 +1849,7 @@ int RGWUser::execute_modify(RGWUserAdminOpState& op_state, std::string *err_msg) // if we're supposed to modify keys, do so if (op_state.has_key_op()) { - ret = keys.add(op_state, &subprocess_msg, defer_user_update); + ret = keys.add(op_state, &subprocess_msg, true); if (ret < 0) { set_err_msg(err_msg, "unable to create or modify keys, " + subprocess_msg); return ret; -- 2.47.3