Skip to content

Commit

Permalink
Fixed regression of issue #644
Browse files Browse the repository at this point in the history
  • Loading branch information
rays committed Jul 16, 2017
1 parent 4b6f0b9 commit f4f66f8
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions lstm/lstmtrainer.h
Expand Up @@ -301,6 +301,7 @@ class LSTMTrainer : public LSTMRecognizer {
// language model. trainer is the model that is restored.
bool ReadTrainingDump(const GenericVector<char>& data,
LSTMTrainer* trainer) const {
if (data.empty()) return false;

This comment has been minimized.

Copy link
@stweil

stweil Jul 16, 2017

Contributor

That was my initial fix, too (see comment). The working fix was applied in commit 34d1e73. It was reverted by commit dc8745e.

return ReadSizedTrainingDump(&data[0], data.size(), trainer);
}
bool ReadSizedTrainingDump(const char* data, int size,
Expand Down

0 comments on commit f4f66f8

Please sign in to comment.