On 29/6/2010 12:04 PM, Kostka Bořivoj wrote:
> My cycle starts at this->postingsFreeCountDW, not at 0
Sorry, I misread you. I thought you just replaced the line within the
loop. So yes, it seems to be the same, except with my solution you don't
have to search for more copy/delete occurrences in the code, and it only
happens once - so its faster (not only the use of memset).
I thought about doing this as follows, but will only do that once I know
cl_demo runs clean:
memset(this->postingsFreeListDW.values + this->postingsFreeCountDW
, NULL
, sizeof(Posting*));
postingsFreeListDW.deleteUntilNULL();
I also think we better move postingsFreeListDW to the heap, since it is
quite large and we wouldn't want to possibly jam the writer. I'll need
to review the code there to make sure it can be done without fragmenting
the memory.
I have had a quick look on the memory leaks, none of them seem to be
related to the indexing process.
Itamar.
------------------------------------------------------------------------------
This SF.net email is sponsored by Sprint
What will you do first with EVO, the first 4G phone?
Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first
_______________________________________________
CLucene-developers mailing list
CLucene-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/clucene-developers
