Le 04/03/2013 16:33, Colomban Wendling a écrit :
Le 04/03/2013 14:43, Frank Lanitz a écrit :
Am 04.03.2013 10:23, schrieb Lex Trotman:
On 4 March 2013 19:41, Frank Lanitz frank@frank.uvena.de wrote:
Am 19.02.2013 22:22, schrieb Lex Trotman:
Additional patch to treebrowser.c make sure every path leading to a g_free has a g_strdup(), oops.
Note I am only using the HACKING approved "-Wall -Wextra -O2 -Wno-unused-parameter" not the full War and Peace novel of options that Colomban uses :)
Did somebody apply the patches? I went lost inside the many channels of communication ;)
Hi Frank,
I don't see them in any of the G-P commits since 17 Feb when I posted.
So I'd say "no".
OK. so what to do? Just merging it as none of the maintainer seems to have shown any reaction.
I'll check it and merge it later totady.
Done. It was mostly OK but introduced a leak, so I fixed that.
I also fixed various other problems in TreeBrowser. However, it's till buggy like if you create an element in the root of the display the list gets messed up. I think calling gtk_tree_store_iter_clear_nodes()[1] unconditionally in treebrowser_browse() would fix the problem but since I don't know the plugin much I won't do this.
Regards, Colomban
[1] it is a *really* bad idea to name a custom function using a lib's prefix: it may clash and makes people assume it's a function from that lib.
Colomban _______________________________________________ Devel mailing list Devel@lists.geany.org https://lists.geany.org/cgi-bin/mailman/listinfo/devel