Re: [Geany-devel] Geany memory behavior suspected memory leak - ID: 3415254 - g_slice

2011-10-14 Thread Nick Treleaven
On 14/10/2011 04:09, Lex Trotman wrote: Hi All, I have been investigating bug report suspected memory leak - ID: 3415254 http://sourceforge.net/tracker/?func=detailatid=787791aid=3415254group_id=153444 1. each time all files are closed only 200k is returned to the OS, but since the first

Re: [Geany-devel] GTK+ Version Bump to 2.18 - gtkbuilder and 2.16

2011-10-14 Thread Nick Treleaven
On 13/10/2011 12:53, Nick Treleaven wrote: On 13/10/2011 00:55, Matthew Brush wrote: So I would go for 2.16 overall if this brings us Glade 3 support. It should, but I haven't thoroughly tested it with GTK+ 2.16 yet. Did you try the gtkbuilder branch yet on your 2.16 install by any chance?

[Geany-devel] per-project prefs - Re: Remove extra whitespace at end of lines in all source files?

2011-10-14 Thread Nick Treleaven
On 13/10/2011 22:24, Matthew Brush wrote: On 10/13/2011 04:42 AM, Nick Treleaven wrote: The stripping feature in Geany should be made a project preference, I don't want to cause noise on non-Geany files. This is actually on my TODO list, unless anyone feels like doing it before me :)

Re: [Geany-devel] GTK+ Version Bump to 2.18 - gtkbuilder and 2.16 - on_term_font_set

2011-10-14 Thread Nick Treleaven
On 14/10/2011 15:38, Nick Treleaven wrote: I did get a warning: Gtk WARNING: Could not find signal handler 'on_term_font_set' This is because on_term_font_set is not defined unless VTE support is compiled in (on Windows it's disabled). ___

Re: [Geany-devel] Geany memory behavior suspected memory leak - ID: 3415254

2011-10-14 Thread Colomban Wendling
Le 14/10/2011 05:09, Lex Trotman a écrit : Hi All, Hey, I ran Valgrind quite a lot on Geany lately -- though I never ran it in production because it is slow --, and thus will comment with what I know and have seen. I have been investigating bug report suspected memory leak - ID: 3415254

Re: [Geany-devel] Geany memory behavior suspected memory leak - ID: 3415254 - g_slice

2011-10-14 Thread Colomban Wendling
Le 14/10/2011 16:24, Nick Treleaven a écrit : [...] I don't know if converting our other uses of g_new for fixed size structures is worth it, but I would say it is good practice to use g_slice in all new code. I think if we allocate a struct a significant amount of times it could be a good

Re: [Geany-devel] Geany memory behavior suspected memory leak - ID: 3415254 - g_slice

2011-10-14 Thread Lex Trotman
[...] Since Geany uses three different allocators, each of which has differing policies for holding onto memory, it is going to be difficult to separate real leaks from allocator effects. Interesting. I'd like to continue the discussion from the report about allocator usage: Lex: I think

Re: [Geany-devel] Geany memory behavior suspected memory leak - ID: 3415254

2011-10-14 Thread Lex Trotman
[...] Geany at least keeps all the GeanyDocuments alive and tries to re-use them (document_create() at line 564).  This avoids having to re-allocate the document struct everytime, though it'll then never release this memory.  However, either the user won't open so much files at once or she's

Re: [Geany-devel] GTK+ Version Bump to 2.18 - gtkbuilder and 2.16

2011-10-14 Thread Matthew Brush
On 11-10-14 07:38 AM, Nick Treleaven wrote: On 13/10/2011 12:53, Nick Treleaven wrote: On 13/10/2011 00:55, Matthew Brush wrote: So I would go for 2.16 overall if this brings us Glade 3 support. It should, but I haven't thoroughly tested it with GTK+ 2.16 yet. Did you try the gtkbuilder

Re: [Geany-devel] GTK+ Version Bump to 2.18 - gtkbuilder and 2.16

2011-10-14 Thread Matthew Brush
On 11-10-14 08:18 PM, Matthew Brush wrote: There's seems to be a problem applying, saving and restoring the background and foreground colours for the VTE, I couldn't figure it out last time I looked. Not sure if you can have a peek at this, but I guess you'll need to boot into Linux to test.