spelling: should

src/bin/agent/agent_parser.cc

@@ -490,7 +490,7 @@ namespace isc { namespace agent {
     /// The return value of parse ().
     int yyresult;
 
-    // FIXME: This shoud be completely indented.  It is not yet to
+    // FIXME: This should be completely indented.  It is not yet to
     // avoid gratuitous conflicts when merging into the master branch.
     try
       {

src/bin/agent/tests/ca_controller_unittests.cc

@@ -436,7 +436,7 @@ TEST_F(CtrlAgentControllerTest, registeredCommands) {
     // code as CtrlAgentController is not initialized the same way it is
     // in production code. In particular, the way CtrlAgentController
     // is initialized in tests does not call registerCommands().
-    // This is a crude workaround for this problem. Proper solution shoul
+    // This is a crude workaround for this problem. Proper solution should
     // be developed sooner rather than later.
     const DControllerBasePtr& base = getController();
     const CtrlAgentControllerPtr& ctrl =
@@ -466,7 +466,7 @@ TEST_F(CtrlAgentControllerTest, configWrite) {
     // code as CtrlAgentController is not initialized the same way it is
     // in production code. In particular, the way CtrlAgentController
     // is initialized in tests does not call registerCommands().
-    // This is a crude workaround for this problem. Proper solution shoul
+    // This is a crude workaround for this problem. Proper solution should
     // be developed sooner rather than later.
     const DControllerBasePtr& base = getController();
     const CtrlAgentControllerPtr& ctrl

src/bin/d2/d2_parser.cc

@@ -490,7 +490,7 @@ namespace isc { namespace d2 {
     /// The return value of parse ().
     int yyresult;
 
-    // FIXME: This shoud be completely indented.  It is not yet to
+    // FIXME: This should be completely indented.  It is not yet to
     // avoid gratuitous conflicts when merging into the master branch.
     try
       {

src/bin/dhcp4/dhcp4_parser.cc

@@ -535,7 +535,7 @@ namespace isc { namespace dhcp {
     /// The return value of parse ().
     int yyresult;
 
-    // FIXME: This shoud be completely indented.  It is not yet to
+    // FIXME: This should be completely indented.  It is not yet to
     // avoid gratuitous conflicts when merging into the master branch.
     try
       {

src/bin/dhcp6/dhcp6_parser.cc

@@ -535,7 +535,7 @@ namespace isc { namespace dhcp {
     /// The return value of parse ().
     int yyresult;
 
-    // FIXME: This shoud be completely indented.  It is not yet to
+    // FIXME: This should be completely indented.  It is not yet to
     // avoid gratuitous conflicts when merging into the master branch.
     try
       {

src/bin/perfdhcp/command_options.h

@@ -272,7 +272,7 @@ public:
     /// \return true if server-iD to be taken from first package.
     bool isUseFirst() const { return use_first_; }
 
-    /// \brief Check if generated DHCPv6 messages shuold appear as relayed.
+    /// \brief Check if generated DHCPv6 messages should appear as relayed.
     ///
     /// \return true if generated traffic should appear as relayed.
     bool isUseRelayedV6() const { return (v6_relay_encapsulation_level_ > 0); }

src/lib/dhcp/tests/opaque_data_tuple_unittest.cc

@@ -417,7 +417,7 @@ TEST(OpaqueDataTuple, unpack2Byte) {
     for (int i = 0; i < 400; ++i) {
         wire_data.push_back(i);
     }
-    // The unpack shoud succeed.
+    // The unpack should succeed.
     ASSERT_NO_THROW(tuple.unpack(wire_data.begin(), wire_data.end()));
     // The decoded length should be 400.
     ASSERT_EQ(400, tuple.getLength());

src/lib/eval/parser.cc

@@ -586,7 +586,7 @@ namespace isc { namespace eval {
     /// The return value of parse ().
     int yyresult;
 
-    // FIXME: This shoud be completely indented.  It is not yet to
+    // FIXME: This should be completely indented.  It is not yet to
     // avoid gratuitous conflicts when merging into the master branch.
     try
       {