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 <fr...@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 > _______________________________________________ Devel mailing list Devel@lists.geany.org https://lists.geany.org/cgi-bin/mailman/listinfo/devel