spell check fixes (#456)

liat-grozovik · web-flow · commit af01f37f9afb · 2021-02-10T05:02:07.000-08:00
diff --git a/common/configdb.cpp b/common/configdb.cpp
@@ -382,7 +382,7 @@ void ConfigDBPipeConnector_Native::mod_config(const map<string, map<string, map<
 //     cursor: position to start scanning from
 //
 // Returns:
-//     cur: poition of next item to scan
+//     cur: position of next item to scan
 int ConfigDBPipeConnector_Native::_get_config(DBConnector& client, RedisTransactioner& pipe, map<string, map<string, map<string, string>>>& data, int cursor)
 {
     auto const& rc = client.scan(cursor, "*", REDIS_SCAN_BATCH_SIZE);
diff --git a/common/dbconnector.cpp b/common/dbconnector.cpp
@@ -566,7 +566,7 @@ DBConnector::DBConnector(const string& dbName, unsigned int timeout, bool isTcpC
 DBConnector::DBConnector(const string& dbName, unsigned int timeout, bool isTcpConn)
     : DBConnector(dbName, timeout, isTcpConn, EMPTY_NAMESPACE)
 {
-    // Empty contructor
+    // Empty constructor
 }
 
 int DBConnector::getDbId() const
diff --git a/common/dbconnector.h b/common/dbconnector.h
@@ -181,7 +181,7 @@ class DBConnector : public RedisContext
 #ifdef SWIG
     // SWIG interface file (.i) globally rename map C++ `del` to python `delete`,
     // but applications already followed the old behavior of auto renamed `_del`.
-    // So we implemented old behavior for backward compatiblity
+    // So we implemented old behavior for backward compatibility
     // TODO: remove this function after applications use the function name `delete`
     %pythoncode %{
         def _del(self, *args, **kwargs):
diff --git a/common/json.h b/common/json.h
@@ -52,7 +52,7 @@ class JSon
          fs: the input ifstream representing the json file
          fv: the output vector
        return: boolean
-         True: the input json file has been succefully parsed
+         True: the input json file has been successfully parsed
          False: there are some errors found
      */
     static bool loadJsonFromFile(std::ifstream &fs, std::vector<KeyOpFieldsValuesTuple> &db_items);
diff --git a/common/netdispatcher.cpp b/common/netdispatcher.cpp
@@ -18,7 +18,7 @@ void NetDispatcher::registerMessageHandler(int nlmsg_type, NetMsg *callback)
     MUTEX;
 
     if (m_handlers.find(nlmsg_type) != m_handlers.end())
-        throw "Trying to register on already registerd netlink message";
+        throw "Trying to register on already registered netlink message";
 
     m_handlers[nlmsg_type] = callback;
 }
diff --git a/common/netdispatcher.h b/common/netdispatcher.h
@@ -13,7 +13,7 @@ namespace swss
     {
         public:
 
-            /** Get singlton instance. */
+            /** Get singleton instance. */
             static NetDispatcher& getInstance();
 
             /**
diff --git a/common/netlink.cpp b/common/netlink.cpp
@@ -90,7 +90,7 @@ uint64_t NetLink::readData()
     if (err < 0)
     {
         if (err == -NLE_NOMEM)
-            SWSS_LOG_ERROR("netlink reports out of memory on reading a netlink socket. High possiblity of a lost message");
+            SWSS_LOG_ERROR("netlink reports out of memory on reading a netlink socket. High possibility of a lost message");
         else if (err == -NLE_AGAIN)
             SWSS_LOG_DEBUG("netlink reports NLE_AGAIN on reading a netlink socket");
         else
diff --git a/common/nfnetlink.cpp b/common/nfnetlink.cpp
@@ -134,7 +134,7 @@ uint64_t NfNetlink::readData()
     if (err < 0)
     {
         if (err == -NLE_NOMEM)
-            SWSS_LOG_ERROR("netlink reports out of memory on reading a netfilter netlink socket. High possiblity of a lost message");
+            SWSS_LOG_ERROR("netlink reports out of memory on reading a netfilter netlink socket. High possibility of a lost message");
         else if (err == -NLE_AGAIN)
             SWSS_LOG_DEBUG("netlink reports NLE_AGAIN on reading a netfilter netlink socket");
         else
diff --git a/common/producerstatetable.h b/common/producerstatetable.h
@@ -27,7 +27,7 @@ class ProducerStateTable : public TableBase, public TableName_KeySet
 #ifdef SWIG
     // SWIG interface file (.i) globally rename map C++ `del` to python `delete`,
     // but applications already followed the old behavior of auto renamed `_del`.
-    // So we implemented old behavior for backward compatiblity
+    // So we implemented old behavior for backward compatibility
     // TODO: remove this function after applications use the function name `delete`
     %pythoncode %{
         def _del(self, *args, **kwargs):
diff --git a/common/producertable.h b/common/producertable.h
@@ -38,7 +38,7 @@ class ProducerTable : public TableBase, public TableName_KeyValueOpQueues
 #ifdef SWIG
     // SWIG interface file (.i) globally rename map C++ `del` to python `delete`,
     // but applications already followed the old behavior of auto renamed `_del`.
-    // So we implemented old behavior for backward compatiblity
+    // So we implemented old behavior for backward compatibility
     // TODO: remove this function after applications use the function name `delete`
     %pythoncode %{
         def _del(self, *args, **kwargs):
diff --git a/common/redisreply.cpp b/common/redisreply.cpp
@@ -189,7 +189,7 @@ template<> string RedisReply::getReply<string>()
 template<> RedisMessage RedisReply::getReply<RedisMessage>()
 {
     RedisMessage ret;
-    /* if the Key-space notification is empty, try emtpy message. */
+    /* if the Key-space notification is empty, try empty message. */
     if (getContext()->type == REDIS_REPLY_NIL)
     {
         return ret;
diff --git a/common/redisreply.h b/common/redisreply.h
@@ -62,7 +62,7 @@ class RedisReply
     RedisReply(RedisContext *ctx, const RedisCommand& command, int expectedType);
     RedisReply(RedisContext *ctx, const std::string& command, int expectedType);
 
-    /* auto_ptr for native structue (Free the memory on destructor) */
+    /* auto_ptr for native structure (Free the memory on destructor) */
     RedisReply(redisReply *reply);
 
     /* Free the resources */
@@ -86,10 +86,10 @@ class RedisReply
     template<typename TYPE>
     TYPE getReply();
 
-    /* Check that the staus is OK, throw exception otherwise */
+    /* Check that the status is OK, throw exception otherwise */
     void checkStatusOK();
 
-    /* Check that the staus is QUEUED, throw exception otherwise */
+    /* Check that the status is QUEUED, throw exception otherwise */
     void checkStatusQueued();
 
 private:
diff --git a/common/redistran.cpp b/common/redistran.cpp
@@ -37,7 +37,7 @@ bool RedisTransactioner::exec()
 
     if (size != m_expectedResults.size())
         throw system_error(make_error_code(errc::io_error),
-                "Got to different nuber of answers!");
+                "Got to different number of answers!");
 
     clearResults();
     for (unsigned int i = 0; i < size; i++)
diff --git a/common/schema.h b/common/schema.h
@@ -187,7 +187,7 @@ namespace swss {
 #define FLEX_COUNTER_STATUS_FIELD     "FLEX_COUNTER_STATUS"
 #define FLEX_COUNTER_GROUP_TABLE      "FLEX_COUNTER_GROUP_TABLE"
 
-/***** CONFIGUATION DATABASE *****/
+/***** CONFIGURATION DATABASE *****/
 
 #define CFG_PORT_TABLE_NAME           "PORT"
 #define CFG_PORT_CABLE_LEN_TABLE_NAME "CABLE_LENGTH"
diff --git a/common/selectable.h b/common/selectable.h
@@ -20,7 +20,7 @@ class Selectable
     /* return file handler for the Selectable */
     virtual int getFd() = 0;
 
-    /* Read all data from the fd assicaited with Selectable */
+    /* Read all data from the fd associated with Selectable */
     virtual uint64_t readData() = 0;
 
     /*
diff --git a/common/table.h b/common/table.h
@@ -156,7 +156,7 @@ class Table : public TableBase, public TableEntryEnumerable {
 #ifdef SWIG
     // SWIG interface file (.i) globally rename map C++ `del` to python `delete`,
     // but applications already followed the old behavior of auto renamed `_del`.
-    // So we implemented old behavior for backward compatiblity
+    // So we implemented old behavior for backward compatibility
     // TODO: remove this function after applications use the function name `delete`
     %pythoncode %{
         def _del(self, *args, **kwargs):
diff --git a/tests/redis_piped_state_ut.cpp b/tests/redis_piped_state_ut.cpp
@@ -39,7 +39,7 @@ static inline string field(int i)
 
 static inline string value(int i)
 {
-    if (i == 0) return string(); // emtpy
+    if (i == 0) return string(); // empty
     return string("value") + to_string(i);
 }
 
@@ -474,7 +474,7 @@ TEST(ConsumerStateTable, async_singlethread)
     while ((ret = cs.select(&selectcs, 1000)) == Select::OBJECT)
     {
         c.pop(kco);
-        // keys have been droppd, expecting empty kco.
+        // keys have been dropped, expecting empty kco.
         EXPECT_EQ(kfvOp(kco), "");
         numberOfNotification++;
 
diff --git a/tests/redis_state_ut.cpp b/tests/redis_state_ut.cpp
@@ -38,7 +38,7 @@ static inline string field(int i)
 
 static inline string value(int i)
 {
-    if (i == 0) return string(); // emtpy
+    if (i == 0) return string(); // empty
     return string("value ") + to_string(i);
 }
 
@@ -477,7 +477,7 @@ TEST(ConsumerStateTable, set_pop_del_set_pop_get)
     EXPECT_EQ(values.size(), (unsigned int)maxNumOfFields);
 
     /*
-     * Third pop operation, consumer received two consectuive signals.
+     * Third pop operation, consumer received two consecutive signals.
      * data depleted upon first one
      */
     /*
@@ -537,7 +537,7 @@ TEST(ConsumerStateTable, view_switch)
         }
         p.set(key(i), fields);
     }
-    // Set opeartions should go into temp view
+    // Set operations should go into temp view
     EXPECT_EQ(p.count(), 0);
     for (int i=0; i<numOfKeysToDel; ++i)
     {
diff --git a/tests/redis_subscriber_state_ut.cpp b/tests/redis_subscriber_state_ut.cpp
@@ -39,7 +39,7 @@ static inline string value(int index, int keyid)
 {
     if (keyid == 0)
     {
-        return string(); // emtpy
+        return string(); // empty
     }
 
     return string("value ") + to_string(index) + ":" + to_string(keyid);

Original file line number	Diff line number	Diff line change
`@@ -382,7 +382,7 @@ void ConfigDBPipeConnector_Native::mod_config(const map<string, map<string, map<`
`382`	`382`	`// cursor: position to start scanning from`
`383`	`383`	`//`
`384`	`384`	`// Returns:`
`385`		`-// cur: poition of next item to scan`
	`385`	`+// cur: position of next item to scan`
`386`	`386`	`int ConfigDBPipeConnector_Native::_get_config(DBConnector& client, RedisTransactioner& pipe, map<string, map<string, map<string, string>>>& data, int cursor)`
`387`	`387`	`{`
`388`	`388`	`auto const& rc = client.scan(cursor, "*", REDIS_SCAN_BATCH_SIZE);`
Original file line number	Diff line number	Diff line change
`@@ -566,7 +566,7 @@ DBConnector::DBConnector(const string& dbName, unsigned int timeout, bool isTcpC`
`566`	`566`	`DBConnector::DBConnector(const string& dbName, unsigned int timeout, bool isTcpConn)`
`567`	`567`	`: DBConnector(dbName, timeout, isTcpConn, EMPTY_NAMESPACE)`
`568`	`568`	`{`
`569`		`- // Empty contructor`
	`569`	`+ // Empty constructor`
`570`	`570`	`}`
`571`	`571`
`572`	`572`	`int DBConnector::getDbId() const`
Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@ void NetDispatcher::registerMessageHandler(int nlmsg_type, NetMsg *callback)`
`18`	`18`	`MUTEX;`
`19`	`19`
`20`	`20`	`if (m_handlers.find(nlmsg_type) != m_handlers.end())`
`21`		`- throw "Trying to register on already registerd netlink message";`
	`21`	`+ throw "Trying to register on already registered netlink message";`
`22`	`22`
`23`	`23`	`m_handlers[nlmsg_type] = callback;`
`24`	`24`	`}`
Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@ namespace swss`
`13`	`13`	`{`
`14`	`14`	`public:`
`15`	`15`
`16`		`- /** Get singlton instance. */`
	`16`	`+ /** Get singleton instance. */`
`17`	`17`	`static NetDispatcher& getInstance();`
`18`	`18`
`19`	`19`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -90,7 +90,7 @@ uint64_t NetLink::readData()`
`90`	`90`	`if (err < 0)`
`91`	`91`	`{`
`92`	`92`	`if (err == -NLE_NOMEM)`
`93`		`- SWSS_LOG_ERROR("netlink reports out of memory on reading a netlink socket. High possiblity of a lost message");`
	`93`	`+ SWSS_LOG_ERROR("netlink reports out of memory on reading a netlink socket. High possibility of a lost message");`
`94`	`94`	`else if (err == -NLE_AGAIN)`
`95`	`95`	`SWSS_LOG_DEBUG("netlink reports NLE_AGAIN on reading a netlink socket");`
`96`	`96`	`else`
Original file line number	Diff line number	Diff line change
`@@ -134,7 +134,7 @@ uint64_t NfNetlink::readData()`
`134`	`134`	`if (err < 0)`
`135`	`135`	`{`
`136`	`136`	`if (err == -NLE_NOMEM)`
`137`		`- SWSS_LOG_ERROR("netlink reports out of memory on reading a netfilter netlink socket. High possiblity of a lost message");`
	`137`	`+ SWSS_LOG_ERROR("netlink reports out of memory on reading a netfilter netlink socket. High possibility of a lost message");`
`138`	`138`	`else if (err == -NLE_AGAIN)`
`139`	`139`	`SWSS_LOG_DEBUG("netlink reports NLE_AGAIN on reading a netfilter netlink socket");`
`140`	`140`	`else`