[2835] editorial cleanups: use _ for var names, fixed a typo

src/lib/datasrc/client_list.cc

@@ -114,8 +114,8 @@ ConfigurableClientList::configure(const ConstElementPtr& config,
                                   dconf->contains("cache-enable") &&
                                   dconf->get("cache-enable")->boolValue());
             // Get the name (either explicit, or guess)
-            const ConstElementPtr nameElem(dconf->get("name"));
-            const string name(nameElem ? nameElem->stringValue() : type);
+            const ConstElementPtr name_elem(dconf->get("name"));
+            const string name(name_elem ? name_elem->stringValue() : type);
             if (!used_names.insert(name).second) {
                 isc_throw(ConfigurationError, "Duplicit name in client list: "
                           << name);

src/lib/datasrc/client_list.h

@@ -65,7 +65,7 @@ enum MemorySegmentState {
 /// \brief Status of one data source.
 ///
 /// This indicates the status a data soure is in. It is used with segment
-/// and cache management, to discover the data sources than need external
+/// and cache management, to discover the data sources that need external
 /// mapping or local loading.
 class DataSourceStatus {
 public: