GNOME Bugzilla – Bug 604840
Add introspection support
Last modified: 2010-06-20 09:57:09 UTC
GtkSourceView should provide introspection support by itself. I'm also including a gtk-doc bug patch found by the g-ir-scanner
Created attachment 149916 [details] [review] Add introspection support
Created attachment 149917 [details] [review] Fix a gtk-doc bug found by g-ir-scanner
This is a dup of https://bugzilla.gnome.org/show_bug.cgi?id=598536. I leave it opened to check both patches.
Review of attachment 149916 [details] [review]: Just to know, do introspection works in win32? If not is it going to be added support soon? ::: Makefile.am @@ +21,1 @@ This is wrong, it should be .m4 ::: gtksourceview/Makefile.am @@ +105,3 @@ +INTROSPECTION_COMPILER_ARGS = --includedir=$(srcdir) +INTROSPECTION_SCANNER_ARGS = --add-include-path=$(srcdir) +INTROSPECTION_GIRS = includedir and add-include-path is suppose to work recursively? if not you should add also the completion-provider folder
It was tested some time ago on win32, probably bitrotten by now. There's nothing fundamental against getting it to work on windows, mainly build issues. Can't see why it can't be enabled on other platforms until it builds properly on windows, as it clearly provides application features on other platforms. The completion-provider directory should only be added if it contains public API, as it doesn't install any headers I assumed it was private so I didn't add it. The includedir/add-include-path are not really what you think they are, they're mainly internal to introspection and has nothing to do with C headers/sources.
Well, I didn't say to not include it until win32 is supported, as I said above it was just to know. About the completion-provider it has public api. gtksourcecompletionwords.h <- so it is needed something else for that? Apart from that the patch looks good to me.
Oops, I missed that it was installed, I stand corrected. There's however a slight API problem, the GtkSourceCompletionWords uses the "GtkSource" prefix instead of "GtkSourceView" but it's part of the same .so. It should ideally use another namespace prefix OR be put in another separate .so. I think it's safe to add introspection support without this until it's decided how it relates to the GtkSourceView API.
mmm, but the right prefix for is GtkSource, isn't it? I mean for gtksourcecompletion is gtksource, for gtksourcelanguage <- again gtksource, same for gtksourcegutter etc. Not sure if that's what you are asking here.
*** This bug has been marked as a duplicate of bug 598536 ***