@techee commented on this pull request.


> +     void (*calltips_show)(GeanyDocument *doc, gboolean force);
+
+       gboolean (*goto_provided)(GeanyDocument *doc);
+       void (*goto_perform)(GeanyDocument *doc, gint pos, gboolean definition);
+
+       gboolean (*doc_symbols_provided)(GeanyDocument *doc);
+       GPtrArray *(*doc_symbols_get)(GeanyDocument *doc);
+
+       gboolean (*symbol_highlight_provided)(GeanyDocument *doc);
+
+       gchar _dummy[1024];
+} PluginExtension;
+
+
+void plugin_extension_register(PluginExtension *extension);
+void plugin_extension_unregister(PluginExtension *extension);

OK, what about modifying every `_provided(GeanyDocument *doc)` to 
```
_provided(GeanyDocument *doc, PluginExtension *ext)
```
which:
* if `ext != NULL`, returns TRUE only if the passed `ext` is the one that won, 
FALSE otherwise, 
* if `ext == NULL` it behaves like now?

(And, in addition, maybe `_active()` is a better naming than `_provided()`)

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3849#discussion_r1625685381
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/3849/review/[email protected]>

Reply via email to