|
From: Itamar Syn-H. <it...@co...> - 2010-09-20 15:29:08
|
On 20/9/2010 9:44 AM, Šplíchal Jiří wrote: > Something must have changed in the code during the merges because I am sure > that at least the testExtractTerms test didn't produce memory leaks. > > Let us get rid of the leaks! I will start from the bottom > with the testExtractTerms test. Should we create a new branch for this issue? > Yes - one that is tracking master. Use that same branch as long as you're working on the same issue, and when you start working on anything else that is self contained start a new branch tracking master. Its easy to see what has changed between the branches - and I suspect one of the conflict resolutions during merges was incorrect. Don't forget to remove your old branches when they are not needed anymore! Itamar. |