stanfordnlp / GloVe

Software in C and data files for the popular GloVe model for distributed word representations, a.k.a. word vectors or embeddings
Apache License 2.0
6.86k stars 1.51k forks source link

Memory leak issue #158

Closed MhYao2014 closed 4 years ago

MhYao2014 commented 4 years ago

It appears that in the vocab_count.c, the authors malloced some memory to store the word read from the corpus when they build the vocabulary, but they didn't free these memories in the end. I think this will lead to a memory leak issue right? Can anyone tell me that if I am right, and how to fix this issue?

AngledLuffa commented 4 years ago

Should now be fixed.