@techee commented on this pull request.


>  {
+       GeanyDocument *doc = document_get_current();
+
+       if (plugin_extension_goto_provided(doc, NULL))
+               return plugin_extension_goto_perform(doc, pos, definition);

I'd just point out that this eliminates `get_current_word_or_sel()` from the 
original `goto_tag()` so `editor_find_current_word()` is used both in the 
ctrl+click and keybinding case. Is the goto selection something worth 
preserving? I've never used it myself and didn't even know it existed.

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

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

Reply via email to