From 1983ed8013637e9b80ca4a0e8d2800d6a36b120e Mon Sep 17 00:00:00 2001 From: Joao Eduardo Luis Date: Tue, 19 Nov 2013 23:21:11 +0000 Subject: [PATCH] mon: OSDMonitor: drop cmdval_get() for unused variable We don't ever use any value as a float, so just drop obtaining it. This makes it easier to partially revert 2fe0d0d9 in an upcoming patch. Backport: emperor Signed-off-by: Joao Eduardo Luis (cherry picked from commit 7191bb2b2485c7819ca7b9d9434d803d0c94db7a) Reviewed-by: Sage Weil Reviewed-by: Greg Farnum --- src/mon/OSDMonitor.cc | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/src/mon/OSDMonitor.cc b/src/mon/OSDMonitor.cc index 31da21d9946df..8d13387ba5474 100644 --- a/src/mon/OSDMonitor.cc +++ b/src/mon/OSDMonitor.cc @@ -2734,7 +2734,7 @@ int OSDMonitor::prepare_command_pool_set(map &cmdmap, if (pending_inc.new_pools.count(pool)) p = pending_inc.new_pools[pool]; - // accept val as a json string or int, and parse out int or float + // accept val as a json string or int, and parse out int // values from the string as needed string raw_val = cmd_vartype_stringify(cmdmap["val"]); string val; @@ -2743,10 +2743,6 @@ int OSDMonitor::prepare_command_pool_set(map &cmdmap, int64_t n = 0; if (!cmd_getval(g_ceph_context, cmdmap, "val", n)) n = strict_strtoll(val.c_str(), 10, &interr); - string floaterr; - float f; - if (!cmd_getval(g_ceph_context, cmdmap, "val", f)) - f = strict_strtod(val.c_str(), &floaterr); if (var == "size") { if (interr.length()) { -- 2.39.5