[Github-comments] [geany/geany] Overview of currently deprecated symbols (Issue #3019)

[xiota]

Here is a list of deprecated symbols that are marked with `@deprecated`, `GEANY_DEPRECATED`, `GEANY_DEPRECATED_FOR`, and `GEANY_DISABLE_DEPRECATED`.  Also noted are the version/API/date of deprecation (if known) and plugins they are used in.

As far as I can tell, the legacy plugin entry points (eg, `plugin_init`) are *not* marked deprecated, so should not be candidates for removal.  Also, 22 plugins still reference 'void plugin_init': geanyextrasel, geanyinsertnum, geanylua, geanymacro, geanyminiscript, geanynumberedbookmarks, geanypg, geanyprj, geniuspaste, markdown, multiterm, pairtaghighlighter, pretty-printer, projectorganizer, scope, sendmail, shiftcolumn, spellcheck, tableconvert, updatechecker, vimode, xmlsnippets

`documents.h`
* `documents_foreach(i)` (use `foreach_document` instead)
  + not used by any plugins

`filetypes.h`

* `filetype_id` (use `GeanyFiletypeID` instead)
  + not used by any plugins

`keybindings.h`

* `GEANY_KEYS_EDITOR_MACROLIST` (since 1.25)
  + used by geanylua
  + removal would affect values of other symbols in the enum; it could be "removed" by changing the doxygen comment into a normal one

`plugindata.h`

* `GeanyFunctions`
  + not used by any plugins
* `GeanyKeyGroupInfo` (use `plugin_set_key_group` instead)
  + not used by any plugins
* `PLUGIN_KEY_GROUP` (use plugin_set_key_group() instead)
  + not used by any plugins
* `PluginFlags` (use `ui_add_document_sensitive` instead)
  + used by multiterm
* `PluginFields` (use `ui_add_document_sensitive` instead)
  + used by multiterm
* `document_reload_file` (use `document_reload_force` instead)
  + used by multiterm
* `DOC_IDX(doc_ptr)`
  + not used by any plugins
* `DOC_IDX_VALID(doc_idx)`
  + not used by any plugins
* `GEANY_WINDOW_MINIMAL_WIDTH`
  + not used by any plugins
* `GEANY_WINDOW_MINIMAL_HEIGHT` (use `GEANY_DEFAULT_DIALOG_HEIGHT` instead)
  + not used by any plugins
* `PROXY_IGNORED` (since 1.26/226, use `GEANY_PROXY_IGNORE` instead)
  + used by geanypy
* `PROXY_MATCHED` (since 1.26/226, use `GEANY_PROXY_MATCH` instead)
  + used by geanypy
* `PROXY_NOLOAD` (since 1.26/226)
  + not used by any plugin

`sciwrappers.h`
* `sci_get_text` (use `sci_get_contents` instead)
  + used by `document.c`
  + used by geanygendoc
* `sci_get_selected_text` (use `sci_get_selection_contents` instead)
  + used by `ui_utils.c`
  + used by plugins: addons, geanyctags, geanypy, pretty-printer, scope
* `sci_get_text_range` (use `sci_get_contents_range` instead)
  + used in `editor.c`
  + not used by any plugins

`tagmanager/tm_source_file.c`

* `tm_get_real_path` (since 1.32/235, use `utils_get_real_path` instead)
  + used by `utils.c`
  + not used by any plugins

`ui_utils.h`

* `ui_frame_new_with_alignment` (since 1.29, use GTK API directly)
  + used by plugins: geanypy, webhelper
* `ui_widget_set_tooltip_text` (since 0.21, use `gtk_widget_set_tooltip_text` instead);
  + not used by any plugins

`utils.h`
* `NZV(ptr)`, deprecated 2013/08, use `!EMPTY()` instead
  + not used by any plugins
* `setptr(ptr, result)`, deprecated 2011/11/15, use `SETPTR()` instead.
  + used by plugins: addons, geanydoc, geanymacro, geanyprj, geanyvc, spellcheck, treebrowser

-- 
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/issues/3019
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20211125/083fa2d2/attachment.htm>