[Github-comments] [geany/geany-plugins] MultiTerm: Add support for GTK3 (#95)

Matthew Brush notifications at xxxxx
Fri Jul 12 00:07:10 UTC 2019


codebrainz commented on this pull request.



> @@ -53,6 +53,8 @@ public void plugin_init(Geany.Data data)
 	Alignment align;
 	MultiTerm.Notebook notebook;
 
+	toplevel_widgets = new List<Widget>();

This probably needs to be assigned `null` in the cleanup function to avoid a leak in the GC and so there's no dangling junk left in it.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/95#pullrequestreview-261009125
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20190711/d94a42dc/attachment.html>


More information about the Github-comments mailing list