Bug#34006439: Spell check issues in code comments

Description:
-----------
Running "codespell" tool on the source code repository gives large number
(5k) of spell check issues.

The focus of this bug is to fix only those related to code comments (fixing
the spelling within the generated documentation in the process).
Due to high volume of issues, currently these subprojects are ignored (3k out
of 5k total issues): ndb, innodb, rapid, router, group_replication, innodb_memcached.
Code variable names, output string spell check issues were ignored to avoid
the impact on programs or tests.

Approved by: Stefan Hinz <stefan.hinz@oracle.com>
Approved by: Jon Stephens <jon.stephens@oracle.com>

Change-Id: Ife855a31bd71f7d56d5d613753730c0c2132e241

Author: MRajcic

client/base/abstract_options_provider.h

@@ -1,5 +1,5 @@
 /*
-   Copyright (c) 2014, 2021, Oracle and/or its affiliates.
+   Copyright (c) 2014, 2022, Oracle and/or its affiliates.
 
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License, version 2.0,
@@ -61,7 +61,7 @@ class Abstract_options_provider : public I_options_provider {
    */
   Simple_option *create_new_option(std::string name, std::string description);
   /**
-    Creates and attach new disabled option. This option is to mark existance
+    Creates and attach new disabled option. This option is to mark existence
     of options inavailable due to distribution configuration.
     @param name Name of option. It is used in command-line option name as
       --name.

client/base/disabled_option.h

@@ -1,5 +1,5 @@
 /*
-   Copyright (c) 2014, 2021, Oracle and/or its affiliates.
+   Copyright (c) 2014, 2022, Oracle and/or its affiliates.
 
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License, version 2.0,
@@ -34,7 +34,7 @@ namespace Base {
 namespace Options {
 
 /**
-  Disabled option. This option is used to mark existance of options
+  Disabled option. This option is used to mark existence of options
   inavailable due to distribution configuration.
  */
 class Disabled_option : public Abstract_option<Disabled_option> {

client/dump/mysql_crawler.cc

@@ -1,5 +1,5 @@
 /*
-  Copyright (c) 2015, 2021, Oracle and/or its affiliates.
+  Copyright (c) 2015, 2022, Oracle and/or its affiliates.
 
   This program is free software; you can redistribute it and/or modify
   it under the terms of the GNU General Public License, version 2.0,
@@ -113,7 +113,7 @@ void Mysql_crawler::enumerate_objects() {
 
     std::optional<std::string> stmt = this->get_create_statement(
         runner, "", db_name, "DATABASE IF NOT EXISTS");
-    if (!stmt.has_value()) continue;  // some error ocurred
+    if (!stmt.has_value()) continue;  // some error occurred
 
     Database *database =
         new Database(this->generate_new_object_id(), db_name, stmt.value());
@@ -275,7 +275,7 @@ void Mysql_crawler::enumerate_tables(const Database &db) {
     bool isInnoDB = table_data[1] == "InnoDB";         // "Engine"
     std::optional<std::string> stmt =
         this->get_create_statement(runner, db.get_name(), table_name, "TABLE");
-    if (!stmt.has_value()) continue;  // some error ocurred
+    if (!stmt.has_value()) continue;  // some error occurred
     Table *table = new Table(this->generate_new_object_id(), table_name,
                              db.get_name(), stmt.value(), fields, table_data[1],
                              rows, (uint64)(rows * (isInnoDB ? 1.5 : 1)),
@@ -390,7 +390,7 @@ void Mysql_crawler::enumerate_functions(const Database &db, std::string type) {
 
     std::optional<std::string> stmt = this->get_create_statement(
         runner, db.get_name(), function_row[1], type, 2);
-    if (!stmt.has_value())  // some error ocurred
+    if (!stmt.has_value())  // some error occurred
       break;
 
     TObject *function = new TObject(
@@ -431,7 +431,7 @@ void Mysql_crawler::enumerate_event_scheduler_events(const Database &db) {
     std::optional<std::string> stmt = this->get_create_statement(
         runner, db.get_name(), event_row[1], "EVENT", 3);
 
-    if (!stmt.has_value())  // some error ocurred
+    if (!stmt.has_value())  // some error occurred
       break;
 
     Event_scheduler_event *event = new Event_scheduler_event(
@@ -523,7 +523,7 @@ void Mysql_crawler::enumerate_table_triggers(const Table &table,
     const Mysql::Tools::Base::Mysql_query_runner::Row &trigger_row = **it;
     std::optional<std::string> stmt = this->get_create_statement(
         runner, table.get_schema(), trigger_row[0], "TRIGGER", 2);
-    if (!stmt.has_value())  // some error occured
+    if (!stmt.has_value())  // some error occurred
       break;
     Trigger *trigger = new Trigger(
         this->generate_new_object_id(), trigger_row[0], table.get_schema(),

client/migrate_keyring/options.h

@@ -1,5 +1,5 @@
 /*
-   Copyright (c) 2021, Oracle and/or its affiliates.
+   Copyright (c) 2021, 2022, Oracle and/or its affiliates.
 
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License, version 2.0,
@@ -51,7 +51,7 @@ class Options {
   static char *s_destination_keyring_configuration_dir;
 
   /*
-    Following paramters are needed if migration involves an active MySQL server
+    Following parameters are needed if migration involves an active MySQL server
   */
 
   /** Flag for online migration */

client/mysql.cc

@@ -1452,8 +1452,8 @@ int main(int argc, char *argv[]) {
 void mysql_end(int sig) {
 #ifndef _WIN32
   /*
-    Ingnoring SIGQUIT, SIGINT and SIGHUP signals when cleanup process starts.
-    This will help in resolving the double free issues, which occures in case
+    Ignoring SIGQUIT, SIGINT and SIGHUP signals when cleanup process starts.
+    This will help in resolving the double free issues, which occurs in case
     the signal handler function is started in between the clean up function.
   */
   signal(SIGQUIT, SIG_IGN);
@@ -4350,7 +4350,7 @@ static int com_use(String *buffer [[maybe_unused]], char *line) {
 static int normalize_dbname(const char *line, char *buff, uint buff_size) {
   MYSQL_RES *res = nullptr;
 
-  /* Send the "USE db" commmand to the server. */
+  /* Send the "USE db" command to the server. */
   if (mysql_query(&mysql, line)) return 1;
 
   /*
@@ -4638,7 +4638,7 @@ static int sql_real_connect(char *host, char *database, char *user, char *,
     /*
       Don't convert trailing '\n' character - it was appended during
       last batch_readline_command() call.
-      Oherwise we'll get an extra line, which makes some tests fail.
+      Otherwise we'll get an extra line, which makes some tests fail.
     */
     if (status.line_buff->buffer[len - 1] == '\n') len--;
     if (tmp.copy(status.line_buff->buffer, len, &my_charset_utf8mb4_bin,
@@ -4813,7 +4813,7 @@ static int com_status(String *buffer [[maybe_unused]],
   tee_fprintf(stdout, "\nConnection id:\t\t%lu\n", mysql_thread_id(&mysql));
   /*
     Don't remove "limit 1",
-    it is protection againts SQL_SELECT_LIMIT=0
+    it is protection against SQL_SELECT_LIMIT=0
   */
   if (!mysql_store_result_for_lazy(&result)) {
     MYSQL_ROW cur = mysql_fetch_row(result);

client/mysql_secure_installation.cc

@@ -1,5 +1,5 @@
 /*
-   Copyright (c) 2013, 2021, Oracle and/or its affiliates.
+   Copyright (c) 2013, 2022, Oracle and/or its affiliates.
 
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License, version 2.0,
@@ -350,7 +350,7 @@ static int install_password_validation_component() {
       int tmp = sizeof("SET GLOBAL validate_password.policy = ") + 3;
       size_t strength_length = strlen(strength);
       /*
-        query string needs memory which is atleast the length of initial part
+        query string needs memory which is at least the length of initial part
         of query plus twice the size of variable being appended.
       */
       query = (char *)my_malloc(PSI_NOT_INSTRUMENTED,
@@ -384,7 +384,7 @@ static void estimate_password_strength(char *password_string) {
   size_t tmp = sizeof("SELECT validate_password_strength(") + 3;
   size_t password_length = strlen(password_string);
   /*
-    query string needs memory which is atleast the length of initial part
+    query string needs memory which is at least the length of initial part
     of query plus twice the size of variable being appended.
   */
   query = (char *)my_malloc(PSI_NOT_INSTRUMENTED,
@@ -517,7 +517,7 @@ static void set_opt_user_password(int component_set) {
       char *query = nullptr, *end;
       int tmp = sizeof("SET PASSWORD=") + 3;
       /*
-        query string needs memory which is atleast the length of initial part
+        query string needs memory which is at least the length of initial part
         of query plus twice the size of variable being appended.
       */
       query = (char *)my_malloc(PSI_NOT_INSTRUMENTED,
@@ -643,7 +643,7 @@ static void drop_users(MYSQL_RES *result) {
     user_length = strlen(user_tmp);
     host_length = strlen(host_tmp);
     /*
-      query string needs memory which is atleast the length of initial part
+      query string needs memory which is at least the length of initial part
       of query plus twice the size of variable being appended.
     */
     query = (char *)my_malloc(

client/mysql_ssl_rsa_setup.cc

@@ -1,5 +1,5 @@
 /*
-   Copyright (c) 2015, 2021, Oracle and/or its affiliates.
+   Copyright (c) 2015, 2022, Oracle and/or its affiliates.
 
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License, version 2.0,
@@ -98,7 +98,7 @@ Sql_string_t ext_files[] = {create_string("cav3.ext"),
 #define MAX_PATH_LEN \
   (FN_REFLEN - strlen(FN_DIRSEP) - cert_files[SERVER_CERT].length() - 1)
 /*
-  Higest number of fixed characters in subject line is 47:
+  Highest number of fixed characters in subject line is 47:
   MySQL_SERVER_<suffix>_Auto_Generated_Server_Certificate
   Maximum size of subject is 64. So suffix can't be longer
   than 17 characters.

client/mysqladmin.cc

@@ -484,11 +484,11 @@ int main(int argc, char *argv[]) {
     }
   } else {
     /*
-      --count=0 aborts right here. Otherwise iff --sleep=t ("interval")
+      --count=0 aborts right here. Otherwise if --sleep=t ("interval")
       is given a t!=0, we get an endless loop, or n iterations if --count=n
       was given an n!=0. If --sleep wasn't given, we get one iteration.
 
-      To wit, --wait loops the connection-attempts, while --sleep loops
+      To wait, --wait loops the connection-attempts, while --sleep loops
       the command execution (endlessly if no --count is given).
     */
 
@@ -532,7 +532,7 @@ int main(int argc, char *argv[]) {
             */
           } else {
             /*
-              connexion broke, and we have no order to re-establish it. fail.
+              connection broke, and we have no order to re-establish it. fail.
             */
             if (!option_force) error = 1;
             break;

client/mysqlbinlog.cc

@@ -313,7 +313,7 @@ class Database_rewrite {
     database name that is to be rewritten into the target one.
 
     The key of the map is the "from" database name. The value of the
-    map is is the "to" database name that we are rewritting the
+    map is is the "to" database name that we are rewriting the
     name into.
    */
   std::map<std::string, std::string> m_dict;
@@ -564,7 +564,7 @@ class Database_rewrite {
   /**
     Shall unregister a rewrite rule for a given database. If the name is
     not registered, then no action is taken and no error reported.
-    The name of database to be used in this invokation is the original
+    The name of database to be used in this invocation is the original
     database name.
 
     @param from the original database name used when the rewrite rule
@@ -667,10 +667,10 @@ char server_version[SERVER_VERSION_LENGTH];
 ulong filter_server_id = 0;
 
 /*
-  This strucure is used to store the event and the log postion of the events
-  which is later used to print the event details from correct log postions.
+  This structure is used to store the event and the log position of the events
+  which is later used to print the event details from correct log positions.
   The Log_event *event is used to store the pointer to the current event and
-  the event_pos is used to store the current event log postion.
+  the event_pos is used to store the current event log position.
 */
 struct buff_event_info {
   Log_event *event;
@@ -682,7 +682,7 @@ struct buff_event_info {
   User_var_log_events, and Rand_log_events, followed by one
   Query_log_event. If statements are filtered out, the filter has to be
   checked for the Query_log_event. So we have to buffer the Intvar,
-  User_var, and Rand events and their corresponding log postions until we see
+  User_var, and Rand events and their corresponding log positions until we see
   the Query_log_event. This dynamic array buff_ev is used to buffer a structure
   which stores such an event and the corresponding log position.
 */
@@ -2622,7 +2622,7 @@ static Exit_status dump_remote_log_entries(PRINT_EVENT_INFO *print_event_info,
   if ((retval = check_master_version()) != OK_CONTINUE) return retval;
 
   /*
-    Fake a server ID to log continously. This will show as a
+    Fake a server ID to log continuously. This will show as a
     slave on the mysql server.
   */
   if (to_last_remote_log && stop_never) {
@@ -3468,7 +3468,7 @@ void Transaction_payload_log_event::print(FILE *,
     process_event(info, ev, header()->log_pos, "", true);
 
     // lets make the buffer be allocated again, as the current
-    // buffer ownership has been handed over to the defferred event
+    // buffer ownership has been handed over to the deferred event
     if (is_deferred_event) {
       buffer = nullptr;        /* purecov: inspected */
       current_buffer_size = 0; /* purecov: inspected */

client/mysqlimport.cc

@@ -409,7 +409,7 @@ static int lock_table(MYSQL *mysql, int tablecount, char **raw_tablename) {
     dynstr_append(&query, " WRITE,");
   }
   if (mysql_real_query(mysql, query.str, (ulong)(query.length - 1)))
-    return db_error(mysql); /* We shall countinue here, if --force was given */
+    return db_error(mysql); /* We shall continue here, if --force was given */
   return 0;
 }
 
@@ -544,7 +544,7 @@ static char *field_escape(char *to, const char *from, uint length) {
       end_backslashes ^= 1; /* find odd number of backslashes */
     else {
       if (*from == '\'' && !end_backslashes)
-        *to++ = *from; /* We want a dublicate of "'" for MySQL */
+        *to++ = *from; /* We want a duplicate of "'" for MySQL */
       end_backslashes = 0;
     }
   }
@@ -571,7 +571,7 @@ static void *worker_thread(void *arg) {
   if (mysql_query(mysql, "/*!40101 set @@character_set_database=binary */;") &&
       (error = db_error(mysql))) {
     if (exitcode == 0) exitcode = error;
-    /* We shall countinue here, if --force was given */
+    /* We shall continue here, if --force was given */
     goto error;
   }
 
@@ -691,7 +691,7 @@ int main(int argc, char **argv) {
                     "/*!40101 set @@character_set_database=binary */;") &&
         (error = db_error(mysql))) {
       if (exitcode == 0) exitcode = error;
-      /* We shall countinue here, if --force was given */
+      /* We shall continue here, if --force was given */
       goto end;
     }