elextr commented on this pull request.


> @@ -554,6 +554,24 @@ void plugin_builder_connect_signals(GeanyPlugin *plugin,
 }
 
 
+/** Get the additional data that corresponds to the plugin.
+ *
+ * @param plugin The plugin provided by Geany
+ * @return The data corresponding to the plugin or @c NULL if none set.
+ *
+ * @since 1.29 (API 230)
+ *
+ * @see geany_plugin_set_data()
+ */
+gpointer geany_plugin_get_data(const GeanyPlugin *plugin)
+{
+       g_return_val_if_fail (plugin != NULL, NULL);
+       g_return_val_if_fail (PLUGIN_LOADED_OK (plugin->priv), NULL);

Indeed, but plugins are external code, and can pass bad parameters at any time. 
All functions in the API should be permanently checked.  (yes Geany is not 
consistent about that, but you can have the honour of being the first to get it 
right :)

-- 
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/pull/1234

Reply via email to