]> Kevux Git Server - fll/commitdiff
Cleanup: Don't bother with != for existence modulus checks.
authorKevin Day <kevin@kevux.org>
Tue, 4 Jul 2023 03:05:03 +0000 (22:05 -0500)
committerKevin Day <kevin@kevux.org>
Tue, 4 Jul 2023 03:05:03 +0000 (22:05 -0500)
The 'X % 2' and 'X % 3' will be non-zero with they are not divisible by the given unit.
The '!= 0' check is not necessary.

level_3/fss_write/c/main/common.c
level_3/iki_read/c/main/common.c

index ee59d680929f26bc1b6a23fe7330ef108461a881..dfcce31cbc5b8f88645207922b350ab14bac2a28 100644 (file)
@@ -360,7 +360,7 @@ extern "C" {
     }
 
     if ((main->program.parameters.array[fss_write_parameter_ignore_e].result & f_console_result_value_e) && main->program.parameters.array[fss_write_parameter_ignore_e].values.used) {
-      if (main->program.parameters.array[fss_write_parameter_ignore_e].values.used % 2 != 0) {
+      if (main->program.parameters.array[fss_write_parameter_ignore_e].values.used % 2) {
         main->setting.state.status = F_status_set_error(F_parameter);
 
         if ((main->setting.flag & fss_write_main_flag_print_first_e) && main->program.message.verbosity > f_console_verbosity_error_e) {
index 3dfe8e4a05134744390dd6a12665c0b49eba7e1d..29cbadea9364c91d05d8e874ac346c468726637b 100644 (file)
@@ -259,7 +259,7 @@ extern "C" {
     }
 
     if (main->program.parameters.array[iki_read_parameter_replace_e].result != f_console_result_none_e) {
-      if ((main->program.parameters.array[iki_read_parameter_replace_e].result & f_console_result_found_e) || main->program.parameters.array[iki_read_parameter_replace_e].values.used % 2 != 0) {
+      if ((main->program.parameters.array[iki_read_parameter_replace_e].result & f_console_result_found_e) || main->program.parameters.array[iki_read_parameter_replace_e].values.used % 2) {
         main->setting.state.status = F_status_set_error(F_parameter);
 
         if ((main->setting.flag & iki_read_main_flag_print_first_e) && main->program.message.verbosity > f_console_verbosity_error_e) {
@@ -354,7 +354,7 @@ extern "C" {
     if (F_status_is_error(main->setting.state.status)) return;
 
     if (main->program.parameters.array[iki_read_parameter_wrap_e].result != f_console_result_none_e) {
-      if ((main->program.parameters.array[iki_read_parameter_wrap_e].result & f_console_result_found_e) || main->program.parameters.array[iki_read_parameter_wrap_e].values.used % 3 != 0) {
+      if ((main->program.parameters.array[iki_read_parameter_wrap_e].result & f_console_result_found_e) || main->program.parameters.array[iki_read_parameter_wrap_e].values.used % 3) {
         main->setting.state.status = F_status_set_error(F_parameter);
 
         if ((main->setting.flag & iki_read_main_flag_print_first_e) && main->program.message.verbosity > f_console_verbosity_error_e) {
@@ -605,7 +605,7 @@ extern "C" {
 
     if (!(parameter.result & f_console_result_found_e)) return F_false;
 
-    if ((parameter.result & f_console_result_found_e) || parameter.values.used % 3 != 0) {
+    if ((parameter.result & f_console_result_found_e) || parameter.values.used % 3) {
       main->setting.state.status = F_status_set_error(F_parameter);
 
       if ((main->setting.flag & iki_read_main_flag_print_first_e) && main->program.message.verbosity > f_console_verbosity_error_e) {