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 <[email protected]>
stweil · Nov 28, 2016 · 9dab030 · 9dab030
1 parent 037a7ff
commit 9dab030
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
@@ -149,7 +149,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) {
   // Call before InitNetwork.
   ASSERT_HOST(network_ == NULL);
   EmptyConstructor();

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.