Skip to content

Commit

Permalink
Merge pull request #50 from tetsuok/remove-unused-vars
Browse files Browse the repository at this point in the history
Remove unused variables
  • Loading branch information
taku910 committed Mar 31, 2018
2 parents ea41f40 + 4256fc3 commit 1ff5904
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 6 deletions.
1 change: 0 additions & 1 deletion src/unigram_model.cc
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@
namespace sentencepiece {
namespace unigram {
namespace {
constexpr size_t kNodeChunkSize = 512;

// Returns log(exp(x) + exp(y)).
// if init_mode is true, returns log(exp(y)) == y.
Expand Down
6 changes: 1 addition & 5 deletions src/unigram_model_trainer.cc
Original file line number Diff line number Diff line change
Expand Up @@ -35,10 +35,6 @@ namespace sentencepiece {
namespace unigram {
namespace {

constexpr char32 kWSChar = L'\u2581';
constexpr char32 kUNKChar = L'\u2585';
constexpr char kUNKStr[] = "\xe2\x96\x85";

double Digamma(double x) {
double result = 0.0;
for (; x < 7; ++x) result -= 1 / x;
Expand Down Expand Up @@ -113,7 +109,7 @@ TrainerModel::SentencePieces Trainer::MakeSeedSentencePieces() const {

// Merges all sentences into one array with 0x0000 delimiter.
std::vector<char32> array;
std::unordered_map<std::string, int64> all_chars, substrs;
std::unordered_map<std::string, int64> all_chars;
constexpr char32 kSentenceBoundary = 0x0000;

const size_t mining_size =
Expand Down

0 comments on commit 1ff5904

Please sign in to comment.