From 47b84d9660d5f023ab7572a34be7bf571d61ed56 Mon Sep 17 00:00:00 2001 From: Pekka Enberg Date: Tue, 1 Sep 2015 08:59:29 +0300 Subject: [PATCH] locator: Fix minor formatting issues Fix minor formatting issues introduced in commit b8211c436b7223085c1e1b9097b3a4b03cdb7805 ("locator/abstract_replication_strategy: Add validate_options() helper"). Signed-off-by: Pekka Enberg --- locator/network_topology_strategy.cc | 4 ++-- locator/simple_strategy.cc | 3 ++- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/locator/network_topology_strategy.cc b/locator/network_topology_strategy.cc index b2ec91b366..03c55e4308 100644 --- a/locator/network_topology_strategy.cc +++ b/locator/network_topology_strategy.cc @@ -183,11 +183,11 @@ network_topology_strategy::calculate_natural_endpoints( void network_topology_strategy::validate_options() const { for (auto& c : _config_options) { - if (c.first == sstring("replication_factor")) + if (c.first == sstring("replication_factor")) { throw exceptions::configuration_exception( "replication_factor is an option for simple_strategy, not " "network_topology_strategy"); - + } validate_replication_factor(c.second); } } diff --git a/locator/simple_strategy.cc b/locator/simple_strategy.cc index 5e37c8e3c4..e4e2fd3ba2 100644 --- a/locator/simple_strategy.cc +++ b/locator/simple_strategy.cc @@ -54,8 +54,9 @@ size_t simple_strategy::get_replication_factor() const { void simple_strategy::validate_options() const { auto it = _config_options.find("replication_factor"); - if (it == _config_options.end()) + if (it == _config_options.end()) { throw exceptions::configuration_exception("SimpleStrategy requires a replication_factor strategy option."); + } validate_replication_factor(it->second); }