]> git.ipfire.org Git - thirdparty/kea.git/commitdiff
[#2601] remove empty constructors and destructors
authorAndrei Pavel <andrei@isc.org>
Mon, 31 Oct 2022 13:26:43 +0000 (15:26 +0200)
committerRazvan Becheriu <razvan@isc.org>
Fri, 25 Nov 2022 16:15:33 +0000 (18:15 +0200)
98 files changed:
src/bin/netconf/control_socket.cc
src/bin/netconf/control_socket.h
src/bin/netconf/http_control_socket.cc
src/bin/netconf/http_control_socket.h
src/bin/netconf/location.hh
src/bin/netconf/netconf.cc
src/bin/netconf/netconf.h
src/bin/netconf/netconf_cfg_mgr.cc
src/bin/netconf/netconf_cfg_mgr.h
src/bin/netconf/netconf_config.h
src/bin/netconf/netconf_controller.cc
src/bin/netconf/netconf_controller.h
src/bin/netconf/netconf_log.cc
src/bin/netconf/netconf_log.h
src/bin/netconf/netconf_messages.cc
src/bin/netconf/netconf_messages.h
src/bin/netconf/netconf_parser.cc
src/bin/netconf/netconf_parser.h
src/bin/netconf/netconf_process.cc
src/bin/netconf/netconf_process.h
src/bin/netconf/parser_context.h
src/bin/netconf/parser_context_decl.h
src/bin/netconf/simple_parser.h
src/bin/netconf/stdout_control_socket.cc
src/bin/netconf/stdout_control_socket.h
src/bin/netconf/tests/control_socket_unittests.cc
src/bin/netconf/tests/get_config_unittest.cc
src/bin/netconf/tests/netconf_cfg_mgr_unittests.cc
src/bin/netconf/tests/netconf_controller_unittests.cc
src/bin/netconf/tests/netconf_process_unittests.cc
src/bin/netconf/tests/netconf_unittests.cc
src/bin/netconf/tests/parser_unittests.cc
src/bin/netconf/tests/test_libraries.h.in
src/bin/netconf/unix_control_socket.cc
src/bin/netconf/unix_control_socket.h
src/lib/yang/adaptor.cc
src/lib/yang/adaptor.h
src/lib/yang/adaptor_config.cc
src/lib/yang/adaptor_config.h
src/lib/yang/adaptor_host.cc
src/lib/yang/adaptor_host.h
src/lib/yang/adaptor_option.cc
src/lib/yang/adaptor_option.h
src/lib/yang/adaptor_pool.cc
src/lib/yang/adaptor_pool.h
src/lib/yang/adaptor_subnet.cc
src/lib/yang/adaptor_subnet.h
src/lib/yang/netconf_error.h
src/lib/yang/tests/adaptor_config_unittests.cc
src/lib/yang/tests/adaptor_option_unittests.cc
src/lib/yang/tests/adaptor_pool_unittests.cc
src/lib/yang/tests/adaptor_subnet_unittests.cc
src/lib/yang/tests/adaptor_unittests.cc
src/lib/yang/tests/config_unittests.cc
src/lib/yang/tests/sysrepo_setup.h
src/lib/yang/tests/translator_class_unittests.cc
src/lib/yang/tests/translator_control_socket_unittests.cc
src/lib/yang/tests/translator_database_unittests.cc
src/lib/yang/tests/translator_host_unittests.cc
src/lib/yang/tests/translator_logger_unittests.cc
src/lib/yang/tests/translator_option_data_unittests.cc
src/lib/yang/tests/translator_option_def_unittests.cc
src/lib/yang/tests/translator_pd_pool_unittests.cc
src/lib/yang/tests/translator_pool_unittests.cc
src/lib/yang/tests/translator_shared_network_unittests.cc
src/lib/yang/tests/translator_subnet_unittests.cc
src/lib/yang/tests/translator_unittests.cc
src/lib/yang/tests/yang_configs.h
src/lib/yang/testutils/translator_test.h
src/lib/yang/translator.cc
src/lib/yang/translator.h
src/lib/yang/translator_class.cc
src/lib/yang/translator_class.h
src/lib/yang/translator_config.cc
src/lib/yang/translator_config.h
src/lib/yang/translator_control_socket.cc
src/lib/yang/translator_control_socket.h
src/lib/yang/translator_database.cc
src/lib/yang/translator_database.h
src/lib/yang/translator_host.cc
src/lib/yang/translator_host.h
src/lib/yang/translator_logger.cc
src/lib/yang/translator_logger.h
src/lib/yang/translator_option_data.cc
src/lib/yang/translator_option_data.h
src/lib/yang/translator_option_def.cc
src/lib/yang/translator_option_def.h
src/lib/yang/translator_pd_pool.cc
src/lib/yang/translator_pd_pool.h
src/lib/yang/translator_pool.cc
src/lib/yang/translator_pool.h
src/lib/yang/translator_shared_network.cc
src/lib/yang/translator_shared_network.h
src/lib/yang/translator_subnet.cc
src/lib/yang/translator_subnet.h
src/lib/yang/yang_models.h
src/lib/yang/yang_revisions.h
src/lib/yang/yang_revisions.h.skel

index 56a90ac4b1f8148ccbd1dcb9296037849efdb5f7..8faadfbf52ad959dff15ac73317ef900cc7d432f 100644 (file)
@@ -37,5 +37,5 @@ controlSocketFactory(CfgControlSocketPtr ctrl_sock) {
     }
 }
 
-} // namespace netconf
-} // namespace isc
+}  // namespace netconf
+}  // namespace isc
index a3741a6ca97569974af948750324adf25900c0ad..1fd8555fbea18384eb5d7247ccaf7d8019d5f831 100644 (file)
@@ -20,8 +20,8 @@ namespace netconf {
 class ControlSocketError : public isc::Exception {
 public:
     ControlSocketError(const char* file, size_t line, const char* what) :
-        isc::Exception(file, line, what) { };
-};
+        isc::Exception(file, line, what) { }
+};  // ControlSocketError
 
 /// @brief Base class for control socket communication.
 ///
@@ -31,7 +31,6 @@ public:
 /// communicate with other Kea daemons.
 class ControlSocketBase {
 public:
-
     /// @brief Constructor.
     ///
     /// @param ctrl_sock The control socket configuration.
@@ -44,8 +43,7 @@ public:
     }
 
     /// @brief Destructor (does nothing).
-    virtual ~ControlSocketBase() {
-    }
+    virtual ~ControlSocketBase() = default;
 
     /// @brief Getter which returns the socket type.
     ///
@@ -101,7 +99,7 @@ public:
 
     /// @brief The control socket configuration.
     CfgControlSocketPtr socket_cfg_;
-};
+};  // ControlSocketBase
 
 /// @brief Type definition for the pointer to the @c ControlSocketBase.
 typedef boost::shared_ptr<ControlSocketBase> ControlSocketBasePtr;
@@ -125,7 +123,7 @@ createControlSocket(CfgControlSocketPtr ctrl_sock) {
 ControlSocketBasePtr
 controlSocketFactory(CfgControlSocketPtr ctrl_sock);
 
-} // namespace netconf
-} // namespace isc
+}  // namespace netconf
+}  // namespace isc
 
 #endif // CONTROL_SOCKET_H
index ec1f9ae569016e204a1582379397e292d85dd4d5..192ad08f3c63ecaa3ca8ed84ac1967b85f466bdc 100644 (file)
@@ -125,6 +125,6 @@ HttpControlSocket::sendCommand(ConstElementPtr command) {
     }
 }
 
-} // namespace netconf
-} // namespace isc
+}  // namespace netconf
+}  // namespace isc
 
index 74c57613d34b10cf1010972e5f9c6f162c812e7f..94a495f65e1650269ddff1dc68d9188b9428ae7b 100644 (file)
@@ -22,7 +22,6 @@ namespace netconf {
 /// This class implements config-get, config-test and config-set.
 class HttpControlSocket : public ControlSocketBase {
 public:
-
     /// @brief Constructor.
     ///
     /// @param ctrl_sock The control socket configuration.
@@ -71,7 +70,7 @@ private:
     /// @param command The command to send.
     /// @return The answer.
     data::ConstElementPtr sendCommand(data::ConstElementPtr command);
-};
+};  // HttpControlSocket
 
 /// @brief Type definition for the pointer to the @c HttpControlSocket.
 typedef boost::shared_ptr<HttpControlSocket> HttpControlSocketPtr;
@@ -83,7 +82,7 @@ typedef boost::shared_ptr<HttpControlSocket> HttpControlSocketPtr;
 template <> ControlSocketBasePtr
 createControlSocket<CfgControlSocket::Type::HTTP>(CfgControlSocketPtr ctrl_sock);
 
-} // namespace netconf
-} // namespace isc
+}  // namespace netconf
+}  // namespace isc
 
 #endif // HTTP_CONTROL_SOCKET_H
index 842a1114e856c3d44fc03873ed851109aeb8b34f..40ace584a96c448e849385bc7160709d93a8c5d6 100644 (file)
@@ -300,7 +300,7 @@ namespace isc { namespace netconf {
   }
 
 #line 14 "netconf_parser.yy"
-} } // isc::netconf
+} }  // isc::netconf
 #line 305 "location.hh"
 
 #endif // !YY_NETCONF_LOCATION_HH_INCLUDED
index d8fcc673105200aa5b83f28b198f6677c12c7ad4..798a651f1838462c6d5661721f92f6763cded164 100644 (file)
@@ -136,9 +136,9 @@ public:
             .arg(service_pair_.first)
             .arg(tree);
     }
-};
+};  // NetconfAgentCallback
 
-} // end of anonymous namespace
+}  //anonymous namespace
 
 namespace isc {
 namespace netconf {
@@ -741,5 +741,5 @@ bool NetconfAgent::shouldShutdown() const {
         ->shouldShutdown();
 }
 
-} // namespace netconf
-} // namespace isc
+}  // namespace netconf
+}  // namespace isc
index 7b3cda9eec5e3cc446275d9ac793e1381e818195..e53252d6d1aae85fbfd21e2beffe18f4088c33b2 100644 (file)
@@ -172,9 +172,9 @@ protected:
 
     /// @brief Subscription map.
     std::map<const std::string, sysrepo::Subscription> subscriptions_;
-};
+};  // NetconfAgent
 
-} // namespace netconf
-} // namespace isc
+}  // namespace netconf
+}  // namespace isc
 
 #endif // NETCONF_H
index 82a19f2fe7c56c444d57af84d2c2d9a47fe8633c..0945e755f1c137ab786c605d1d64aec7664e2a0c 100644 (file)
@@ -169,5 +169,5 @@ NetconfCfgMgr::jsonPathsToRedact() const {
     return list;
 }
 
-} // namespace isc::netconf
-} // namespace isc
+}  // namespace netconf
+}  // namespace isc
index ca9e80c52b4d274977bd9b0783cecb34b79f238a..a7c9f5a67acc9a8e877edd5e5cbda219b89e451e 100644 (file)
@@ -115,7 +115,7 @@ private:
 
     /// @brief Configured hooks libraries.
     isc::hooks::HooksConfig hooks_config_;
-};
+};  // NetconfConfig
 
 /// @brief Ctrl Netconf Configuration Manager.
 ///
@@ -123,7 +123,6 @@ private:
 /// configuration.
 class NetconfCfgMgr : public process::DCfgMgrBase {
 public:
-
     /// @brief Constructor.
     NetconfCfgMgr();
 
@@ -175,12 +174,12 @@ protected:
     ///
     /// @return Returns a ConfigPtr to the new context instance.
     virtual process::ConfigPtr createNewContext();
-};
+};  // NetconfCfgMgr
 
 /// @brief Defines a shared pointer to NetconfCfgMgr.
 typedef boost::shared_ptr<NetconfCfgMgr> NetconfCfgMgrPtr;
 
-} // namespace isc::netconf
-} // namespace isc
+}  // namespace netconf
+}  // namespace isc
 
 #endif // NETCONF_CFG_MGR_H
index 18e7a9d8789e1fa9ab2d9219225647a456eddafd..ab8bab4cb5f7aead6184f401195872cf9e39261e 100644 (file)
@@ -75,7 +75,7 @@ public:
         UNIX,    //< Unix socket.
         HTTP,    //< HTTP socket.
         STDOUT   //< standard output.
-    };
+    };  // Type
 
     /// @brief Constructor.
     ///
@@ -138,7 +138,7 @@ private:
 
     /// @brief The HTTP server URL.
     const isc::http::Url url_;
-};
+};  // CfgControlSocket
 
 /// @brief Defines a pointer for CfgControlSocket instances.
 typedef boost::shared_ptr<CfgControlSocket> CfgControlSocketPtr;
@@ -267,7 +267,7 @@ private:
 
     /// @brief The control socket.
     CfgControlSocketPtr control_socket_;
-};
+};  // CfgServer
 
 /// @brief Defines a pointer for CfgServer instances.
 typedef boost::shared_ptr<CfgServer> CfgServerPtr;
@@ -303,7 +303,7 @@ public:
     ///
     /// @return pointer to the new CfgControlSocket instance.
     CfgControlSocketPtr parse(data::ConstElementPtr ctrl_sock_config);
-};
+};  // ControlSocketConfigParser
 
 /// @brief Parser for CfgServer.
 ///
@@ -320,7 +320,7 @@ public:
     /// @param server_config is the value from the "managed-servers" map to parse.
     /// @return pointer to the new CfgServer instance.
     CfgServerPtr parse(data::ConstElementPtr server_config);
-};
+};  // ServerConfigParser
 
 }  // namespace netconf
 }  // namespace isc
index 7d853291448305e39789e7e19502f7124287f5aa..2d8c438d1b05263ccd62c94528f3b4a8177220e7 100644 (file)
@@ -71,5 +71,5 @@ NetconfController::getNetconfProcess() {
 // Refer to config_report so it will be embedded in the binary.
 const char* const* netconf_config_report = isc::detail::config_report;
 
-} // namespace isc::netconf
-} // namespace isc
+}  // namespace netconf
+}  // namespace isc
index b1d8cf674e244dcc89266c02f694066b742f9982..34ebb3424a4d8b553a90042e3218ae29634dc648 100644 (file)
@@ -74,12 +74,12 @@ private:
     /// @brief Constructor is declared private to maintain the integrity of
     /// the singleton instance.
     NetconfController();
-};
+};  // NetconfController
 
 // @Defines a shared pointer to NetconfController
 typedef boost::shared_ptr<NetconfController> NetconfControllerPtr;
 
-} // namespace isc::netconf
-} // namespace isc
+}  // namespace netconf
+}  // namespace isc
 
 #endif // NETCONF_CONTROLLER_H
index d44463e430e142f5a9793b6cd6bdf3e2e86dc316..96b192c262766955af8481040baa6c2d955cac05 100644 (file)
@@ -23,6 +23,6 @@ const char* NETCONF_LOGGER_NAME = "netconf";
 
 isc::log::Logger netconf_logger(NETCONF_LOGGER_NAME);
 
-} // namespace netconf
-} // namespace isc
+}  // namespace netconf
+}  // namespace isc
 
index 72a51d04174082f85544dd03b1e3ad0711f07536..ded7b40dc237583dc828a473c51338e221875525 100644 (file)
@@ -48,7 +48,7 @@ extern const char* NETCONF_LOGGER_NAME;
 /// @brief Base logger for the netconf agent
 extern isc::log::Logger netconf_logger;
 
-} // namespace netconf
-} // namespace isc
+}  // namespace netconf
+}  // namespace isc
 
 #endif // NETCONF_LOG_H
index 883e2ee554f689c0c219b8aba70456a175984f9e..6d0d77d02b2082ac4302999ad0a3e718ff86b6b6 100644 (file)
@@ -42,8 +42,8 @@ extern const isc::log::MessageID NETCONF_VALIDATE_CONFIG_FAILED = "NETCONF_VALID
 extern const isc::log::MessageID NETCONF_VALIDATE_CONFIG_REJECTED = "NETCONF_VALIDATE_CONFIG_REJECTED";
 extern const isc::log::MessageID NETCONF_VALIDATE_CONFIG_STARTED = "NETCONF_VALIDATE_CONFIG_STARTED";
 
-} // namespace netconf
-} // namespace isc
+}  // namespace netconf
+}  // namespace isc
 
 namespace {
 
@@ -87,5 +87,5 @@ const char* values[] = {
 
 const isc::log::MessageInitializer initializer(values);
 
-} // Anonymous namespace
+}  // Anonymous namespace
 
index df290bb075207e1e6270b1d45b8777980357c1e0..720de61e07a1ef4c57aaceb2e4cf34b4a3dd18b4 100644 (file)
@@ -43,7 +43,7 @@ extern const isc::log::MessageID NETCONF_VALIDATE_CONFIG_FAILED;
 extern const isc::log::MessageID NETCONF_VALIDATE_CONFIG_REJECTED;
 extern const isc::log::MessageID NETCONF_VALIDATE_CONFIG_STARTED;
 
-} // namespace netconf
-} // namespace isc
+}  // namespace netconf
+}  // namespace isc
 
 #endif // NETCONF_MESSAGES_H
index 8fe964dba129f09d038c2e7973048fc374f878c0..83792aa70076c2b457f259bf3e2ab02c9955bbea 100644 (file)
@@ -2298,7 +2298,7 @@ namespace isc { namespace netconf {
 
 
 #line 14 "netconf_parser.yy"
-} } // isc::netconf
+} }  // isc::netconf
 #line 2303 "netconf_parser.cc"
 
 #line 748 "netconf_parser.yy"
index 3b8e2586661714b43b7bfef7b9e325c24ddf7ae0..6f91bd61911184d82d8a697d5be83280ecf80dc6 100644 (file)
@@ -2378,7 +2378,7 @@ switch (yykind)
 
 
 #line 14 "netconf_parser.yy"
-} } // isc::netconf
+} }  // isc::netconf
 #line 2383 "netconf_parser.h"
 
 
index f8430dd6a7fd080c5be7df6370b8658abba368b7..55fe070d3c6a26cbe6a70baa993ae1e133b13f82 100644 (file)
@@ -95,5 +95,5 @@ NetconfProcess::getNetconfCfgMgr() {
     return (boost::dynamic_pointer_cast<NetconfCfgMgr>(getCfgMgr()));
 }
 
-} // namespace isc::netconf
-} // namespace isc
+}  // namespace netconf
+}  // namespace isc
index e8b24b1ccfa75b75f2163d872527c51b5f0e9710..27409cc6ab74851463dce6c6d91a2b049aa9c010 100644 (file)
@@ -95,12 +95,12 @@ private:
 
     /// @brief Netconf agent.
     NetconfAgent agent_;
-};
+};  // NetconfProcess
 
 /// @brief Defines a shared pointer to NetconfProcess.
 typedef boost::shared_ptr<NetconfProcess> NetconfProcessPtr;
 
-}; // namespace isc::netconf
-}; // namespace isc
+}  // namespace netconf
+}  // namespace isc
 
 #endif // NETCONF_PROCESS_H
index 2498f830061992b481f4d3b2e3ada610bf87a11d..7438adb354f040e61f8578db9af7cc25cacea769 100644 (file)
@@ -305,9 +305,9 @@ public:
     ///
     /// @return Element structure representing parsed text.
     isc::data::ElementPtr parseCommon();
-};
+};  // ParserContext
 
-} // end of isc::netconf namespace
-} // end of isc namespace
+}  // namespace netconf
+}  // namespace isc
 
 #endif
index 07c8c7a3a0c207ca549a9682ac0dd7d32fc9ce28..6697a362d0aafbf0358a66bfb95ff7004c87c811 100644 (file)
@@ -14,7 +14,7 @@ namespace netconf {
 
 class ParserContext;
 
-}; // end of isc::dhcp namespace
-}; // end of isc namespace
+}  // namespace dhcp
+}  // namespace isc
 
 #endif
index 5c9324dae904356e663d6cc195cac784b947c3ec..dc9a00db7d32d6876bd44361cec3c5720e557bb7 100644 (file)
@@ -67,8 +67,8 @@ public:
     static const isc::data::SimpleDefaults D2_DEFAULTS;
     static const isc::data::SimpleDefaults CA_DEFAULTS;
     static const isc::data::ParamsList INHERIT_TO_SERVERS;
-};
+};  // NetconfSimpleParser
 
-};
-};
+}  // namespace netconf
+}  // namespace isc
 #endif
index 8dc925ba6d331b8223fdd5f317d9788268432de0..6422993c953ed6f2c4503305d7366b63641efd90 100644 (file)
@@ -52,5 +52,5 @@ StdoutControlSocket::configSet(ConstElementPtr config,
     return (createAnswer());
 }
 
-} // namespace netconf
-} // namespace isc
+}  // namespace netconf
+}  // namespace isc
index 4bbd82f63e707f530b0635fb5efd0ead594ea933..20fa2b58db13c6bff1912c211c1d03d921e188e5 100644 (file)
@@ -23,7 +23,6 @@ namespace netconf {
 /// This class implements config-test (always OK) and config-set.
 class StdoutControlSocket : public ControlSocketBase {
 public:
-
     /// @brief Constructor.
     ///
     /// Use std::cout.
@@ -72,7 +71,7 @@ protected:
 
     /// @brief The output stream (std::cout outside tests).
     std::ostream& output_;
-};
+};  // StdoutControlSocket
 
 /// @brief Type definition for the pointer to the @c StdoutControlSocket.
 typedef boost::shared_ptr<StdoutControlSocket> StdoutControlSocketPtr;
@@ -84,7 +83,7 @@ typedef boost::shared_ptr<StdoutControlSocket> StdoutControlSocketPtr;
 template <> ControlSocketBasePtr
 createControlSocket<CfgControlSocket::Type::STDOUT>(CfgControlSocketPtr ctrl_sock);
 
-} // namespace netconf
-} // namespace isc
+}  // namespace netconf
+}  // namespace isc
 
 #endif // STDOUT_CONTROL_SOCKET_H
index 1c79a061edf1dc2726c3f955533c02bb6885e11d..84b224b784fa6b85708e4fe53f145a5c8cae5093 100644 (file)
@@ -58,11 +58,7 @@ public:
     TestStdoutControlSocket(CfgControlSocketPtr ctrl_sock, ostream& output)
         : StdoutControlSocket(ctrl_sock, output) {
     }
-
-    /// @brief Destructor.
-    virtual ~TestStdoutControlSocket() {
-    }
-};
+};  // TestStdoutControlSocket
 
 /// @brief Type definition for the pointer to the @c TestStdoutControlSocket.
 typedef boost::shared_ptr<TestStdoutControlSocket> TestStdoutControlSocketPtr;
@@ -205,7 +201,7 @@ public:
 
     /// @brief IOService object.
     IOService io_service_;
-};
+};  // UnixControlSocketTest
 
 /// @brief Server method running in a thread reflecting the command.
 ///
@@ -500,7 +496,7 @@ protected:
         response->finalize();
         return (response);
     }
-};
+};  // TestHttpResponseCreator
 
 /// @brief Implementation of the test HttpResponseCreatorFactory.
 class TestHttpResponseCreatorFactory : public HttpResponseCreatorFactory {
@@ -511,7 +507,7 @@ public:
         HttpResponseCreatorPtr response_creator(new TestHttpResponseCreator());
         return (response_creator);
     }
-};
+};  // TestHttpResponseCreatorFactory
 
 /// @brief Test fixture class for http control sockets.
 class HttpControlSocketTest : public ThreadedTest {
@@ -609,7 +605,7 @@ public:
 
     /// @brief Pointer to listener.
     HttpListenerPtr listener_;
-};
+};  // HttpControlSocketTest
 
 /// @brief Create the reflecting listener.
 void
index fd5c2e6a9e23b5743ee9dcfa651a2fb24353c7ee..03dde5d17d6f4493851e142650e20b1e96f5ff75 100644 (file)
@@ -116,7 +116,7 @@ pathReplacer(ConstElementPtr netconf_cfg) {
 class NakedNetconfCfgMgr : public NetconfCfgMgr {
 public:
     using NetconfCfgMgr::parse;
-};
+};  // NakedNetconfCfgMgr
 
 }  // namespace
 
@@ -227,7 +227,7 @@ public:
     boost::scoped_ptr<NakedNetconfCfgMgr> srv_; ///< Netconf server under test
     int rcode_;                       ///< Return code from element parsing
     ConstElementPtr comment_;         ///< Reason for parse fail
-};
+};  // NetconfGetCfgTest
 
 // Test a simple configuration.
 TEST_F(NetconfGetCfgTest, simple) {
index 23eea6d49789070d9f36b6eb7953268d5deb52e6..7ee08a4784120d5f129ab2b0e133f959bfe436cc 100644 (file)
@@ -37,7 +37,7 @@ namespace  {
 class NakedNetconfCfgMgr : public NetconfCfgMgr {
 public:
     using NetconfCfgMgr::parse;
-};
+};  // NakedNetconfCfgMgr
 
 // Tests construction of NetconfCfgMgr class.
 TEST(NetconfCfgMgr, construction) {
@@ -357,7 +357,7 @@ const char* NETCONF_CONFIGS[] = {
     "        }\n"
     "    }\n"
     "}"
-};
+};  // NETCONF_CONFIGS
 
 // Tests the handling of bad socket type. Can't use the fixture class
 // because the Netconf parser does not allow bad socket types.
@@ -405,7 +405,7 @@ public:
 
     /// Configuration Manager (used in tests)
     NakedNetconfCfgMgr cfg_mgr_;
-};
+};  // NetconfParserTest
 
 // This test verifies if an empty config is handled properly. In practice such
 // a config makes little sense, but perhaps it's ok for a default deployment.
index 7fc2a4a3fc0cb8c1d21656bb44e5b43bb68e2c37..2b2f8bb633cc725626dbaf1fe4d96b6c43db44f3 100644 (file)
@@ -55,7 +55,6 @@ const char* valid_netconf_config =
 /// has extensive set of unit tests that are independent from Netconf.
 class NetconfControllerTest : public DControllerTest {
 public:
-
     /// @brief Constructor.
     NetconfControllerTest()
         : DControllerTest(NetconfController::instance) {
@@ -83,7 +82,7 @@ public:
         }
         return (p);
     }
-};
+};  // NetconfControllerTest
 
 // Basic Controller instantiation testing.
 // Verifies that the controller singleton gets created and that the
index 4cf9105fce1935c6ab08836dfbff1d7a87d21b6b..7e30f3e0575509ad12af2d87995f973b6b0e0efd 100644 (file)
@@ -37,15 +37,11 @@ public:
         NetconfConfigPtr ctx = getNetconfCfgMgr()->getNetconfConfig();
     }
 
-    /// @brief Destructor
-    virtual ~NetconfProcessTest() {
-    }
-
     /// @brief Callback that will invoke shutdown method.
     void genShutdownCallback() {
         shutdown(isc::data::ConstElementPtr());
     }
-};
+};  // NetconfProcessTest
 
 // Test construction of the NetconfProcess object.
 TEST(NetconfProcess, construction) {
index 8082b262318c259456e0d00f606b2260a0ff3822..4b87dbddad026be95fbdb2e5b4347dd0d3b92ffc 100644 (file)
@@ -58,14 +58,6 @@ typedef boost::shared_ptr<thread> ThreadPtr;
 /// @brief Test version of the NetconfAgent class.
 class NakedNetconfAgent : public NetconfAgent {
 public:
-    /// @brief Constructor.
-    NakedNetconfAgent() {
-    }
-
-    /// @brief Destructor.
-    virtual ~NakedNetconfAgent() {
-    }
-
     /// Export protected methods and fields.
     using NetconfAgent::keaConfig;
     using NetconfAgent::initSysrepo;
@@ -77,7 +69,7 @@ public:
     using NetconfAgent::running_sess_;
     using NetconfAgent::modules_;
     using NetconfAgent::subscriptions_;
-};
+};  // NakedNetconfAgent
 
 /// @brief Type definition for the pointer to NakedNetconfAgent objects.
 typedef boost::shared_ptr<NakedNetconfAgent> NakedNetconfAgentPtr;
@@ -183,7 +175,7 @@ public:
 
     /// @brief Response list.
     vector<string> responses_;
-};
+};  // NetconfAgentTest
 
 /// @brief Special test fixture for logging tests.
 class NetconfAgentLogTest : public dhcp::test::LogContentTest {
@@ -252,7 +244,7 @@ public:
 
     /// @brief Test netconf agent.
     NakedNetconfAgentPtr agent_;
-};
+};  // NetconfAgentLogTest
 
 /// @brief Fake server (returns OK answer).
 void
index 27d9f93deeb5a19216a379375c12c383c6db1b87..186c149440becc80e0c0474b305e03fd86901db4 100644 (file)
@@ -923,7 +923,7 @@ public:
         log += "A piece of configuration may have been omitted.";
         addString(log);
     }
-};
+};  // TrailingCommasTest
 
 // Test that trailing commas are allowed.
 TEST_F(TrailingCommasTest, tests) {
index f2a5a6597bfd0839bd447f35409ff1170228ce46..f90d682cf2cf47438e64b903b5cef43b7cd0d49b 100644 (file)
@@ -19,6 +19,6 @@ namespace {
 // Basic library with context_create and three "standard" callouts.
 static const char* BASIC_CALLOUT_LIBRARY = "@abs_builddir@/.libs/libbasic.so";
 
-} // anonymous namespace
+}  // anonymous namespace
 
 #endif // TEST_LIBRARIES_H
index 51ef5d1be67375ec23aa840cc8d5aba204f76f51..62dbd80d3979d14a15956dcbf11993952f325c0a 100644 (file)
@@ -93,5 +93,5 @@ UnixControlSocket::sendCommand(ConstElementPtr command) {
     }
 }
 
-} // namespace netconf
-} // namespace isc
+}  // namespace netconf
+}  // namespace isc
index 538ae170c28ff1839e9eb1f741b138ce98aa341c..fb6b725b0867f2b93ea2ec7f94348b1183486fdd 100644 (file)
@@ -22,7 +22,6 @@ namespace netconf {
 /// This class implements config-get, config-test and config-set.
 class UnixControlSocket : public ControlSocketBase {
 public:
-
     /// @brief Constructor.
     ///
     /// @param ctrl_sock The control socket configuration.
@@ -71,7 +70,7 @@ private:
     /// @param command The command to send.
     /// @return The answer.
     data::ConstElementPtr sendCommand(data::ConstElementPtr command);
-};
+};  // UnixControlSocket
 
 /// @brief Type definition for the pointer to the @c UnixControlSocket.
 typedef boost::shared_ptr<UnixControlSocket> UnixControlSocketPtr;
@@ -83,7 +82,7 @@ typedef boost::shared_ptr<UnixControlSocket> UnixControlSocketPtr;
 template <> ControlSocketBasePtr
 createControlSocket<CfgControlSocket::Type::UNIX>(CfgControlSocketPtr ctrl_sock);
 
-} // namespace netconf
-} // namespace isc
+}  // namespace netconf
+}  // namespace isc
 
 #endif // UNIX_CONTROL_SOCKET_H
index 2de0a2942fc4cfa00f64da32f836cf6bbcb3b813..3af89a7d7ed693f98a802eb6dba49826a9c937e3 100644 (file)
@@ -17,12 +17,6 @@ using namespace isc::data;
 namespace isc {
 namespace yang {
 
-Adaptor::Adaptor() {
-}
-
-Adaptor::~Adaptor() {
-}
-
 ConstElementPtr
 Adaptor::getContext(ConstElementPtr parent)
 {
@@ -295,7 +289,7 @@ void applyDown(ConstElementPtr path, ConstElementPtr actions, ElementPtr scope,
     }
 }
 
-} // end of anonymous namespace
+}  //anonymous namespace
 
 /// Apply recursively starting at the beginning of the path.
 void
@@ -304,5 +298,5 @@ Adaptor::modify(ConstElementPtr path, ConstElementPtr actions,
     applyDown(path, actions, config, 0);
 }
 
-}  // end of namespace isc::yang
-}  // end of namespace isc
+}  // namespace yang
+}  // namespace isc
index c0c04aac2d09801e8ca10c82ffc5e696bf4553e6..35efeca08a158a6a766a0a9718bc75d635b54b93 100644 (file)
@@ -19,7 +19,7 @@ public:
     MissingKey(const char* file, size_t line, const char* what) :
         isc::Exception(file, line, what)
     {}
-};
+};  // MissingKey
 
 /// @brief JSON adaptor between canonical Kea and YANG models.
 ///
@@ -29,12 +29,8 @@ public:
 /// The basic adaptor provides a set of tools.
 class Adaptor {
 public:
-
-    /// @brief Constructor.
-    Adaptor();
-
     /// @brief Destructor.
-    virtual ~Adaptor();
+    virtual ~Adaptor() = default;
 
     /// @brief Get user context.
     ///
@@ -125,9 +121,9 @@ public:
                        isc::data::ConstElementPtr actions,
                        isc::data::ElementPtr config);
 
-};
+};  // Adaptor
 
-}  // end of namespace isc::yang
-}  // end of namespace isc
+}  // namespace yang
+}  // namespace isc
 
 #endif // ISC_ADAPTOR_H
index ca388306528f20baf4219ff934341efe7b25a04e..bf1d0a90eb280dacba5717ff2b4c774a6553e52f 100644 (file)
@@ -20,12 +20,6 @@ const string DHCP6_SPACE = "dhcp6";
 namespace isc {
 namespace yang {
 
-AdaptorConfig::AdaptorConfig() {
-}
-
-AdaptorConfig::~AdaptorConfig() {
-}
-
 bool
 AdaptorConfig::subnetsCollectID(ConstElementPtr subnets, SubnetIDSet& set) {
     bool have_ids = true;
@@ -654,5 +648,5 @@ AdaptorConfig::preProcess6(ConstElementPtr config) {
     preProcess(mutable_dhcp, "subnet6", DHCP6_SPACE);
 }
 
-}  // end of namespace isc::yang
-}  // end of namespace isc
+}  // namespace yang
+}  // namespace isc
index 88cfd4c3f4d831856a48d57ce181e2db013fca9d..a9422aaacbbce61c6a3bbc067de4456032da5248 100644 (file)
@@ -25,13 +25,6 @@ namespace yang {
 class AdaptorConfig : public AdaptorHost, public AdaptorOption,
     public AdaptorSubnet {
 public:
-
-    /// @brief Constructor.
-    AdaptorConfig();
-
-    /// @brief Destructor.
-    virtual ~AdaptorConfig();
-
     /// @brief Pre process a DHCPv4 configuration.
     ///
     /// Assign subnet IDs, check and set defaults in options, etc.
@@ -284,9 +277,9 @@ protected:
     static void preProcess(isc::data::ElementPtr dhcp,
                            const std::string& subsel,
                            const std::string& space);
-};
+};  // AdaptorConfig
 
-}  // end of namespace isc::yang
-}  // end of namespace isc
+}  //namespace yang
+}  // namespace isc
 
 #endif // ISC_ADAPTOR_CONFIG_H
index 9df7d5b7a0a670154acaaa9da2184b652cb4646f..3a85c419afdf1e46475b81123b5b63ed77c60ca4 100644 (file)
@@ -23,12 +23,6 @@ const string
 AdaptorHost::STD_CHARACTERS =
     "0123456789@ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz-.@_";
 
-AdaptorHost::AdaptorHost() {
-}
-
-AdaptorHost::~AdaptorHost() {
-}
-
 void
 AdaptorHost::quoteIdentifier(ElementPtr host) {
     ConstElementPtr flex_id = host->get("flex-id");
@@ -64,5 +58,5 @@ AdaptorHost::quoteIdentifier(ElementPtr host) {
     host->set("flex-id", Element::create(tmp.str()));
 }
 
-}  // end of namespace isc::yang
-}  // end of namespace isc
+}  // namespace yang
+}  // namespace isc
index 62f80c113b0d0350efb27a1c58e6f837d78bda52..bc9aff4952712977ec2cf584111b04481207d178 100644 (file)
@@ -25,11 +25,8 @@ public:
     /// characters (digit, letters, -, ., @, _).
     static const std::string STD_CHARACTERS;
 
-    /// @brief Constructor.
-    AdaptorHost();
-
     /// @brief Destructor.
-    virtual ~AdaptorHost();
+    virtual ~AdaptorHost() = default;
 
     /// @brief Quote when needed a host identifier.
     ///
@@ -39,9 +36,9 @@ public:
     ///
     /// @param host The host.
     static void quoteIdentifier(isc::data::ElementPtr host);
-};
+};  // AdaptorHost
 
-}  // end of namespace isc::yang
-}  // end of namespace isc
+}  //namespace yang
+}  // namespace isc
 
 #endif // ISC_ADAPTOR_HOST_H
index c40554a0312a9fe009118796518dc9f4d3b0a2af..02acde1c30fb8c7c2b9e1dba9c914c8923cf28e6 100644 (file)
@@ -17,12 +17,6 @@ using namespace isc::dhcp;
 namespace isc {
 namespace yang {
 
-AdaptorOption::AdaptorOption() {
-}
-
-AdaptorOption::~AdaptorOption() {
-}
-
 void
 AdaptorOption::setSpace(ElementPtr option, const string& space) {
     if (!option->contains("space")) {
@@ -124,5 +118,5 @@ AdaptorOption::initCodesInternal(OptionCodes& codes, const string& space,
     }
 }
 
-}  // end of namespace isc::yang
-}  // end of namespace isc
+}  // namespace yang
+}  // namespace isc
index a3842c7a3262966dd46a3823620b3ce0a4a55023..8e5aa551ac47bb5188adc903b649097d1a300ca5 100644 (file)
@@ -7,17 +7,13 @@
 #ifndef ISC_ADAPTOR_OPTION_H
 #define ISC_ADAPTOR_OPTION_H 1
 
+#include <dhcp/option_data_types.h>
 #include <yang/adaptor.h>
-#include <map>
+
 #include <list>
+#include <map>
 
 namespace isc {
-
-namespace dhcp {
-/// @brief Forward declaration of option definition parameters.
-struct OptionDefParams;
-};
-
 namespace yang {
 
 /// @brief Map for DHCP option definitions handling code and
@@ -44,12 +40,8 @@ typedef std::map<std::string, uint16_t> OptionCodes;
 /// and raises an error when it can't.
 class AdaptorOption {
 public:
-
-    /// @brief Constructor.
-    AdaptorOption();
-
     /// @brief Destructor.
-    virtual ~AdaptorOption();
+    virtual ~AdaptorOption() = default;
 
     /// @brief Set space.
     ///
@@ -103,9 +95,9 @@ protected:
     static void initCodesInternal(OptionCodes& codes, const std::string& space,
                                   const isc::dhcp::OptionDefParams* params,
                                   size_t params_size);
-};
+};  // AdaptorOption
 
-}  // end of namespace isc::yang
-}  // end of namespace isc
+}  //namespace yang
+}  // namespace isc
 
 #endif // ISC_ADAPTOR_OPTION_H
index 4a84bc012aea6e444dd41908704575386ad76c90..2edfcfde1090770c458cd30b67817d0a5e48bf15 100644 (file)
@@ -15,12 +15,6 @@ using namespace isc::data;
 namespace isc {
 namespace yang {
 
-AdaptorPool::AdaptorPool() {
-}
-
-AdaptorPool::~AdaptorPool() {
-}
-
 void
 AdaptorPool::canonizePool(ElementPtr pool) {
     const string& orig = pool->get("pool")->stringValue();
@@ -83,5 +77,5 @@ AdaptorPool::toSubnetIetf6(ElementPtr subnet, ConstElementPtr pools) {
     Adaptor::toParent("rebind-timer", subnet, pools);
 }
 
-}  // end of namespace isc::yang
-}  // end of namespace isc
+}  //namespace yang
+}  // namespace isc
index 2c9268180dc79b02f397a9d2c9068cbb34d7c2e0..84af9dba2d3717111eb1ce695cd47f4adb001727 100644 (file)
@@ -27,12 +27,8 @@ namespace yang {
 /// removes timers from pools as they are not expected by Kea at this scope.
 class AdaptorPool {
 public:
-
-    /// @brief Constructor.
-    AdaptorPool();
-
     /// @brief Destructor.
-    virtual ~AdaptorPool();
+    virtual ~AdaptorPool() = default;
 
     /// @brief Canonize pool.
     ///
@@ -90,9 +86,9 @@ protected:
     /// @param pools The children pools.
     static void toSubnetIetf6(isc::data::ElementPtr subnet,
                               isc::data::ConstElementPtr pools);
-};
+};  // AdaptorPool
 
-}  // end of namespace isc::yang
-}  // end of namespace isc
+}  // namespace yang
+}  // namespace isc
 
 #endif // ISC_ADAPTOR_POOL_H
index 5e4d6f395d38a01ed88e7cf23d690a46835b4d88..c22af2d05067979d29a406c37dee2e5f2b2bbb1f 100644 (file)
@@ -15,12 +15,6 @@ using namespace isc::dhcp;
 namespace isc {
 namespace yang {
 
-AdaptorSubnet::AdaptorSubnet() {
-}
-
-AdaptorSubnet::~AdaptorSubnet() {
-}
-
 bool
 AdaptorSubnet::collectID(ConstElementPtr subnet, SubnetIDSet& set) {
     ConstElementPtr id = subnet->get("id");
@@ -69,5 +63,5 @@ AdaptorSubnet::updateRelay(ElementPtr subnet) {
     }
 }
 
-}  // end of namespace isc::yang
-}  // end of namespace isc
+}  // namespace yang
+}  // namespace isc
index acdf541291350f8d6e3cda64167b919a5727a548..3268d77a60707c8c96aae322397790ce1a7a43f1 100644 (file)
@@ -29,12 +29,8 @@ typedef std::set<isc::dhcp::SubnetID> SubnetIDSet;
 /// code should work without it.
 class AdaptorSubnet {
 public:
-
-    /// @brief Constructor.
-    AdaptorSubnet();
-
     /// @brief Destructor.
-    virtual ~AdaptorSubnet();
+    virtual ~AdaptorSubnet() = default;
 
     /// @brief Collect a subnet ID.
     ///
@@ -58,9 +54,9 @@ public:
     ///
     /// @param subnet The subnet.
     static void updateRelay(isc::data::ElementPtr subnet);
-};
+};  // AdaptorSubnet
 
-}  // end of namespace isc::yang
-}  // end of namespace isc
+}  // namespace yang
+}  // namespace isc
 
 #endif // ISC_ADAPTOR_SUBNET_H
index 46ca3c735b2ff035e1f03990a9e8294c96064e1f..d575d0c8f3fba916af6cb5d294a45832aeaa1a81 100644 (file)
@@ -18,9 +18,9 @@ public:
     NetconfError(const char* file, size_t line, const char* what) :
         isc::Exception(file, line, what)
     {}
-};
+};  // NetconfError
 
-}  // end of namespace isc::yang
-}  // end of namespace isc
+}  // namespace yang
+}  // namespace isc
 
 #endif // ISC_SYSREPO_ERROR_H
index 0ea628be4734cc72e610395b177ebaa6aecceb00..76d05820704244559c982da59338c12cfc7a3906 100644 (file)
@@ -57,7 +57,7 @@ void testFile(const std::string& fname, bool v6, ElementPtr& result) {
     result = json;
 }
 
-};
+};  // AdaptorConfigTest
 
 TEST_F(AdaptorConfigTest, loadExamples4) {
     vector<string> configs = {
@@ -144,4 +144,4 @@ TEST_F(AdaptorConfigTest, loadExamples6) {
 /// @todo: Check option data using kea6/with-ddns.json
 /// @todo: Check option defs using kea6/dhcpv4-over-dhcpv6.json
 
-}; // end of anonymous namespace
+}  // anonymous namespace
index 3c151f56ff476ec5d142a2caa823c29e05c4196f..d9bfd563aaf296cb7cff0e571cdcbf2e19f4caaa 100644 (file)
@@ -184,7 +184,7 @@ TEST(AdaptorOptionTest, setCodeNotInMap) {
 class TestAdaptorOption : public AdaptorOption {
 public:
     using AdaptorOption::initCodesInternal;
-};
+};  // TestAdaptorOption
 
 // Verifies that initCodesInternal works as expected.
 TEST(AdaptorOptionTest, initCodesInternal) {
@@ -241,4 +241,4 @@ TEST(AdaptorOptionTest, initCodes6) {
     EXPECT_EQ(ISC_V6_4O6_INTERFACE, codes["vendor-2495@4o6-interface"]);
 }
 
-} // end of anonymous namespace
+}  //anonymous namespace
index 05d77259c22afd9b8ca74fd1db0298deb3d0950c..367502af6fd7bd0a28bb3b7a72164fe4f1b038e4 100644 (file)
@@ -273,4 +273,4 @@ TEST(AdaptorPoolTest, toSubnetBad) {
                      "\"rebind-timer\": 20 } ]");
 }
 
-}; // end of anonymous namespace
+}  // anonymous namespace
index 54ec71094d758f54c4b7dbf3d9413d9d8fbcf738..00ac3ca39e23c2f4399f64899fb65142250dd5be 100644 (file)
@@ -210,4 +210,4 @@ TEST(AdaptorSubnetTest, updateRelayAddress) {
 
 // It does not make sense to have both ip-address and ip-addresses...
 
-}; // end of anonymous namespace
+}  // anonymous namespace
index b819e25d65edc14db0f9feb06ff10e9547c80bd8..431e8b0ad2b4e22b485c4724c03e05e202033462 100644 (file)
@@ -395,4 +395,4 @@ TEST(AdaptorTest, modifyListAllDelete) {
     EXPECT_TRUE(expected->equals(*json));
 }
 
-}; // end of anonymous namespace
+}  // anonymous namespace
index 5d1c7978a5ed65d85c63502f357cebc70079215d..fdb286482ca63119ed797adcd60d385467ce246d 100644 (file)
@@ -181,23 +181,23 @@ public:
     Session session_;
 
     std::unique_ptr<TranslatorBasic> translator_;
-};
+};  // ConfigTest
 
 struct ConfigTestKeaV4 : ConfigTest {
     ConfigTestKeaV4() {
         model_ = KEA_DHCP4_SERVER;
     }
-};
+};  // ConfigTestKeav4
 struct ConfigTestKeaV6 : ConfigTest {
     ConfigTestKeaV6() {
         model_ = KEA_DHCP6_SERVER;
     }
-};
+};  // ConfigTestKeaV6
 struct ConfigTestIetfV6 : ConfigTest {
     ConfigTestIetfV6() {
         model_ = IETF_DHCPV6_SERVER;
     }
-};
+};  // ConfigTestIetfV6
 
 // Check empty config with ietf-dhcpv6-server model.
 TEST_F(ConfigTestIetfV6, emptyIetf6) {
index 05954dc9fe4863f3fa66ba7d92a5bc04ad34daed..bbc891c953fd29a4902353fd64db4880469ae086 100644 (file)
@@ -48,7 +48,7 @@ private:
                       << std::endl;
         }
     }
-};
+};  // SysrepoSetup
 
 /// @brief Test Fixture template for translator tests.
 ///
@@ -87,10 +87,10 @@ public:
     /// @brief Shared pointer to the transaction object.
     boost::shared_ptr<translator_t> t_obj_;
     std::string model_;
-};
+};  // GenericTranslatorTest
 
-} // namespace test
-} // namespace yang
-} // namespace isc
+}  // namespace test
+}  // namespace yang
+}  // namespace isc
 
 #endif // SYSREPO_SETUP_H
index 865faf6c5e2583a291e27c2e6f51aff0f83bc5bf..e31f6b9df782236083686eb00d9de82d797be9b4 100644 (file)
@@ -29,21 +29,20 @@ extern char const client_classes[] = "client classes";
 class TranslatorClassesTestv4 :
     public GenericTranslatorTest<client_classes, TranslatorClasses> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorClassesTestv4() {
         model_ = KEA_DHCP4_SERVER;
     }
-};
+};  // TranslatorClassesTestv4
+
 class TranslatorClassesTestv6 :
     public GenericTranslatorTest<client_classes, TranslatorClasses> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorClassesTestv6() {
         model_ = KEA_DHCP6_SERVER;
     }
-};
+};  // TranslatorClassesTestv6
 
 // This test verifies that an empty client class list can be properly
 // translated from YANG to JSON.
index 321f074ca42feb72ea1abb60b6e02f742ca19fb7..485d59c4b764c5c8d63c6f9489cf31561f9c7e2d 100644 (file)
@@ -29,36 +29,35 @@ extern char const control_socket[] = "control socket";
 class TranslatorControlSocketTestv4 :
     public GenericTranslatorTest<control_socket, TranslatorControlSocket> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorControlSocketTestv4() {
         model_ = KEA_DHCP4_SERVER;
     }
 
     virtual ~TranslatorControlSocketTestv4() = default;
-};
+};  // TranslatorControlSocketTestv4
+
 class TranslatorControlSocketTestv6 :
     public GenericTranslatorTest<control_socket, TranslatorControlSocket> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorControlSocketTestv6() {
         model_ = KEA_DHCP6_SERVER;
     }
 
     virtual ~TranslatorControlSocketTestv6() = default;
-};
+};  // TranslatorControlSocketTestv6
+
 class TranslatorControlSocketTestCtrlAgent :
     public GenericTranslatorTest<control_socket, TranslatorControlSocket> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorControlSocketTestCtrlAgent() {
         model_ = KEA_CTRL_AGENT;
     }
 
     virtual ~TranslatorControlSocketTestCtrlAgent() = default;
-};
+};  // TranslatorControlSocketTestCtrlAgent
 
 // This test verifies that an empty control socket can be properly
 // translated from YANG to JSON.
index 507276a035cf7c27b9cc6d635365537bab418b67..f46ca263d8f3a61873744e1c8df16306d3c4ea16 100644 (file)
@@ -33,7 +33,7 @@ public:
     }
 
     virtual ~TranslatorDatabaseTestv4() = default;
-};
+};  // TranslatorDatabaseTestv4
 
 class TranslatorDatabaseTestv6 :
     public GenericTranslatorTest<database_access, TranslatorDatabase> {
@@ -43,7 +43,7 @@ public:
     }
 
     virtual ~TranslatorDatabaseTestv6() = default;
-};
+};  // TranslatorDatabaseTestv6
 
 // This test verifies that an empty database can be properly
 // translated from YANG to JSON.
@@ -140,22 +140,20 @@ extern char const database_accesses[] = "database accesses";
 class TranslatorDatabasesTestv4 :
     public GenericTranslatorTest<database_accesses, TranslatorDatabases> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorDatabasesTestv4() {
         model_ = KEA_DHCP4_SERVER;
     }
-};
+};  // TranslatorDatabasesTestv4
 
 class TranslatorDatabasesTestv6 :
     public GenericTranslatorTest<database_accesses, TranslatorDatabases> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorDatabasesTestv6() {
         model_ = KEA_DHCP6_SERVER;
     }
-};
+};  // TranslatorDatabasesTestv6
 
 // This test verifies that an empty database list can be properly
 // translated from YANG to JSON.
index 9d9d057f4cb1f77ce647e21d19b04658c7489191..80d0dcbe2827f29e0acc6954c4cbf014800704d9 100644 (file)
@@ -30,22 +30,20 @@ extern char const host_reservations[] = "host reservations";
 class TranslatorHostsTestv4 :
     public GenericTranslatorTest<host_reservations, TranslatorHosts> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorHostsTestv4() {
         model_ = KEA_DHCP4_SERVER;
     }
-};
+};  // TranslatorHostsTestv4
 
 class TranslatorHostsTestv6 :
     public GenericTranslatorTest<host_reservations, TranslatorHosts> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorHostsTestv6() {
         model_ = KEA_DHCP6_SERVER;
     }
-};
+};  // TranslatorHostsTestv6
 
 // This test verifies that an empty host reservation list can be properly
 // translated from YANG to JSON.
@@ -190,4 +188,4 @@ TEST_F(TranslatorHostsTestv6, getMany) {
               "\"ip-addresses\": [ \"2001:db8::2\" ] } ]");
 }
 
-}; // end of anonymous namespace
+}  // anonymous namespace
index e493a9dab11d01067790ecd6a346bf2bb077f7bc..6a784c186d30610def2d089759132b4f60a2469a 100644 (file)
@@ -28,21 +28,20 @@ extern char const logger_list[] = "logger list";
 class TranslatorLoggersTestv4 :
     public GenericTranslatorTest<logger_list, TranslatorLoggers> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorLoggersTestv4() {
         model_ = KEA_DHCP4_SERVER;
     }
-};
+};  // TranslatorLoggersTestv4
+
 class TranslatorLoggersTestv6 :
     public GenericTranslatorTest<logger_list, TranslatorLoggers> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorLoggersTestv6() {
         model_ = KEA_DHCP6_SERVER;
     }
-};
+};  // TranslatorLoggersTestv6
 
 // This test verifies that an empty logger list can be properly
 // translated from YANG to JSON.
index 3d9ce6e4a34d5786d768ddb4f8a048e163dbf7c2..9ca9d62cbcdf0eb281d7d9f7aac736bc60020bd8 100644 (file)
@@ -29,22 +29,20 @@ extern char const option_data_list[] = "option data list";
 class TranslatorOptionDataListTestv4 :
     public GenericTranslatorTest<option_data_list, TranslatorOptionDataList> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorOptionDataListTestv4() {
         model_ = KEA_DHCP4_SERVER;
      }
-};
+};  // TranslatorOptionDataListTestv4
 
 class TranslatorOptionDataListTestv6 :
     public GenericTranslatorTest<option_data_list, TranslatorOptionDataList> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorOptionDataListTestv6() {
         model_ = KEA_DHCP6_SERVER;
      }
-};
+};  // TranslatorOptionDataListTestv6
 
 // This test verifies that an empty option data list can be properly
 // translated from YANG to JSON.
index 2da3af6632b8bc11cd822f5004cea8eae0246dd7..8a024fd5c0e3a3c168fa0c1db0793c3139d8305a 100644 (file)
@@ -29,30 +29,27 @@ extern char const option_definition_list[] = "option definition list";
 class TranslatorOptionDefListTestKeaV4 :
     public GenericTranslatorTest<option_definition_list, TranslatorOptionDefList> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorOptionDefListTestKeaV4() {
         model_ = KEA_DHCP4_SERVER;
     }
-};
+};  // TranslatorOptionDefListTestKeaV4
 class TranslatorOptionDefListTestKeaV6 :
     public GenericTranslatorTest<option_definition_list, TranslatorOptionDefList> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorOptionDefListTestKeaV6() {
         model_ = KEA_DHCP6_SERVER;
     }
-};
+};  // TranslatorOptionDefListTestKeaV6
 class TranslatorOptionDefListTestIetfV6 :
     public GenericTranslatorTest<option_definition_list, TranslatorOptionDefList> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorOptionDefListTestIetfV6() {
         model_ = IETF_DHCPV6_SERVER;
     }
-};
+};  // TranslatorOptionDefListTestIetfV6
 
 // This test verifies that an empty option definition list can be properly
 // translated from YANG to JSON.
@@ -139,4 +136,4 @@ TEST_F(TranslatorOptionDefListTestKeaV6, set) {
     EXPECT_TRUE(def->equals(*got->get(0)));
 }
 
-}; // end of anonymous namespace
+}  // anonymous namespace
index 2b5b43c128c4def0167fc5545950388cafdde484..9cdd1dd3904d4b1da6c0aa21ee7aac45dffe3420 100644 (file)
@@ -29,21 +29,19 @@ extern char const pd_pool_list[] = "pd pool list";
 class TranslatorPdPoolsTestKeaV6 :
     public GenericTranslatorTest<pd_pool_list, TranslatorPdPools> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorPdPoolsTestKeaV6() {
         model_ = KEA_DHCP6_SERVER;
     }
-};
+};  // TranslatorPdPoolsTestKeaV6
 class TranslatorPdPoolsTestIetfV6 :
     public GenericTranslatorTest<pd_pool_list, TranslatorPdPools> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorPdPoolsTestIetfV6() {
         model_ = IETF_DHCPV6_SERVER;
     }
-};
+};  // TranslatorPdPoolsTestIetfV6
 
 // This test verifies that an empty pd pool list can be properly
 // translated from YANG to JSON using the IETF model.
index 823569e2fb98b649acd218bd22ecf77184a77d55..62e039228d62b48c3f33290929f72aad0a90f881 100644 (file)
@@ -29,30 +29,27 @@ extern char const pool_list[] = "pool list";
 class TranslatorPoolsTestKeaV4 :
     public GenericTranslatorTest<pool_list, TranslatorPools> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorPoolsTestKeaV4() {
         model_ = KEA_DHCP4_SERVER;
     }
-};
+};  // TranslatorPoolsTestKeaV4
 class TranslatorPoolsTestKeaV6 :
     public GenericTranslatorTest<pool_list, TranslatorPools> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorPoolsTestKeaV6() {
         model_ = KEA_DHCP6_SERVER;
     }
-};
+};  // TranslatorPoolsTestKeaV6
 class TranslatorPoolsTestIetfV6 :
     public GenericTranslatorTest<pool_list, TranslatorPools> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorPoolsTestIetfV6() {
         model_ = IETF_DHCPV6_SERVER;
     }
-};
+};  // TranslatorPoolsTestIetfV6
 
 // This test verifies that an empty pool list can be properly
 // translated from YANG to JSON using IETF model.
index 3358b3236324882c0231c191c65eb3e063ece9c7..9d617c01da31e4c68c863bda1a71a87eb00be322 100644 (file)
@@ -28,21 +28,19 @@ extern char const shared_networks[] = "shared networks";
 class TranslatorSharedNetworksTestKeaV4 :
     public GenericTranslatorTest<shared_networks, TranslatorSharedNetworks> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorSharedNetworksTestKeaV4() {
         model_ = KEA_DHCP4_SERVER;
     }
-};
+};  // TranslatorSharedNetworksTestKeaV4
 class TranslatorSharedNetworksTestKeaV6 :
     public GenericTranslatorTest<shared_networks, TranslatorSharedNetworks> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorSharedNetworksTestKeaV6() {
         model_ = KEA_DHCP6_SERVER;
     }
-};
+};  // TranslatorSharedNetworksTestKeaV6
 
 // This test verifies that an empty shared network list can be properly
 // translated from YANG to JSON.
index 3d3dc674f46ebe0875ca0d4ca5bc4c3006e65cf9..05db775de32c4db395bc60e8a8eceafd9ce445e2 100644 (file)
@@ -28,30 +28,29 @@ extern char const subnet_list[] = "subnet list";
 class TranslatorSubnetsTestKeaV4 :
     public GenericTranslatorTest<subnet_list, TranslatorSubnets> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorSubnetsTestKeaV4() {
         model_ = KEA_DHCP4_SERVER;
     }
-};
+};  // TranslatorSubnetsTestKeaV4
+
 class TranslatorSubnetsTestKeaV6 :
     public GenericTranslatorTest<subnet_list, TranslatorSubnets> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorSubnetsTestKeaV6() {
         model_ = KEA_DHCP6_SERVER;
     }
-};
+};  // TranslatorSubnetsTestKeaV6
+
 class TranslatorSubnetsTestIetfV6 :
     public GenericTranslatorTest<subnet_list, TranslatorSubnets> {
 public:
-
-    /// Constructor.
+    /// @brief Constructor
     TranslatorSubnetsTestIetfV6() {
         model_ = IETF_DHCPV6_SERVER;
     }
-};
+};  // TranslatorSubnetsTestIetfV6
 
 // This test verifies that an empty subnet list can be properly
 // translated from YANG to JSON using IETF model.
index c17f8020c67373999e05bc8801b3d083bbf73ce9..74d1323f5845f93a4da6346daf2b72570de44596 100644 (file)
@@ -44,7 +44,7 @@ private:
         session.deleteItem("/keatest-module:presence-container");
         session.applyChanges();
     }
-};
+};  // TranslatorBasicTest
 
 namespace {
 
@@ -545,4 +545,4 @@ TEST_F(TranslatorBasicTest, list) {
     EXPECT_FALSE(element);
 }
 
-} // anonymous namespace
+}  // anonymous namespace
index 7f6b7356e5d179e9c464fef85b1fdd597a628f51..e94d302d2024ea0790b462120c334907cf72e05a 100644 (file)
@@ -633,7 +633,7 @@ const std::vector<std::pair<std::string, YRTree> > TEST_CONFIGS =
     { subnetOptionsModelKeaDhcp4,  subnetOptionsTreeKeaDhcp4 },
     { subnetOptionsModelKeaDhcp6,  subnetOptionsTreeKeaDhcp6 },
     { designExampleModel,          designExampleTree }
-};
+};  // TEST_CONFIGS
 
 }  // namespace test
 }  // namespace yang
index b49991d4500d3d346b5678cc76f2f5a309db3d10..c82954189a2449f02a69df1dbc489b1f7a11f4b7 100644 (file)
@@ -22,7 +22,6 @@ namespace test {
 /// It is used in tests to conduct operations on whole configurations.
 class YangRepr {
 public:
-
     /// @brief Constructor.
     ///
     /// @param model The model name.
@@ -139,7 +138,7 @@ public:
 private:
     /// @brief The model name.
     std::string model_;
-};
+};  // YangRepr
 
 /// @brief Alias for Items.
 typedef YangRepr::YangReprItem YRItem;
index 9cfa804eb080289099c4f25cd3a455150a0b4a40..4e002d68ad3b29a771c19e45a13fc6718542cb63 100644 (file)
@@ -26,9 +26,6 @@ TranslatorBasic::TranslatorBasic(Session session, const string& model)
     : session_(session), model_(model) {
 }
 
-TranslatorBasic::~TranslatorBasic() {
-}
-
 void
 TranslatorBasic::checkAndGetLeaf(ElementPtr& storage,
                                  DataNode const& data_node,
index 31d941efc793a62ebc643a6b7ef39beaa2b44665..ae81cacb6b0d071b7ca1fbf935fa6f04b4095e11 100644 (file)
@@ -26,7 +26,7 @@ public:
     TranslatorBasic(sysrepo::Session session, const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorBasic();
+    virtual ~TranslatorBasic() = default;
 
     /// @brief Retrieves a child YANG data node identified by name from the
     /// given parent YANG container node and stores it in the specified storage.
@@ -209,7 +209,7 @@ protected:
 
     /// @brief The model.
     std::string model_;
-};
+};  // TranslatorBasic
 
 }  // namespace yang
 }  // namespace isc
index d24c8eb497246236324bd0859c854875566bf64c..54c88647c34729ff86fcc6e91244b13e011728e3 100644 (file)
@@ -27,9 +27,6 @@ TranslatorClass::TranslatorClass(Session session, const string& model)
       TranslatorOptionDefList(session, model) {
 }
 
-TranslatorClass::~TranslatorClass() {
-}
-
 ElementPtr
 TranslatorClass::getClass(DataNode const& data_node) {
     try {
@@ -183,9 +180,6 @@ TranslatorClasses::TranslatorClasses(Session session, const string& model)
       TranslatorClass(session, model) {
 }
 
-TranslatorClasses::~TranslatorClasses() {
-}
-
 ElementPtr
 TranslatorClasses::getClasses(DataNode const& data_node) {
     try {
index c02da15d030ad1613c3a16b90bb6da4b653dfcf5..17b80bcd2c7993f6d67ebb8a6ef560c0e2b7bef8 100644 (file)
@@ -76,7 +76,6 @@ namespace yang {
 class TranslatorClass : virtual public TranslatorOptionDataList,
     virtual public TranslatorOptionDefList  {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -84,7 +83,7 @@ public:
     TranslatorClass(sysrepo::Session session, const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorClass();
+    virtual ~TranslatorClass() = default;
 
     /// @brief Translate a client class from YANG to JSON.
     ///
@@ -131,7 +130,7 @@ protected:
     /// @param elem The JSON element.
     void setClassKea(const std::string& xpath,
                      isc::data::ConstElementPtr elem);
-};
+};  // TranslatorClass
 
 /// @brief A translator class for converting a client class list between
 /// YANG and JSON.
@@ -140,7 +139,6 @@ protected:
 /// not define client class contents.
 class TranslatorClasses : virtual public TranslatorClass {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -148,7 +146,7 @@ public:
     TranslatorClasses(sysrepo::Session session, const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorClasses();
+    virtual ~TranslatorClasses() = default;
 
     /// @brief Translate client classes from YANG to JSON.
     ///
@@ -198,7 +196,7 @@ protected:
     /// @throw BadValue on client class without name.
     void setClassesKea(const std::string& xpath,
                        isc::data::ConstElementPtr elem);
-};
+};  // TranslatorClasses
 
 }  // namespace yang
 }  // namespace isc
index ed32ec8942e1090e912ba0ab87b7c7df9a876d85..1c1dafbb394eb6b73133f2a96ecd851c9c8f992f 100644 (file)
@@ -44,9 +44,6 @@ TranslatorConfig::TranslatorConfig(Session session, const string& model)
       TranslatorLoggers(session, model) {
 }
 
-TranslatorConfig::~TranslatorConfig() {
-}
-
 ElementPtr
 TranslatorConfig::getConfig() {
     try {
index 40bfef641c9ea2f821e21432313e3acd4c5d279a..a049fe0d84a0bad12634b2ed46a7e540aa0f4f9e 100644 (file)
@@ -408,7 +408,6 @@ class TranslatorConfig : virtual public TranslatorControlSocket,
     virtual public TranslatorSharedNetworks,
     virtual public TranslatorLoggers {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -416,7 +415,7 @@ public:
     TranslatorConfig(sysrepo::Session session, const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorConfig();
+    virtual ~TranslatorConfig() = default;
 
     /// @brief Translate the whole DHCP server configuration from YANG to JSON.
     ///
@@ -550,9 +549,9 @@ protected:
     ///
     /// @param elem The JSON element.
     void setServerKeaDhcp6(isc::data::ConstElementPtr elem);
-};
+};  // TranslatorConfig
 
-}  // end of namespace isc::yang
-}  // end of namespace isc
+}  // namespace yang
+}  // namespace isc
 
 #endif // ISC_TRANSLATOR_CONFIG_H
index cc26d992a3c276e0a75b51141f19a032403de027..73893869c7c7b39c676f762eb0034c9ebdd18548 100644 (file)
@@ -24,9 +24,6 @@ TranslatorControlSocket::TranslatorControlSocket(Session session,
     : TranslatorBasic(session, model) {
 }
 
-TranslatorControlSocket::~TranslatorControlSocket() {
-}
-
 ElementPtr
 TranslatorControlSocket::getControlSocket(DataNode const& data_node) {
     try {
index 0e9ea65085a4b88b0db8b62cd36cfe7918314b21..f78168627b4fe4a86f44f681a8870c1330b927bb 100644 (file)
@@ -68,7 +68,6 @@ namespace yang {
 /// - kea-ctrl-agent
 class TranslatorControlSocket : virtual public TranslatorBasic {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -77,7 +76,7 @@ public:
                             const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorControlSocket();
+    virtual ~TranslatorControlSocket() = default;
 
     /// @brief Translate a control socket from YANG to JSON.
     ///
@@ -128,7 +127,7 @@ protected:
     /// @throw BadValue on control socket without socket type or name.
     void setControlSocketKea(const std::string& xpath,
                              isc::data::ConstElementPtr elem);
-};
+};  // TranslatorControlSocket
 
 }  // namespace yang
 }  // namespace isc
index 745c7d215c3b30669bf1cd10eb9781229926b749..e64e3fa15388f50e7bf191c38d0daf90f92131cc 100644 (file)
@@ -23,9 +23,6 @@ TranslatorDatabase::TranslatorDatabase(Session session, const string& model)
     : TranslatorBasic(session, model) {
 }
 
-TranslatorDatabase::~TranslatorDatabase() {
-}
-
 ElementPtr
 TranslatorDatabase::getDatabase(DataNode const& data_node) {
     try {
@@ -147,9 +144,6 @@ TranslatorDatabases::TranslatorDatabases(Session session,
       TranslatorDatabase(session, model) {
 }
 
-TranslatorDatabases::~TranslatorDatabases() {
-}
-
 ElementPtr
 TranslatorDatabases::getDatabases(DataNode const& data_node,
                                   string const& xpath) {
index 93e13cd1cd0ac86f34154ab7b3454dfefa3f83fb..fed3f9aba1f5f6ea0e63607872275a409f4b1300 100644 (file)
@@ -103,7 +103,6 @@ namespace yang {
 /// - kea-dhcp6-server
 class TranslatorDatabase : virtual public TranslatorBasic {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -111,7 +110,7 @@ public:
     TranslatorDatabase(sysrepo::Session session, const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorDatabase();
+    virtual ~TranslatorDatabase() = default;
 
     /// @brief Translate a database access from YANG to JSON.
     ///
@@ -166,7 +165,7 @@ protected:
     void setDatabaseKea(const std::string& xpath,
                         isc::data::ConstElementPtr elem,
                         bool skip);
-};
+};  // TranslatorDatabase
 
 /// @brief A translator class for converting a database access list between
 /// YANG and JSON.
@@ -174,7 +173,6 @@ protected:
 /// Supports kea-dhcp[46]-server, does not exist in ietf-dhcpv6-server.
 class TranslatorDatabases : virtual public TranslatorDatabase {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -182,7 +180,7 @@ public:
     TranslatorDatabases(sysrepo::Session session, const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorDatabases();
+    virtual ~TranslatorDatabases() = default;
 
     /// @brief Translate database accesses from YANG to JSON.
     ///
@@ -238,7 +236,7 @@ protected:
     /// @throw BadValue on database without type,
     void setDatabasesKea(const std::string& xpath,
                          isc::data::ConstElementPtr elem);
-};
+};  // TranslatorDatabases
 
 }  // namespace yang
 }  // namespace isc
index 01bea528fea0992e11c72ced39f45dddd928e249..69e22418c8e1facb804bb02a48320c6368ab7b10 100644 (file)
@@ -25,9 +25,6 @@ TranslatorHost::TranslatorHost(Session session, const string& model)
       TranslatorOptionDataList(session, model) {
 }
 
-TranslatorHost::~TranslatorHost() {
-}
-
 ElementPtr
 TranslatorHost::getHost(DataNode const& data_node) {
     try {
@@ -196,9 +193,6 @@ TranslatorHosts::TranslatorHosts(Session session, const string& model)
       TranslatorHost(session, model) {
 }
 
-TranslatorHosts::~TranslatorHosts() {
-}
-
 ElementPtr
 TranslatorHosts::getHosts(DataNode const& data_node) {
     return getList<TranslatorHost>(data_node, "host", *this,
index a8ae41bad291d39661617f6cc5176bd2128c809f..6c0c4fdf826b36d499fef71c05d9d3a0f874ad02 100644 (file)
@@ -113,7 +113,6 @@ namespace yang {
 /// ietf-dhcpv6-server is not supported yet.
 class TranslatorHost : virtual public TranslatorOptionDataList {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -121,7 +120,7 @@ public:
     TranslatorHost(sysrepo::Session session, const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorHost();
+    virtual ~TranslatorHost() = default;
 
     /// @brief Translate a host reservation from YANG to JSON.
     ///
@@ -165,7 +164,7 @@ protected:
     /// @param xpath The xpath of the host reservation.
     /// @param elem The JSON element.
     void setHostKea(const std::string& xpath, isc::data::ConstElementPtr elem);
-};
+};  // TranslatorHost
 
 /// @brief A translator class for converting host reservations list between
 /// YANG and JSON.
@@ -177,7 +176,6 @@ protected:
 /// The ietf-dhcpv6-server model is not yet supported.
 class TranslatorHosts : virtual public TranslatorHost {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -185,7 +183,7 @@ public:
     TranslatorHosts(sysrepo::Session session, const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorHosts();
+    virtual ~TranslatorHosts() = default;
 
     /// @brief Translate host reservations from YANG to JSON.
     ///
@@ -224,7 +222,7 @@ protected:
     /// @throw BadValue on host reservation without known identifier type.
     void setHostsKea(const std::string& xpath,
                      isc::data::ConstElementPtr elem);
-};
+};  // TranslatorHosts
 
 }  // namespace yang
 }  // namespace isc
index a9e4c19d62d00d7ddb797569546e9b6f63577321..be7f1f5d9f26683fd88b5d68ba17f7769cf54732 100644 (file)
@@ -23,9 +23,6 @@ TranslatorLogger::TranslatorLogger(Session session, const string& model)
     : TranslatorBasic(session, model) {
 }
 
-TranslatorLogger::~TranslatorLogger() {
-}
-
 ElementPtr
 TranslatorLogger::getLogger(DataNode const& data_node) {
     try {
@@ -189,9 +186,6 @@ TranslatorLoggers::TranslatorLoggers(Session session, const string& model)
       TranslatorLogger(session, model) {
 }
 
-TranslatorLoggers::~TranslatorLoggers() {
-}
-
 ConstElementPtr
 TranslatorLoggers::getLoggers(DataNode const& data_node) {
     try {
index a9104d16f2a4b5ed4b0b933498dcf024196c3b4d..11afe63e1a2ad6f70dcbfdeb3579fe1b971c5754 100644 (file)
@@ -93,7 +93,6 @@ namespace yang {
 /// - kea-ctrl-agent
 class TranslatorLogger : virtual public TranslatorBasic {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -101,7 +100,7 @@ public:
     TranslatorLogger(sysrepo::Session session, const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorLogger();
+    virtual ~TranslatorLogger() = default;
 
     /// @brief Translate a logger from YANG to JSON.
     ///
@@ -167,7 +166,7 @@ protected:
     /// @param elem The JSON element.
     void setLoggerKea(const std::string& xpath,
                       isc::data::ConstElementPtr elem);
-};
+};  // TranslatorLogger
 
 /// @brief A translator class for converting a logger list between
 /// YANG and JSON.
@@ -175,7 +174,6 @@ protected:
 /// Currently supports all kea servers and agents. Specific to Kea.
 class TranslatorLoggers : virtual public TranslatorLogger {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -183,7 +181,7 @@ public:
     TranslatorLoggers(sysrepo::Session session, const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorLoggers();
+    virtual ~TranslatorLoggers() = default;
 
     /// @brief Translate loggers from YANG to JSON.
     ///
@@ -228,7 +226,7 @@ protected:
     /// @throw BadValue on a logger without name.
     void setLoggersKea(const std::string& xpath,
                        isc::data::ConstElementPtr elem);
-};
+};  // TranslatorLoggers
 
 }  // namespace yang
 }  // namespace isc
index 1a6364d7e3e0077fd0d23ec345ebbf3c8afe9951..9f6447e0747ba779fe3df7b7b26ceb98b93ce319 100644 (file)
@@ -24,9 +24,6 @@ TranslatorOptionData::TranslatorOptionData(Session session,
     : TranslatorBasic(session, model) {
 }
 
-TranslatorOptionData::~TranslatorOptionData() {
-}
-
 ElementPtr
 TranslatorOptionData::getOptionData(DataNode const& data_node) {
     try {
@@ -138,9 +135,6 @@ TranslatorOptionDataList::TranslatorOptionDataList(Session session,
       TranslatorOptionData(session, model) {
 }
 
-TranslatorOptionDataList::~TranslatorOptionDataList() {
-}
-
 ConstElementPtr
 TranslatorOptionDataList::getOptionDataList(DataNode const& data_node) {
     try {
index d6c8e5fc49202b04de471e8ebb63f457da36c1e8..f05b6c0784d7f14abfc290c2a836a5fe5fab5980 100644 (file)
@@ -75,7 +75,6 @@ namespace yang {
 /// Currently supports on kea-dhcp[46]-server, not yet ietf-dhcpv6-server.
 class TranslatorOptionData : virtual public TranslatorBasic {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -83,7 +82,7 @@ public:
     TranslatorOptionData(sysrepo::Session session, const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorOptionData();
+    virtual ~TranslatorOptionData() = default;
 
     /// @brief Translate an option data from YANG to JSON.
     ///
@@ -126,7 +125,7 @@ protected:
     /// @param elem The JSON element.
     void setOptionDataKea(const std::string& xpath,
                           isc::data::ConstElementPtr elem);
-};
+};  // TranslatorOptionData
 
 /// @brief A translator class for converting an option data list between
 /// YANG and JSON.
@@ -134,7 +133,6 @@ protected:
 /// Currently supports on kea-dhcp[46]-server, not yet ietf-dhcpv6-server.
 class TranslatorOptionDataList : virtual public TranslatorOptionData {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -143,7 +141,7 @@ public:
                              const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorOptionDataList();
+    virtual ~TranslatorOptionDataList() = default;
 
     /// @brief Translate option data list from YANG to JSON.
     ///
@@ -185,7 +183,7 @@ protected:
     /// @throw BadValue on option data without code or space.
     void setOptionDataListKea(const std::string& xpath,
                               isc::data::ConstElementPtr elem);
-};
+};  // TranslatorOptionDataList
 
 }  // namespace yang
 }  // namespace isc
index 1d3a5d7e1f5656fcd29bc7a7c05a3f0e48b822ff..eac2f4db0778cff2bb38ea217ad2e39352db99d4 100644 (file)
@@ -24,9 +24,6 @@ TranslatorOptionDef::TranslatorOptionDef(Session session,
     : TranslatorBasic(session, model) {
 }
 
-TranslatorOptionDef::~TranslatorOptionDef() {
-}
-
 ElementPtr
 TranslatorOptionDef::getOptionDef(DataNode const& data_node) {
     try {
@@ -146,9 +143,6 @@ TranslatorOptionDefList::TranslatorOptionDefList(Session session,
       TranslatorOptionDef(session, model) {
 }
 
-TranslatorOptionDefList::~TranslatorOptionDefList() {
-}
-
 ConstElementPtr
 TranslatorOptionDefList::getOptionDefList(DataNode const& data_node) {
     try {
index fec7d8c1f5c162009adceae7dd972b3add208e06..0d0f0f6e7576e33f0d83523fe839d6ff225e0130 100644 (file)
@@ -78,7 +78,6 @@ namespace yang {
 /// @todo: Support for ietf-dhcpv6-server model.
 class TranslatorOptionDef : virtual public TranslatorBasic {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -86,7 +85,7 @@ public:
     TranslatorOptionDef(sysrepo::Session session, const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorOptionDef();
+    virtual ~TranslatorOptionDef() = default;
 
     /// @brief Translate an option definition from YANG to JSON.
     ///
@@ -131,7 +130,7 @@ protected:
     /// @throw BadValue on option definition without name or type.
     void setOptionDefKea(const std::string& xpath,
                          isc::data::ConstElementPtr elem);
-};
+};  // TranslatorOptionDef
 
 // @brief A translator class for converting an option definition list
 // between YANG and JSON.
@@ -140,7 +139,6 @@ protected:
 /// @todo: Support for ietf-dhcpv6-server model.
 class TranslatorOptionDefList : virtual public TranslatorOptionDef {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -149,7 +147,7 @@ public:
                             const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorOptionDefList();
+    virtual ~TranslatorOptionDefList() = default;
 
     /// @brief Translate option definition list from YANG to JSON.
     ///
@@ -193,7 +191,7 @@ protected:
     /// @throw BadValue on option definition without code or space.
     void setOptionDefListKea(const std::string& xpath,
                              isc::data::ConstElementPtr elem);
-};
+};  // TranslatorOptionDefList
 
 }  // namespace yang
 }  // namespace isc
index 19e1707f08f597474714d13ae0b66cd05f40975d..30c5e302454766dfca1c82d370bee0a31f93eeb6 100644 (file)
@@ -26,9 +26,6 @@ TranslatorPdPool::TranslatorPdPool(Session session, const string& model)
       TranslatorOptionDataList(session, model) {
 }
 
-TranslatorPdPool::~TranslatorPdPool() {
-}
-
 ElementPtr
 TranslatorPdPool::getPdPool(DataNode const& data_node) {
     try {
@@ -292,9 +289,6 @@ TranslatorPdPools::TranslatorPdPools(Session session, const string& model)
       TranslatorPdPool(session, model) {
 }
 
-TranslatorPdPools::~TranslatorPdPools() {
-}
-
 ElementPtr
 TranslatorPdPools::getPdPools(DataNode const& data_node) {
     try {
index 6492db78a99ca33f41849fb55373c07e577edf3b..3667edb13eb67f1e9bbe901c94c96110833590f8 100644 (file)
@@ -116,7 +116,6 @@ namespace yang {
 /// - ietf-dhcpv6-server (partial support)
 class TranslatorPdPool : virtual public TranslatorOptionDataList {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -124,7 +123,7 @@ public:
     TranslatorPdPool(sysrepo::Session session, const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorPdPool();
+    virtual ~TranslatorPdPool() = default;
 
     /// @brief Translate a pd-pool from YANG to JSON.
     ///
@@ -185,7 +184,7 @@ protected:
     /// @param elem The JSON element.
     void setPdPoolKea(const std::string& xpath,
                       isc::data::ConstElementPtr elem);
-};
+};  // TranslatorPdPool
 
 /// @brief A translator class for converting a pd-pool list between
 /// YANG and JSON.
@@ -195,7 +194,6 @@ protected:
 /// - ietf-dhcpv6-server (partial support)
 class TranslatorPdPools : virtual public TranslatorPdPool {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -203,7 +201,7 @@ public:
     TranslatorPdPools(sysrepo::Session session, const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorPdPools();
+    virtual ~TranslatorPdPools() = default;
 
     /// @brief Translate pd-pools from YANG to JSON.
     ///
@@ -251,7 +249,7 @@ protected:
     /// @throw BadValue on pd-pool without prefix or prefix length.
     void setPdPoolsPrefix(const std::string& xpath,
                           isc::data::ConstElementPtr elem);
-};
+};  // TranslatorPdPools
 
 }  // namespace yang
 }  // namespace isc
index e891d17f5592a6c704adb7e376148e776d4f38d1..8f13f21791c59363b5e738e9636a3b324158f9f1 100644 (file)
@@ -29,9 +29,6 @@ TranslatorPool::TranslatorPool(Session session, const string& model)
       TranslatorOptionDataList(session, model) {
 }
 
-TranslatorPool::~TranslatorPool() {
-}
-
 ElementPtr
 TranslatorPool::getPool(DataNode const& data_node) {
     try {
@@ -276,9 +273,6 @@ TranslatorPools::TranslatorPools(Session session, const string& model)
       TranslatorPool(session, model) {
 }
 
-TranslatorPools::~TranslatorPools() {
-}
-
 ElementPtr
 TranslatorPools::getPools(DataNode const& data_node) {
     try {
index 0dbfd1f1176e7b97bcb6b1c5e74334d4ee6df70e..0000efafa644262e76cd2305a6ec800e7b298afe 100644 (file)
@@ -124,7 +124,6 @@ namespace yang {
 /// @endcode
 class TranslatorPool : virtual public TranslatorOptionDataList {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -132,7 +131,7 @@ public:
     TranslatorPool(sysrepo::Session session, const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorPool();
+    virtual ~TranslatorPool() = default;
 
     /// @brief Translate a pool from YANG to JSON.
     ///
@@ -202,14 +201,13 @@ protected:
     /// @param elem The JSON element.
     /// @throw BadValue on a pool without a well formed prefix.
     void setPoolKea(const std::string& xpath, isc::data::ConstElementPtr elem);
-};
+};  // TranslatorPool
 
 /// @brief A translator class for converting pools between YANG and JSON.
 ///
 /// Currently supports on kea-dhcp[46]-server and partially ietf-dhcpv6-server.
 class TranslatorPools : virtual public TranslatorPool {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -217,7 +215,7 @@ public:
     TranslatorPools(sysrepo::Session session, const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorPools();
+    virtual ~TranslatorPools() = default;
 
     /// @brief Translate pools from YANG to JSON.
     ///
@@ -272,7 +270,7 @@ protected:
     /// @throw BadValue on a pool without a prefix.
     void setPoolsByAddresses(const std::string& xpath,
                              isc::data::ConstElementPtr elem);
-};
+};  // TranslatorPools
 
 }  // namespace yang
 }  // namespace isc
index 571d940094a66d91bb0007cd1d9d912531d2e8e5..8d2d200d605ffa50d6d0897ab97896343c8dca1f 100644 (file)
@@ -34,9 +34,6 @@ TranslatorSharedNetwork::TranslatorSharedNetwork(Session session,
       TranslatorSubnets(session, model) {
 }
 
-TranslatorSharedNetwork::~TranslatorSharedNetwork() {
-}
-
 ElementPtr
 TranslatorSharedNetwork::getSharedNetwork(DataNode const& data_node) {
     try {
@@ -384,9 +381,6 @@ TranslatorSharedNetworks::TranslatorSharedNetworks(Session session,
       TranslatorSharedNetwork(session, model) {
 }
 
-TranslatorSharedNetworks::~TranslatorSharedNetworks() {
-}
-
 ElementPtr
 TranslatorSharedNetworks::getSharedNetworks(DataNode const& data_node) {
     return getList<TranslatorSharedNetwork>(data_node, "shared-network", *this,
index d060875121916834db32530aeac12c75a3e7a581..ff0158836df3caa45b986bbd2915877f53d5c714 100644 (file)
@@ -165,7 +165,6 @@ namespace yang {
 /// - kea-dhcp6-server
 class TranslatorSharedNetwork : virtual public TranslatorSubnets {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -174,7 +173,7 @@ public:
                             const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorSharedNetwork();
+    virtual ~TranslatorSharedNetwork() = default;
 
     /// @brief Translate a shared network from YANG to JSON.
     ///
@@ -221,7 +220,7 @@ protected:
     void setSharedNetworkKea(const std::string& xpath,
                              isc::data::ConstElementPtr elem,
                              const std::string& subsel);
-};
+};  // TranslatorSharedNetwork
 
 /// @brief A translator class for converting a shared network list between
 /// YANG and JSON.
@@ -231,7 +230,6 @@ protected:
 /// - kea-dhcp6-server
 class TranslatorSharedNetworks : virtual public TranslatorSharedNetwork {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -240,7 +238,7 @@ public:
                              const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorSharedNetworks();
+    virtual ~TranslatorSharedNetworks() = default;
 
     /// @brief Translate shared networks from YANG to JSON.
     ///
@@ -276,7 +274,7 @@ protected:
     /// @throw BadValue on a shared network without name.
     void setSharedNetworksKea(const std::string& xpath,
                               isc::data::ConstElementPtr elem);
-};
+};  // TranslatorSharedNetworks
 
 }  // namespace yang
 }  // namespace isc
index 2bf3f72ee4bac79e0089466a3ba98bb8bca683fa..818a0a3d0495be510d6666bb5d83067c1ba7a7cf 100644 (file)
@@ -32,9 +32,6 @@ TranslatorSubnet::TranslatorSubnet(Session session, const string& model)
       TranslatorHosts(session, model) {
 }
 
-TranslatorSubnet::~TranslatorSubnet() {
-}
-
 ElementPtr
 TranslatorSubnet::getSubnet(DataNode const& data_node) {
     try {
@@ -507,9 +504,6 @@ TranslatorSubnets::TranslatorSubnets(Session session, const string& model)
       TranslatorSubnet(session, model) {
 }
 
-TranslatorSubnets::~TranslatorSubnets() {
-}
-
 ElementPtr
 TranslatorSubnets::getSubnets(DataNode const& data_node) {
     try {
index 1888667dc3d7f882803493a950ac6a9f13b7fd03..cde7f8cbac7ad8796b9aef9356851ffa1f83b996 100644 (file)
@@ -273,7 +273,6 @@ namespace yang {
 class TranslatorSubnet : virtual public TranslatorPools,
     virtual public TranslatorPdPools, virtual public TranslatorHosts {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -281,7 +280,7 @@ public:
     TranslatorSubnet(sysrepo::Session session, const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorSubnet();
+    virtual ~TranslatorSubnet() = default;
 
     /// @brief Get and translate a subnet from YANG to JSON.
     ///
@@ -338,7 +337,7 @@ protected:
     /// @param elem The JSON element.
     void setSubnetKea(const std::string& xpath,
                       isc::data::ConstElementPtr elem);
-};
+};  // TranslatorSubnets
 
 /// @brief A translator class for converting a subnet list between
 /// YANG and JSON.
@@ -346,7 +345,6 @@ protected:
 /// Currently supports on kea-dhcp[46]-server and partially ietf-dhcpv6-server.
 class TranslatorSubnets : virtual public TranslatorSubnet {
 public:
-
     /// @brief Constructor.
     ///
     /// @param session Sysrepo session.
@@ -354,7 +352,7 @@ public:
     TranslatorSubnets(sysrepo::Session session, const std::string& model);
 
     /// @brief Destructor.
-    virtual ~TranslatorSubnets();
+    virtual ~TranslatorSubnets() = default;
 
     /// @brief Get and translate subnets from YANG to JSON.
     ///
@@ -406,7 +404,7 @@ protected:
     void setSubnetsKea(const std::string& xpath,
                        isc::data::ConstElementPtr elem,
                        const std::string& subsel);
-};
+};  // TranslatorSubnets
 
 }  // namespace yang
 }  // namespace isc
index d5b6d1fcaa4b37eff0cd96512f317311a45896a3..d3d67e9aaed3790c0aa691730824cfbcaf8982f6 100644 (file)
@@ -31,7 +31,7 @@ static const std::string KEA_DHCP_DDNS = "kea-dhcp-ddns";
 /// by ISC in the near future.
 static const std::string KEA_CTRL_AGENT = "kea-ctrl-agent";
 
-}  // end of namespace isc::yang
-}  // end of namespace isc
+}  // namespace yang
+}  // namespace isc
 
 #endif // ISC_YANG_MODELS_H
index 1c2f3539b43cb806c34a33b19d7cecb399bc0bf0..9cba6038ddd837013b43d902405c0bee633f062b 100644 (file)
@@ -27,7 +27,7 @@ static const std::map<std::string, std::string> YANG_REVISIONS = {
     { "kea-dhcp6-server", "2022-07-27" },
     { "kea-ctrl-agent", "2019-08-12" },
     { "kea-dhcp-ddns", "2022-07-27" }
-};
+};  // YANG_REVISIONS
 
 }  // namespace yang
 }  // namespace isc
index 200b0bb422aaf44183b15d386a66e756d786c3f1..76edaa25ec575e37b7d3198956bd88ba40069391 100644 (file)
@@ -21,7 +21,7 @@ static const std::map<std::string, std::string> YANG_REVISIONS = {
 // insert the r file here
 };
 
-}; // end of namespace isc::yang
-}; // end of namespace isc
+}  // namespace yang
+}  // namespace isc
 
 #endif // ISC_YANG_REVISIONS_H