@b4n commented on this pull request.


In doc/plugins.dox:

> +static gboolean autocomplete_provided(GeanyDocument *doc, gpointer data)
+{
+    /* Check whether the plugin provides the feature for the passed document */
+    return doc->file_type->id == GEANY_FILETYPES_PYTHON;
+}
+
+
+static void autocomplete_perform(GeanyDocument *doc, gboolean force, gpointer data)
+{
+    /* The autocompletion logic comes here, including the autocompletion UI
+     * display (either using some custom widget or using Scintilla's
+     * SCI_AUTOCSHOW) */
+}
+
+
+/* The PluginExtension instance - we only implement autocompletion here. */

Maybe there's a clarification needed, but I feel that it's pretty self-explanatory that if you pass a structure full of function pointers (and to non-exported functions even), it's the function pointer that matters, not its name. But well, if there's a way to make that explicit, sure.


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <geany/geany/pull/3849/review/2135590119@github.com>