lstm: Pass big parameter by reference (performance)

Coverity report: CID 1366448 (#1 of 1): Big parameter passed by value (PASS_BY_VALUE) pass_by_value: Passing parameter recoder of type tesseract::UnicharCompress const (size 240 bytes) by value. Signed-off-by: Stefan Weil <sw@weilnetz.de>
tesseract-ocr · Dec 1, 2016 · f3e8895 · f3e8895
1 parent bb6cfc1
commit f3e8895
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lstm/lstmtrainer.cpp b/lstm/lstmtrainer.cpp
@@ -147,7 +147,7 @@ void LSTMTrainer::InitCharSet(const UNICHARSET& unicharset,
 // ctc_mode controls how the truth text is mapped to the network targets.
 // Note: Call before InitNetwork!
 void LSTMTrainer::InitCharSet(const UNICHARSET& unicharset,
-                              const UnicharCompress recoder) {
+                              const UnicharCompress& recoder) {
   EmptyConstructor();
   int flags = TF_COMPRESS_UNICHARSET;
   training_flags_ = static_cast<TrainingFlags>(flags);

diff --git a/lstm/lstmtrainer.h b/lstm/lstmtrainer.h
@@ -113,7 +113,7 @@ class LSTMTrainer : public LSTMRecognizer {
   // previously setup UNICHARSET and UnicharCompress.
   // ctc_mode controls how the truth text is mapped to the network targets.
   // Note: Call before InitNetwork!
-  void InitCharSet(const UNICHARSET& unicharset, const UnicharCompress recoder);
+  void InitCharSet(const UNICHARSET& unicharset, const UnicharCompress& recoder);
 
   // Initializes the trainer with a network_spec in the network description
   // net_flags control network behavior according to the NetworkFlags enum.