Replace call of exit function by return statement in main function

Add also a missing return statement and use EXIT_FAILURE
and EXIT_SUCCESS instead of 1 and 0 as return values.

Signed-off-by: Stefan Weil <[email protected]>

Author: stweil

src/training/ambiguous_words.cpp

@@ -33,13 +33,13 @@ int main(int argc, char **argv) {
   // Parse input arguments.
   if (argc > 1 && (!strcmp(argv[1], "-v") || !strcmp(argv[1], "--version"))) {
     printf("%s\n", tesseract::TessBaseAPI::Version());
-    return 0;
+    return EXIT_SUCCESS;
   } else if (argc != 4 && (argc != 6 || strcmp(argv[1], "-l") != 0)) {
     printf(
         "Usage: %s -v | --version | %s [-l lang] tessdata_dir wordlist_file"
         " output_ambiguous_wordlist_file\n",
         argv[0], argv[0]);
-    return 1;
+    return EXIT_FAILURE;
   }
   int argv_offset = 0;
   std::string lang;
@@ -65,7 +65,7 @@ int main(int argc, char **argv) {
   FILE *input_file = fopen(input_file_str, "rb");
   if (input_file == nullptr) {
     tesseract::tprintf("Failed to open input wordlist file %s\n", input_file_str);
-    exit(1);
+    return EXIT_FAILURE;
   }
   char str[CHARS_PER_LINE];
 
@@ -78,4 +78,5 @@ int main(int argc, char **argv) {
   }
   // Clean up.
   fclose(input_file);
+  return EXIT_SUCCESS;
 }

src/training/classifier_tester.cpp

@@ -109,7 +109,7 @@ int main(int argc, char **argv) {
       InitializeClassifier(FLAGS_classifier.c_str(), trainer->unicharset(), argc, argv, &api);
   if (shape_classifier == nullptr) {
     fprintf(stderr, "Classifier init failed!:%s\n", FLAGS_classifier.c_str());
-    return 1;
+    return EXIT_FAILURE;
   }
 
   // We want to test junk as well if it is available.
@@ -123,5 +123,5 @@ int main(int argc, char **argv) {
   delete shape_classifier;
   delete api;
 
-  return 0;
+  return EXIT_SUCCESS;
 } /* main */

src/training/cntraining.cpp

@@ -141,7 +141,7 @@ int main(int argc, char *argv[]) {
     Clusterer = SetUpForClustering(FeatureDefs, CharSample, PROGRAM_FEATURE_TYPE);
     if (Clusterer == nullptr) { // To avoid a SIGSEGV
       fprintf(stderr, "Error: nullptr clusterer!\n");
-      return 1;
+      return EXIT_FAILURE;
     }
     float SavedMinSamples = Config.MinSamples;
     // To disable the tendency to produce a single cluster for all fonts,
@@ -173,7 +173,7 @@ int main(int argc, char *argv[]) {
     FreeProtoList(&freeable_proto);
   }
   printf("\n");
-  return 0;
+  return EXIT_SUCCESS;
 } // main
 
 /*----------------------------------------------------------------------------

src/training/combine_lang_model.cpp

@@ -58,7 +58,7 @@ int main(int argc, char **argv) {
   UNICHARSET unicharset;
   if (!unicharset.load_from_file(FLAGS_input_unicharset.c_str(), false)) {
     tprintf("Failed to load unicharset from %s\n", FLAGS_input_unicharset.c_str());
-    return 1;
+    return EXIT_FAILURE;
   }
   tprintf("Loaded unicharset of size %zu from file %s\n", unicharset.size(),
           FLAGS_input_unicharset.c_str());

src/training/combine_tessdata.cpp

@@ -269,7 +269,7 @@ int main(int argc, char **argv) {
         "Usage for compacting LSTM component to int:\n"
         "  %s -c traineddata_file\n",
         argv[0]);
-    return 1;
+    return EXIT_FAILURE;
   }
   tm.Directory();
   return EXIT_SUCCESS;

src/training/dawg2wordlist.cpp

@@ -60,7 +60,7 @@ static int WriteDawgAsWordlist(const UNICHARSET &unicharset, const tesseract::Da
   FILE *out = fopen(outfile_name, "wb");
   if (out == nullptr) {
     tprintf("Could not open %s for writing.\n", outfile_name);
-    return 1;
+    return EXIT_FAILURE;
   }
   WordOutputter outputter(out);
   using namespace std::placeholders; // for _1
@@ -80,20 +80,20 @@ int main(int argc, char *argv[]) {
         "Usage: %s -v | --version | %s <unicharset> <dawgfile> "
         "<wordlistfile>\n",
         argv[0], argv[0]);
-    return 1;
+    return EXIT_FAILURE;
   }
   const char *unicharset_file = argv[1];
   const char *dawg_file = argv[2];
   const char *wordlist_file = argv[3];
   UNICHARSET unicharset;
   if (!unicharset.load_from_file(unicharset_file)) {
     tprintf("Error loading unicharset from %s.\n", unicharset_file);
-    return 1;
+    return EXIT_FAILURE;
   }
   auto dict = LoadSquishedDawg(unicharset, dawg_file);
   if (dict == nullptr) {
     tprintf("Error loading dictionary from %s.\n", dawg_file);
-    return 1;
+    return EXIT_FAILURE;
   }
   int retval = WriteDawgAsWordlist(unicharset, dict.get(), wordlist_file);
   return retval;

src/training/lstmeval.cpp

@@ -34,38 +34,38 @@ int main(int argc, char **argv) {
   ParseArguments(&argc, &argv);
   if (FLAGS_model.empty()) {
     tprintf("Must provide a --model!\n");
-    return 1;
+    return EXIT_FAILURE;
   }
   if (FLAGS_eval_listfile.empty()) {
     tprintf("Must provide a --eval_listfile!\n");
-    return 1;
+    return EXIT_FAILURE;
   }
   tesseract::TessdataManager mgr;
   if (!mgr.Init(FLAGS_model.c_str())) {
     if (FLAGS_traineddata.empty()) {
       tprintf("Must supply --traineddata to eval a training checkpoint!\n");
-      return 1;
+      return EXIT_FAILURE;
     }
     tprintf("%s is not a recognition model, trying training checkpoint...\n", FLAGS_model.c_str());
     if (!mgr.Init(FLAGS_traineddata.c_str())) {
       tprintf("Failed to load language model from %s!\n", FLAGS_traineddata.c_str());
-      return 1;
+      return EXIT_FAILURE;
     }
     std::vector<char> model_data;
     if (!tesseract::LoadDataFromFile(FLAGS_model.c_str(), &model_data)) {
       tprintf("Failed to load model from: %s\n", FLAGS_model.c_str());
-      return 1;
+      return EXIT_FAILURE;
     }
     mgr.OverwriteEntry(tesseract::TESSDATA_LSTM, &model_data[0], model_data.size());
   }
   tesseract::LSTMTester tester(static_cast<int64_t>(FLAGS_max_image_MB) * 1048576);
   if (!tester.LoadAllEvalData(FLAGS_eval_listfile.c_str())) {
     tprintf("Failed to load eval data from: %s\n", FLAGS_eval_listfile.c_str());
-    return 1;
+    return EXIT_FAILURE;
   }
   double errs = 0.0;
   std::string result = tester.RunEvalSync(0, &errs, mgr,
                                           /*training_stage (irrelevant)*/ 0, FLAGS_verbosity);
   tprintf("%s\n", result.c_str());
-  return 0;
+  return EXIT_SUCCESS;
 } /* main */

src/training/merge_unicharsets.cpp

@@ -24,14 +24,14 @@ int main(int argc, char **argv) {
 
   if (argc > 1 && (!strcmp(argv[1], "-v") || !strcmp(argv[1], "--version"))) {
     printf("%s\n", tesseract::TessBaseAPI::Version());
-    return 0;
+    return EXIT_SUCCESS;
   } else if (argc < 4) {
     // Print usage
     printf(
         "Usage: %s -v | --version |\n"
         "       %s unicharset-in-1 ... unicharset-in-n unicharset-out\n",
         argv[0], argv[0]);
-    return 1;
+    return EXIT_FAILURE;
   }
 
   tesseract::UNICHARSET input_unicharset, result_unicharset;
@@ -42,7 +42,7 @@ int main(int argc, char **argv) {
       result_unicharset.AppendOtherUnicharset(input_unicharset);
     } else {
       printf("Failed to load unicharset from file %s!!\n", argv[arg]);
-      exit(1);
+      return EXIT_FAILURE;
     }
   }
 
@@ -51,7 +51,7 @@ int main(int argc, char **argv) {
     printf("Wrote unicharset file %s.\n", argv[argc - 1]);
   } else {
     printf("Cannot save unicharset file %s.\n", argv[argc - 1]);
-    exit(1);
+    return EXIT_FAILURE;
   }
-  return 0;
+  return EXIT_SUCCESS;
 }

src/training/mftraining.cpp

@@ -201,7 +201,7 @@ int main(int argc, char **argv) {
   // Load the training data.
   auto trainer = tesseract::LoadTrainingData(argv + 1, false, &shape_table, file_prefix);
   if (trainer == nullptr) {
-    return 1; // Failed.
+    return EXIT_FAILURE; // Failed.
   }
 
   // Setup an index mapping from the shapes in the shape table to the classes
@@ -269,5 +269,5 @@ int main(int argc, char **argv) {
       ;
     }
   }
-  return 0;
+  return EXIT_SUCCESS;
 } /* main */

src/training/set_unicharset_properties.cpp

@@ -29,14 +29,14 @@ int main(int argc, char **argv) {
   // Check validity of input flags.
   if (FLAGS_U.empty() || FLAGS_O.empty()) {
     tprintf("Specify both input and output unicharsets!\n");
-    exit(1);
+    return EXIT_FAILURE;
   }
   if (FLAGS_script_dir.empty()) {
     tprintf("Must specify a script_dir!\n");
-    exit(1);
+    return EXIT_FAILURE;
   }
 
   tesseract::SetPropertiesForInputFile(FLAGS_script_dir.c_str(), FLAGS_U.c_str(), FLAGS_O.c_str(),
                                        FLAGS_X.c_str());
-  return 0;
+  return EXIT_SUCCESS;
 }