Re: Alias expansion patch

[Berend De Schouwer <bds jhb ucs co za>]

On Fri, 21 Jul 2000 20:18:15 Peter Williams wrote:
> Thanks for your patch! I'll look it over and try to commit it soon.
> 
> > Here is a much improved alias expansion patch, featuring:
> > For developers:
> > * Spent the last three days hunting a memory leak.  Turns out the leak
> >   also exists in a clean balsa 0.9.0
> 
> Any information on said leak?

start balsa.  Look at the memory usage with ps.
Start compose window, check memory usage: bigger (expected).
close window (don't type anything),  and it doesn't free the RAM.

Every compose window seems to waste approx. 64K when the window is closed.

Per example: clean Balsa-0.9.0, ./configure --enable-all
Start balsa; ps aux | grep balsa
ziggy    20220  7.3  6.8  6912 4312 pts/1    S    09:56   0:00 ./balsa
Start compose; ps aux | grep balsa
ziggy    20220  3.5  7.3  7068 4600 pts/1    S    09:56   0:00 ./balsa
kill compose; ps aux | grep balsa
ziggy    20220  2.5  7.3  7068 4608 pts/1    S    09:56   0:00 ./balsa
Start compose; ps aux | grep balsa
ziggy    20220  2.5  7.4  7128 4664 pts/1    S    09:56   0:01 ./balsa
Kill compose; ps aux | grep balsa
ziggy    20220  2.5  7.4  7128 4664 pts/1    S    09:56   0:01 ./balsa

... after 10 repeats ...
ziggy    20220  2.4  8.1  7592 5128 pts/1    S    09:56   0:03 ./balsa


> > * The code shows that C is not my native language.
> > * Is there a g_strlen()?  I couldn't find one.  Everything else is
> >   gchar, gcompletion, glist, g_strconcat, etc.
> 
> No, gchar is just a char, I never really got why they do it, but
> all the g_ functions work that way. There are some functions eg
> g_strdup that are for portability, but functions that everyone has
> such as strlen aren't reproduced.

Okay.

> My apologies if this comes out as HTML mail...

It didn't.

> --
> --
> Peter Williams -- peterw@helixcode.com

-- 
Kind regards,				  
Berend                                  
                                        
-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
Berend De Schouwer, +27-11-712-1435, UCS