Simplify and clean code related to class TRand

Signed-off-by: Stefan Weil <[email protected]>
tesseract-ocr · Nov 25, 2024 · c1bb752 · c1bb752 · egorpugin · Nov 26, 2024
1 parent 9f17a3f
commit c1bb752
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 9 deletions.
diff --git a/src/ccstruct/imagedata.cpp b/src/ccstruct/imagedata.cpp
@@ -526,7 +526,8 @@ void DocumentData::Shuffle() {
   TRand random;
   // Different documents get shuffled differently, but the same for the same
   // name.
-  random.set_seed(document_name_.c_str());
+  std::hash<std::string> hasher;
+  random.set_seed(static_cast<uint64_t>(hasher(document_name_)));
   int num_pages = pages_.size();
   // Execute one random swap for each page in the document.
   for (int i = 0; i < num_pages; ++i) {

diff --git a/src/ccutil/helpers.h b/src/ccutil/helpers.h
@@ -24,9 +24,7 @@
 #include <climits> // for INT_MIN, INT_MAX
 #include <cmath> // std::isfinite
 #include <cstdio>
-#include <cstring>
 #include <algorithm>  // for std::find
-#include <functional>
 #include <string>
 #include <vector>
 
@@ -77,11 +75,6 @@ class TRand {
   void set_seed(uint64_t seed) {
     seed_ = seed;
   }
-  // Sets the seed using a hash of a string.
-  void set_seed(const std::string &str) {
-    std::hash<std::string> hasher;
-    set_seed(static_cast<uint64_t>(hasher(str)));
-  }
 
   // Returns an integer in the range 0 to INT32_MAX.
   int32_t IntRand() {

diff --git a/src/lstm/lstmrecognizer.h b/src/lstm/lstmrecognizer.h
@@ -286,7 +286,7 @@ class TESS_API LSTMRecognizer {
 protected:
   // Sets the random seed from the sample_iteration_;
   void SetRandomSeed() {
-    int64_t seed = static_cast<int64_t>(sample_iteration_) * 0x10000001;
+    int64_t seed = sample_iteration_ * 0x10000001LL;
     randomizer_.set_seed(seed);
     randomizer_.IntRand();
   }