Re: [Kaldi-developers] Possible memory leak in gmm-latgen-biglm-faster

[Daniel P. <dp...@gm...>]

OK, I found the bug and fixed it.
Dan



On Mon, Jul 7, 2014 at 3:14 PM, Augusto Henrique Hentz <
au...@li...> wrote:

> Hi Dan, it is the code in trunk, latest revision (4143).
>
> On Jul 7, 2014, at 16:01 , Daniel Povey <dp...@gm...> wrote:
>
> > Are you using the code in trunk?  If so, what revision?
> > Dan
> >
> >
> >
> > On Mon, Jul 7, 2014 at 2:19 PM, Augusto Henrique Hentz <
> au...@li...> wrote:
> > Hello all,
> >
> > When using the gmm-latgen-biglm-faster, I get the following warning:
> >
> > WARNING (~HashList():util/hash-list-inl.h:115) Possible memory leak:
> 1815 != 2048
> >
> > for every utterance. I checked the LatticeFasterDecoder class, and this
> is the part that clears the HashList
> >
> > void LatticeFasterDecoder::DeleteElems(Elem *list) {
> >   for (Elem *e = list, *e_tail; e != NULL; e = e_tail) {
> >     // Token::TokenDelete(e->val);
> >     e_tail = e->tail;
> >     toks_.Delete(e);
> >   }
> > }
> >
> > this method is called as
> >
> >   DeleteElems(toks_.Clear());
> >
> > The elements are deleted from the HashList (toks_.Delete(e)), so there
> should be no leak. Also, any ideas why the `Token::TokenDelete(e->val)`
> statement is commented out?
> >
> > Thanks
> >
> > --
> > Augusto Henrique Hentz
> > au...@li...
> >
> >
> >
> >
> >
> >
> ------------------------------------------------------------------------------
> > Open source business process management suite built on Java and Eclipse
> > Turn processes into business applications with Bonita BPM Community
> Edition
> > Quickly connect people, data, and systems into organized workflows
> > Winner of BOSSIE, CODIE, OW2 and Gartner awards
> > http://p.sf.net/sfu/Bonitasoft
> > _______________________________________________
> > Kaldi-developers mailing list
> > Kal...@li...
> > https://lists.sourceforge.net/lists/listinfo/kaldi-developers
> >
>
> --
> Augusto Henrique Hentz
> au...@li...
>
>
>
>
>