elextr commented on this pull request.

Guess its reasonable to get the data you set :)

> @@ -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);

These should be normal `if(..)return val` because they should not be removable 
by compile option as the `g_...` checks are.

-- 
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#pullrequestreview-466730

Reply via email to