Re: [fpc-devel] memleak in fpdoc

2010-11-07 Thread Marco van de Voort
In our previous episode, Michael Van Canneyt said: > > That line was added by me yesterday in response to the same mail. > > Ah. Well, since you applied the patch, I didn't bother to look at > the original code. > > Did you remove the .addref or was it not necessary ? I didn't. I don't know for

Re: [fpc-devel] memleak in fpdoc

2010-11-07 Thread Michael Van Canneyt
On Sun, 7 Nov 2010, Marco van de Voort wrote: In our previous episode, Michael Van Canneyt said: if not assigned(CreateAliasType(alname,clname,cls,cls2)) then The return value of CreateAliasType doesn't seem to be freed/released. I don't know how to fix that. It should be freed, because i

Re: [fpc-devel] memleak in fpdoc

2010-11-07 Thread Marco van de Voort
In our previous episode, Michael Van Canneyt said: > > if not assigned(CreateAliasType(alname,clname,cls,cls2)) then > > The return value of CreateAliasType doesn't seem to be freed/released. > > I don't know how to fix that. > > It should be freed, because it is added to the list of declaration

Re: [fpc-devel] memleak in fpdoc

2010-11-07 Thread Michael Van Canneyt
On Sat, 6 Nov 2010, Vincent Snijders wrote: 2010/11/5 Michael Van Canneyt : On Fri, 5 Nov 2010, Vincent Snijders wrote: Hi, I am trying to fix the memleaks in fpdoc. Attached is a patch for pascal parser package. There are still more leaks, but that is for another time. I debugged som

Re: [fpc-devel] memleak in fpdoc

2010-11-06 Thread Vincent Snijders
2010/11/5 Michael Van Canneyt : > > > On Fri, 5 Nov 2010, Vincent Snijders wrote: > >> Hi, >> >> I am trying to fix the memleaks in fpdoc. Attached is a patch for >> pascal parser package. >> >> There are still more leaks, but that is for another time. I debugged some more. In procedure ProcessInh