Browse Source

[3744] Minor comments update in the JSON parsers.

Marcin Siodelski 10 years ago
parent
commit
81b5979918
2 changed files with 5 additions and 2 deletions
  1. 1 0
      src/bin/dhcp4/json_config_parser.cc
  2. 4 2
      src/bin/dhcp6/json_config_parser.cc

+ 1 - 0
src/bin/dhcp4/json_config_parser.cc

@@ -416,6 +416,7 @@ configureDhcp4Server(Dhcpv4Srv&, isc::data::ConstElementPtr config_set) {
     // Thus we group parsers and will fire them in the right order:
     // all parsers other than: lease-database, subnet4 and option-data parser,
     // then: option-data parser, subnet4 parser, lease-database parser.
+    // Please do not change this order!
     ParserCollection independent_parsers;
     ParserPtr subnet_parser;
     ParserPtr option_parser;

+ 4 - 2
src/bin/dhcp6/json_config_parser.cc

@@ -626,8 +626,10 @@ configureDhcp6Server(Dhcpv6Srv&, isc::data::ConstElementPtr config_set) {
     // depend on the global values. Also, option values configuration
     // must be performed after the option definitions configurations.
     // Thus we group parsers and will fire them in the right order:
-    // all parsers other than subnet6 and option-data parser,
-    // option-data parser, subnet6 parser.
+    // all parsers other than lease-database, subnet6 and
+    // option-data parser, then option-data parser, subnet6 parser,
+    // lease-database parser.
+    // Please do not change this order!
     ParserCollection independent_parsers;
     ParserPtr subnet_parser;
     ParserPtr option_parser;