On 30-Mar-2001 Baruch Even wrote:
> Attached is a patch to fix a couple of memory leaks, I'll also attach
> the report of LeakTracer, it includes various Mathed leaks and two
> non-mathed leaks.
>
> In the fix I provide there is a "fix" for the leak in the idex, this is
> not really a fix as it is a way to hide the problem from LeakTracer.
>
> +// Wouldn't it be better to do { clear(); delete par; } ? (baruch)
Well yes but so we don't have the overhead of allocating and deleting a par.
I don't like patches which change intentation! (especially if the whole
other end of the file has indentation to 4 spaces!)
So I don't see any enhancement in this, I won't apply this part of
the patch (InsetText)! The other one I let decide Lars!
Jürgen
--
-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._
Dr. Jürgen Vigna E-Mail: [EMAIL PROTECTED]
Italienallee 13/N Tel/Fax: +39-0471-450260 / +39-0471-450253
I-39100 Bozen Web: http://www.sad.it/~jug
-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._
"Neighbors!! We got neighbors! We ain't supposed to have any neighbors, and
I just had to shoot one."
-- Post Bros. Comics
