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

[Matthew Brush]

codebrainz commented on this pull request.



> @@ -68,6 +68,20 @@ namespace MultiTerm
 			item.show();
 		}
 
+		private Gtk.MenuItem add_image_menu_item(string label_text, string icon_name)

I guess this should be guarded with `#if MULTITERM_GTK3` too? IIRC Vala compiler warns about unused private functions.

-- 
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-259197098
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20190708/69469ac3/attachment.html>