Merge pull request #1631 from shubham-shahh/ros2

rectified spelling and gramatical errors

mavros/mavros/param.py

@@ -224,7 +224,7 @@ def values(self) -> 'ParamDict':
         #    we'll get bunch of events
         self.call_pull()
 
-        # 3. if to little events come, request whole list
+        # 3. if too small events come, request whole list
         if len(pm) < 10:
             names = call_list_parameters(node=self._node,
                                          client=self.cli_list_parameters)
@@ -241,7 +241,7 @@ class ParamDict(dict):
     ParamDict wrapper.
 
     That class holds states of parameters
-    and allow to upload new items.
+    and allows to upload new items.
     """
 
     class NoSet:

mavros/src/plugins/command.cpp

@@ -254,7 +254,7 @@ class CommandPlugin : public plugin::Plugin
     bool & success)
   {
     /* Note: seems that COMMAND_INT don't produce COMMAND_ACK
-     * so wait don't needed.
+     * so wait is not needed.
      */
     command_int(
       broadcast,

mavros/src/plugins/ftp.cpp

@@ -55,7 +55,7 @@ using utils::enum_value;
 /**
  * @brief FTP Request message abstraction class
  *
- * @note This class not portable, and works on little-endian machines only.
+ * @note This class is not portable, and works on little-endian machines only.
  */
 class FTPRequest : public mavlink::common::msg::FILE_TRANSFER_PROTOCOL
 {
@@ -173,8 +173,8 @@ class FTPRequest : public mavlink::common::msg::FILE_TRANSFER_PROTOCOL
    * @brief Copy string to payload
    *
    * @param[in] s  payload string
-   * @note this function allow null termination inside string
-   *       it used to send multiple strings in one message
+   * @note this function allows null termination inside string
+   *       it is used to send multiple strings in one message
    */
   void set_data_string(const std::string & s)
   {
@@ -204,7 +204,7 @@ class FTPRequest : public mavlink::common::msg::FILE_TRANSFER_PROTOCOL
 #endif
 
     // return uas->get_system_id() == target_system;
-    return true;  // XXX TODO(vooon): probably whole method not needed anymore
+    return true;  // XXX TODO(vooon): probably whole method is not needed anymore
   }
 
   /**

mavros/src/plugins/mission_protocol_base.cpp

@@ -346,7 +346,7 @@ void MissionBase::handle_mission_current(
 {
   unique_lock lock(mutex);
 
-  // NOTE(vooon): this message do not have mission_type
+  // NOTE(vooon): this message does not have mission_type
   // if (filter_message(mcur)) {
   //   return;
   // }
@@ -377,7 +377,7 @@ void MissionBase::handle_mission_item_reached(
   mavlink::common::msg::MISSION_ITEM_REACHED & mitr,
   MFilter filter [[maybe_unused]])
 {
-  // NOTE(vooon): this message do not have mission_type
+  // NOTE(vooon): this message does not have mission_type
   // if (filter_message(mitr)) {
   //   return;
   // }

mavros/src/plugins/param.cpp

@@ -70,7 +70,7 @@ static constexpr const char * events = "/parameter_events";
  * - real32 for float's
  *
  * So no reason to really use boost::any.
- * But feel free to fire an issue if your AP do not like it.
+ * But feel free to fire an issue if your AP does not like it.
  */
 class Parameter
 {
@@ -352,7 +352,7 @@ class Parameter
   }
 
   /**
-   * Exclude this parameters from ~param/push
+   * Exclude these parameters from ~param/push
    */
   static bool check_exclude_param_id(const std::string & param_id)
   {

mavros/src/plugins/setpoint_raw.cpp

@@ -277,7 +277,7 @@ class SetpointRawPlugin : public plugin::Plugin,
     double thrust;
 
     // Set Thrust scaling in px4_config.yaml, setpoint_raw block.
-    // ignore thrust is false by default, unless no thrust scalling is set or thrust is zero
+    // ignore thrust is false by default, unless no thrust scaling is set or thrust is zero
     auto ignore_thrust = req->thrust != 0.0 &&
       !node->get_parameter("thrust_scaling", thrust_scaling);
 

mavros/test/test_uas.cpp

@@ -213,7 +213,7 @@ TEST_F(TestUAS, add_plugin__route_message__filter)
   auto plugin2 = std::make_shared<MockPlugin>(uas);
   auto subs2 = plugin2->rawsubs();
 
-  // XXX(vooon): silence leak warnings: they works badly with shared_ptr
+  // XXX(vooon): silence leak warnings: they work badly with shared_ptr
   testing::Mock::AllowLeak(&(*uas));
   testing::Mock::AllowLeak(&(*plugin1));
   testing::Mock::AllowLeak(&(*plugin2));