Fix various typos throughout the codebase (#1969)

README.md

@@ -256,7 +256,7 @@ You can base a new driver from an existing driver. Look in either the examples o
 
 # Unit tests
 
-In order to run the unit test suite you must first install the [Google Test Framework](https://github.com/google/googletest). You will need to build and install this from source code as Google does not recommend package managers for distributing distros.(This is because each build system is often unique and a one size fits all aproach does not work well).
+In order to run the unit test suite you must first install the [Google Test Framework](https://github.com/google/googletest). You will need to build and install this from source code as Google does not recommend package managers for distributing distros.(This is because each build system is often unique and a one size fits all approach does not work well).
 
 Once you have the Google Test Framework installed follow this alternative build sequence:-
 

drivers/README.md

@@ -110,7 +110,7 @@ sudo apt-get update && sudo apt-get install indi-DRIVER_NAME
 </ul>
 <p class="alert alert-info">The <b>Slew Rate</b> dropdown is used to control the <b>manual</b> speeds when using the NSWE controls either directly or via a joystick. To set the <b>GOTO</b> speeds (when mount moves from one target to another via a GOTO command), you need to update the <b>Custom Speeds</b> control.</p>
 <h3>Site Management</h3>
-<p>Time, Locaiton, and Park settings are configured in the Site Management tab.</p>
+<p>Time, Location, and Park settings are configured in the Site Management tab.</p>
 <p><img style="display: block; margin-right: auto; margin-left: auto;" src="/images/devices/DRIVER_NAME/site_management.jpg" alt="Site Management" /></p>
 <ul>
 	<li><strong>UTC</strong>: UTC time and offsets must be set for proper operation of the driver upon connection. The UTC offset is in hours. East is positive and west is negative.</li>

drivers/auxiliary/STAR2kdriver.c

@@ -106,7 +106,7 @@ int ConnectSTAR2k(char *port)
 }
 
 /* Start a slew in chosen direction at slewRate */
-/* Use auxilliary NexStar command set through the hand control computer */
+/* Use auxiliary NexStar command set through the hand control computer */
 
 void StartPulse(int direction)
 {

drivers/auxiliary/astrometrydriver.cpp

@@ -297,7 +297,7 @@ bool AstrometryDriver::processBLOB(uint8_t *data, uint32_t size, uint32_t len)
 
         if (destLen != size)
         {
-            LOGF_WARN("Discrepency between uncompressed data size %ld and expected size %ld",
+            LOGF_WARN("Discrepancy between uncompressed data size %ld and expected size %ld",
                       size, destLen);
         }
 

drivers/auxiliary/astrometrydriver.h

@@ -39,7 +39,7 @@
  * The solver settings should be set before running the solver in order to ensure correct and timely response from astrometry.net
  * It is assumed that astrometry.net is property set-up in the same machine the driver is running along with the appropriate index files.
  *
- * If the solver is successfull, the driver sets the solver results which include:
+ * If the solver is successful, the driver sets the solver results which include:
  * + Pixel Scale (arcsec/pixel).
  * + Orientation (E or W) degrees.
  * + Center RA (J2000) Hours.

drivers/auxiliary/mydcp4esp32.cpp

@@ -305,7 +305,7 @@ bool MyDCP4ESP::sendCommand(const char *cmd, char *resp)
 
 // Determine which of the 4 channels have temperature probes attached. Only those with probes can be active
 // except for Channel 3 which can mirrior Channels 1 & 2 or be controlled manually.
-// Check to see if each channel can be set to Overide if it doesn't currently have a power output
+// Check to see if each channel can be set to Override if it doesn't currently have a power output
 bool MyDCP4ESP::getActiveChannels()
 {
     char cmd[MDCP_CMD_LENGTH] = {};
@@ -519,7 +519,7 @@ bool MyDCP4ESP::setAmbientOffset(float value)
     return sendCommand(cmd, nullptr); 
 }
 
-// set or reset Channel overide. Channel = 5 resets all channels
+// set or reset Channel override. Channel = 5 resets all channels
 bool MyDCP4ESP::setChannelBoost( unsigned int channel, unsigned int value)
 {
     char cmd[MDCP_CMD_LENGTH] = {};
@@ -839,7 +839,7 @@ bool MyDCP4ESP::readSettings()
 
     if ((ok == 1) && (ch3_mode <= 4))
     {
-        // Enabel/Disable Ch3 Manual Power setting if Ch3 Mode Manual enabled
+        // Enable/Disable Ch3 Manual Power setting if Ch3 Mode Manual enabled
         if ((ch3_mode == CH3MODE_MANUAL) && (!ch3ManualPower))
         {
             defineProperty(Ch3ManualPowerNP);

drivers/auxiliary/mydcp4esp32.h

@@ -85,7 +85,7 @@
 #define MDCP_GET_ALL_CH_POWER_CMD           ":40#"    // Get the power settings for ch1/ch2/ch3/ch4 - Response: lch1pwr,ch2pwr,ch3pwr,ch4pwr#
 #define MDCP_SET_CH3_MODE_CMD               ":41%d#"  // Set the Channel 3 mode (0=disabled, 1=dewstrap1, 2=dewstrap2, 3=Manual, 4=use temp probe3) - Response: none
 #define MDCP_GET_CH3_MODE_CMD               ":42#"    // Get the Channel 3 mode (0=disabled, 1=dewstrap1, 2=dewstrap2, 3=Manual, 4=use temp probe3) - Response: mmode#
-#define MDCP_SET_CH3_MANUAL_POWER_CMD       ":43%d#"  // Set the Channel 3 power to a manual seting of Num - Response: none    
+#define MDCP_SET_CH3_MANUAL_POWER_CMD       ":43%d#"  // Set the Channel 3 power to a manual setting of Num - Response: none    
 
 /**************************** myDCP4ESP32 Command Responses **************************/
 
@@ -128,7 +128,7 @@
 #define MDCP_GET_ALL_CH_POWER_RES           "l%d,%d,%d,%d"    // Get the power settings for ch1/ch2/ch3/ch4 - Response: lch1pwr,ch2pwr,ch3pwr,ch4pwr#
 #define MDCP_SET_CH3_MODE_RES               ""      // Set the Channel 3 mode (0=disabled, 1=dewstrap1, 2=dewstrap2, 3=Manual, 4=use temp probe3) - Response: none
 #define MDCP_GET_CH3_MODE_RES               "m%d"   // Get the Channel 3 mode (0=disabled, 1=dewstrap1, 2=dewstrap2, 3=Manual, 4=use temp probe3) - Response: mmode#
-#define MDCP_SET_CH3_MANUAL_POWER_RES       ""      // Set the Channel 3 power to a manual seting of Num - Response: none 
+#define MDCP_SET_CH3_MANUAL_POWER_RES       ""      // Set the Channel 3 power to a manual setting of Num - Response: none 
 
 /******************************************************************************/
 

drivers/auxiliary/pegasus_upb.cpp

@@ -98,7 +98,7 @@ bool PegasusUPB::initProperties()
     /// Power Group
     ////////////////////////////////////////////////////////////////////////////
 
-    // Dew Labels. Need to delare them here to use in the Power usage section
+    // Dew Labels. Need to declare them here to use in the Power usage section
     IUFillText(&DewControlsLabelsT[0], "DEW_LABEL_1", "Dew A", "Dew A");
     IUFillText(&DewControlsLabelsT[1], "DEW_LABEL_2", "Dew B", "Dew B");
     IUFillText(&DewControlsLabelsT[2], "DEW_LABEL_3", "Dew C", "Dew C");

drivers/auxiliary/planewave_delta.h

@@ -116,7 +116,7 @@ class DeltaT : public INDI::DefaultDevice
         /// Properties
         ///////////////////////////////////////////////////////////////////////////////////
 
-        // Delta-T Informatin
+        // Delta-T Information
         ITextVectorProperty InfoTP;
         IText InfoT[1] {};
         enum

drivers/auxiliary/skysafariclient.cpp

@@ -187,7 +187,7 @@ bool SkySafariClient::setSlewRate(int slewRate)
 
     int finalSlewRate = slewRate;
 
-    // If slew rate is betwee min and max, we intepolate
+    // If slew rate is between min and max, we interpolate
     if (slewRate > 0 && slewRate < maxSlewRate)
         finalSlewRate = static_cast<int>(ceil(slewRate * maxSlewRate / 3.0));
 

drivers/auxiliary/sqm.h

@@ -103,6 +103,6 @@ class SQM : public INDI::DefaultDevice
         static const char DRIVER_STOP_CHAR { 0x0A };
         // Wait up to a maximum of 3 seconds for serial input
         static constexpr const uint8_t DRIVER_TIMEOUT {3};
-        // Maximum buffer for sending/receving.
+        // Maximum buffer for sending/receiving.
         static constexpr const uint8_t DRIVER_LEN {128};
 };

drivers/auxiliary/wanderer_cover.cpp

@@ -124,7 +124,7 @@ bool WandererCover::Handshake()
 {
     if (isSimulation())
     {
-        LOGF_INFO("Connected successfuly to simulated %s. Retrieving startup data...", getDeviceName());
+        LOGF_INFO("Connected successfully to simulated %s. Retrieving startup data...", getDeviceName());
 
         // SetTimer(getCurrentPollingPeriod());
         IUSaveText(&FirmwareT[0], "Simulation version");
@@ -716,7 +716,7 @@ void WandererCover::displayConfigurationMessage()
     LOG_WARN(" - Click on 'Set current position as close' to define the park position");
     LOG_WARN(" - Use again the select list to move your cover panel in close position on the scope");
     LOG_WARN(" - Click on 'Set current position as open' to define the unpark position");
-    LOG_WARN(" - Use the select controler to move your panel to the open position");
+    LOG_WARN(" - Use the select controller to move your panel to the open position");
     LOG_WARN("In order to do so, go to 'Dust cover configurtation' tab and do the following steps :");
     LOG_WARN("Before first use, or when you change your setup, you need to configure Open and closed position first in 'Dust cover configurtation' tab.");
 }

drivers/auxiliary/watchdog.cpp

@@ -606,7 +606,7 @@ void WatchDog::TimerHit()
                 break;
             }
 
-            // Watch mount if requied
+            // Watch mount if required
             if (ShutdownProcedureS[PARK_MOUNT].s == ISS_ON)
                 m_WatchDogClientInstance->setMount(ActiveDeviceT[ACTIVE_TELESCOPE].text);
             // Watch dome

drivers/ccd/ccd_simulator.cpp

@@ -411,7 +411,7 @@ void CCDSim::TimerHit()
             float timeleft;
             timeleft = CalcTimeLeft(ExpStart, ExposureRequest);
 
-            //IDLog("CCD Exposure left: %g - Requset: %g\n", timeleft, ExposureRequest);
+            //IDLog("CCD Exposure left: %g - Request: %g\n", timeleft, ExposureRequest);
             if (timeleft < 0)
                 timeleft = 0;
 
@@ -724,7 +724,7 @@ int CCDSim::DrawCcdFrame(INDI::CCDChip * targetChip)
 
                 while (fgets(line, 256, pp) != nullptr)
                 {
-                    //  ok, lets parse this line for specifcs we want
+                    //  ok, lets parse this line for specifics we want
                     char id[20];
                     char plate[6];
                     char ob[6];

drivers/ccd/guide_simulator.cpp

@@ -333,7 +333,7 @@ void GuideSim::TimerHit()
             float timeleft;
             timeleft = CalcTimeLeft(ExpStart, ExposureRequest);
 
-            //IDLog("CCD Exposure left: %g - Requset: %g\n", timeleft, ExposureRequest);
+            //IDLog("CCD Exposure left: %g - Request: %g\n", timeleft, ExposureRequest);
             if (timeleft < 0)
                 timeleft = 0;
 

drivers/dome/domepro2.h

@@ -119,7 +119,7 @@ class DomePro2 : public INDI::Dome
                  * after the command is successfully sent.
                  * @param cmd_len if -1, it is assumed that the @a cmd is a null-terminated string. Otherwise, it would write @a cmd_len bytes from
                  * the @a cmd buffer.
-                 * @param res_len if -1 and if @a res is not nullptr, the function will read until it detects the default delimeter DRIVER_STOP_CHAR
+                 * @param res_len if -1 and if @a res is not nullptr, the function will read until it detects the default delimiter DRIVER_STOP_CHAR
                  *  up to DRIVER_LEN length. Otherwise, the function will read @a res_len from the device and store it in @a res.
                  * @return True if successful, false otherwise.
         */
@@ -174,7 +174,7 @@ class DomePro2 : public INDI::Dome
         static const char DRIVER_STOP_CHAR { 0x3B };
         // Wait up to a maximum of 3 seconds for serial input
         static constexpr const uint8_t DRIVER_TIMEOUT {3};
-        // Maximum buffer for sending/receving.
+        // Maximum buffer for sending/receiving.
         static constexpr const uint8_t DRIVER_LEN {64};
         // Dome AZ Threshold
         static constexpr const double DOME_AZ_THRESHOLD {0.01};
@@ -225,7 +225,7 @@ class DomePro2 : public INDI::Dome
             {"Parking", "Parking"},
             {"Gauging", "Gauging"},
             {"Timeout", "Azimuth movement timeout"},
-            {"Stall", "Azimuth encoder registering insufficent counts… motor stalled"},
+            {"Stall", "Azimuth encoder registering insufficient counts… motor stalled"},
             {"OCP", "Over Current Protection was tripped"}
         };
 

drivers/dome/nexdome_beaver.h

@@ -200,7 +200,7 @@ class Beaver : public INDI::Dome
         static const char DRIVER_STOP_CHAR { 0x23 };
         // Wait up to a maximum of 3 seconds for serial input
         static constexpr const uint8_t DRIVER_TIMEOUT {3};
-        // Maximum buffer for sending/receving.
+        // Maximum buffer for sending/receiving.
         static constexpr const uint8_t DRIVER_LEN {128};
         int domeDir = 1;
         double lastAzDiff = 1;

drivers/dome/rigel_dome.h

@@ -134,6 +134,6 @@ class RigelDome : public INDI::Dome
         static const char DRIVER_STOP_CHAR { 0x0D };
         // Wait up to a maximum of 3 seconds for serial input
         static constexpr const uint8_t DRIVER_TIMEOUT {3};
-        // Maximum buffer for sending/receving.
+        // Maximum buffer for sending/receiving.
         static constexpr const uint8_t DRIVER_LEN {64};
 };

drivers/dome/scopedome_dome.h

@@ -256,7 +256,7 @@ class ScopeDome : public INDI::Dome
 
         INDI::PropertyText CredentialsTP{2};
 
-        // Dynamic properies initialized based on card type
+        // Dynamic properties initialized based on card type
         INDI::PropertySwitch RelaysSP{0};
         INDI::PropertyNumber SensorsNP{0};
         INDI::PropertySwitch InputsSP{0};

drivers/filter_wheel/ifwoptec.cpp

@@ -418,7 +418,7 @@ bool FilterIFW::ISNewSwitch(const char *dev, const char *name, ISState *states,
 ************************************************************************************/
 void FilterIFW::simulationTriggered(bool enable)
 {
-    // toogle buttons to select 5 or 8 filters depend if Simulation active or not
+    // toggle buttons to select 5 or 8 filters depend if Simulation active or not
     if (enable)
     {
         if (isConnected())
@@ -738,7 +738,7 @@ bool FilterIFW::SetFilterNames()
 
     LOG_INFO("Filters name are saved in IFW");
 
-    // Interface not ready before the message "DATA OK" disapear from the display IFW
+    // Interface not ready before the message "DATA OK" disappear from the display IFW
     for (int i = OPTEC_WAIT_DATA_OK; i > 0; i--)
     {
         LOGF_INFO("Please wait for HOME command start... %d", i);
@@ -943,7 +943,7 @@ bool FilterIFW::GetFirmware()
         return false;
     }
 
-    // remove chars fomr the string to get only the nzuméric value of the Firmware version
+    // remove chars from the string to get only the nzuméric value of the Firmware version
     char *p = nullptr;
 
     for (int i = 0; i < (int)strlen(response); i++)

drivers/filter_wheel/ifwoptec.h

@@ -119,7 +119,7 @@ class FilterIFW : public INDI::FilterWheel
         ISwitchVectorProperty CharSetSP;
         ISwitch CharSetS[2];
 
-        // Firmware of teh IFW
+        // Firmware of the IFW
         ITextVectorProperty FirmwareTP;
         IText FirmwareT[1] {};
 

drivers/filter_wheel/pegasus_indigo.h

@@ -58,7 +58,7 @@ class PegasusINDIGO : public INDI::FilterWheel
                  * after the command is successfully sent.
                  * @param cmd_len if -1, it is assumed that the @a cmd is a null-terminated string. Otherwise, it would write @a cmd_len bytes from
                  * the @a cmd buffer.
-                 * @param res_len if -1 and if @a res is not nullptr, the function will read until it detects the default delimeter DRIVER_STOP_CHAR
+                 * @param res_len if -1 and if @a res is not nullptr, the function will read until it detects the default delimiter DRIVER_STOP_CHAR
                  *  up to DRIVER_LEN length. Otherwise, the function will read @a res_len from the device and store it in @a res.
                  * @return True if successful, false otherwise.
         */

drivers/filter_wheel/xagyl_wheel.h

@@ -148,6 +148,6 @@ class XAGYLWheel : public INDI::FilterWheel
         // Some commands optionally return an extra string.
         static constexpr const int OPTIONAL_TIMEOUT {1};
 
-        // Maximum buffer for sending/receving.
+        // Maximum buffer for sending/receiving.
         static constexpr const int DRIVER_LEN {64};
 };

drivers/focuser/aaf2.h

@@ -69,8 +69,8 @@ class AAF2 : public INDI::Focuser
         bool Ack();
         /**
          * @brief sendCommand Send a string command to AAF2.
-         * @param cmd Command to be sent, must already have the necessary delimeter ('#')
-         * @param res If not nullptr, the function will read until it detects the default delimeter ('#') up to ML_RES length.
+         * @param cmd Command to be sent, must already have the necessary delimiter ('#')
+         * @param res If not nullptr, the function will read until it detects the default delimiter ('#') up to ML_RES length.
          *        if nullptr, no read back is done and the function returns true.
          * @return True if successful, false otherwise.
          */
@@ -93,8 +93,8 @@ class AAF2 : public INDI::Focuser
 
         // AAF2 Buffer
         static const uint8_t DRIVER_RES { 32 };
-        // AAF2 Delimeter
+        // AAF2 Delimiter
         static const char DRIVER_DEL { '#' };
-        // AAF2 Tiemout
+        // AAF2 Timeout
         static const uint8_t DRIVER_TIMEOUT { 3 };
 };

drivers/focuser/celestron.cpp

@@ -93,7 +93,7 @@ bool CelestronSCT::initProperties()
     FocusRelPosN[0].value = 0;
     FocusRelPosN[0].step  = 1000;
 
-    // Absolute Postition Range
+    // Absolute Position Range
     FocusAbsPosN[0].min   = 0.;
     FocusAbsPosN[0].max   = 60000.;
     FocusAbsPosN[0].value = 0;
@@ -116,7 +116,7 @@ bool CelestronSCT::initProperties()
 
     communicator.setDeviceName(getDeviceName());
 
-    // Defualt port to /dev/ttyACM0
+    // Default port to /dev/ttyACM0
     //serialConnection->setDefaultPort("/dev/ttyACM0");
 
     //LOG_INFO("initProperties end");

drivers/focuser/celestron.h

@@ -124,8 +124,8 @@ class CelestronSCT : public INDI::Focuser
         /////////////////////////////////////////////////////////////////////////////
         //        // Celestron Buffer
         //        static const uint8_t CELESTRON_LEN { 32 };
-        //        // Celestorn Delimeter
+        //        // Celestorn Delimiter
         //        static const char CELESTRON_DEL { '#' };
-        //        // Celestron Tiemout in seconds
+        //        // Celestron Timeout in seconds
         //        static const uint8_t CELESTRON_TIMEOUT { 3 };
 };

drivers/focuser/focuslynx.cpp

@@ -120,7 +120,7 @@ const char *FocusLynxF1::getDefaultName()
  *
 * ***********************************************************************************/
 bool FocusLynxF1::Connect()
-/* Overide of connect() function
+/* Override of connect() function
  * different for F1 or F2 focuser
  * F1 connect only himself to the driver and
  * it is the only one who's connect to the communication port to establish the physical communication
@@ -304,7 +304,7 @@ bool FocusLynxF1::getHubConfig()
         IUSaveText(&HubT[0], text);
         IDSetText(&HubTP, nullptr);
 
-        //Save localy the Version of the firmware's Hub
+        //Save locally the Version of the firmware's Hub
         strncpy(version, text, sizeof(version));
 
         LOGF_DEBUG("Text =  %s,  Key = %s", text, key);
@@ -550,7 +550,7 @@ bool FocusLynxF1::getHubConfig()
     memset(response, 0, sizeof(response));
     memset(text, 0, sizeof(text));
 
-    // WIFI IP adress
+    // WIFI IP address
     if (isSimulation())
     {
         strncpy(response, "WF IP = 192.168.1.11\n", 32);
@@ -691,7 +691,7 @@ bool FocusLynxF1::getHubConfig()
     {
         response[nbytes_read - 1] = '\0';
 
-        // Display the response to be sure to have read the complet TTY Buffer.
+        // Display the response to be sure to have read the complete TTY Buffer.
         LOGF_DEBUG("RES <%s>", response);
 
         if (strcmp(response, "END"))
@@ -791,7 +791,7 @@ const char *FocusLynxF2::getDefaultName()
  *
 * ***********************************************************************************/
 bool FocusLynxF2::Connect()
-/* Overide of connect() function
+/* Override of connect() function
  * different for F2 or F1 focuser
  * F2 don't connect himself to the hub
  */
@@ -846,7 +846,7 @@ bool FocusLynxF2::RemoteDisconnect()
         updateProperties();
     }
 
-    // When called by F1, the PortFD should be -1; For debbug purpose
+    // When called by F1, the PortFD should be -1; For debug purposes
     PortFD = lynxDriveF1->getPortFD();
     LOGF_INFO("Remote disconnection: %s is offline.", getDeviceName());
     LOGF_INFO("Value of F2 PortFD = %d", PortFD);