From 5a7ed3f46fa2e0151a0f403824ae3a7df10a6c34 Mon Sep 17 00:00:00 2001 From: David Robillard Date: Thu, 24 Feb 2011 01:31:49 +0000 Subject: Move UI stuff back into slv2.h (avoid breakage). Deprecate slv2_ui_instance_get_descriptor and slv2_ui_instance_get_handle. git-svn-id: http://svn.drobilla.net/lad/trunk/slv2@3020 a436a847-0d15-0410-975c-d299462d15a1 --- slv2/slv2.h | 151 +++++++++++++++++++++++++++++++++++++++++++++ slv2/ui.h | 175 ---------------------------------------------------- src/collections.c | 2 - src/plugin.c | 2 - src/slv2_internal.h | 9 --- test/slv2_test.c | 9 --- utils/lv2_inspect.c | 7 +-- wscript | 16 +---- 8 files changed, 155 insertions(+), 216 deletions(-) delete mode 100644 slv2/ui.h diff --git a/slv2/slv2.h b/slv2/slv2.h index 8c198bd..e703689 100644 --- a/slv2/slv2.h +++ b/slv2/slv2.h @@ -23,6 +23,7 @@ #include #include "lv2/lv2plug.in/ns/lv2core/lv2.h" +#include "lv2/lv2plug.in/ns/extensions/ui/ui.h" #ifdef __cplusplus extern "C" { @@ -46,6 +47,16 @@ extern "C" { #define SLV2_API #endif +#if (__GNUC__ > 3 || (__GNUC__ == 3 && __GNUC_MINOR__ >= 1)) + #ifdef SLV2_INTERNAL // Building SLV2 (not using it) + #define SLV2_DEPRECATED + #else + #define SLV2_DEPRECATED __attribute__((__deprecated__)) + #endif +#else + #define SLV2_DEPRECATED +#endif + #define SLV2_NAMESPACE_LV2 "http://lv2plug.in/ns/lv2core#" #define SLV2_PORT_CLASS_PORT "http://lv2plug.in/ns/lv2core#Port" #define SLV2_PORT_CLASS_INPUT "http://lv2plug.in/ns/lv2core#InputPort" @@ -59,12 +70,14 @@ typedef struct _SLV2Plugin* SLV2Plugin; /**< LV2 Plugin. */ typedef struct _SLV2PluginClass* SLV2PluginClass; /**< Plugin Class. */ typedef struct _SLV2Port* SLV2Port; /**< Port. */ typedef struct _SLV2ScalePoint* SLV2ScalePoint; /**< Scale Point (Notch). */ +typedef struct _SLV2UI* SLV2UI; /**< Plugin UI. */ typedef struct _SLV2Value* SLV2Value; /**< Typed Value. */ typedef struct _SLV2World* SLV2World; /**< SLV2 World. */ typedef void* SLV2PluginClasses; /**< set. */ typedef void* SLV2Plugins; /**< set. */ typedef void* SLV2ScalePoints; /**< array. */ +typedef void* SLV2UIs; /**< set. */ typedef void* SLV2Values; /**< array. */ /** @defgroup slv2 SLV2 @@ -293,6 +306,7 @@ prefix ## _get_at(CollType collection, \ SLV2_COLLECTION(SLV2PluginClasses, SLV2PluginClass, slv2_plugin_classes) SLV2_COLLECTION(SLV2ScalePoints, SLV2ScalePoint, slv2_scale_points) +SLV2_COLLECTION(SLV2UIs, SLV2UI, slv2_uis) SLV2_COLLECTION(SLV2Values, SLV2Value, slv2_values) /** Free a plugin collection. @@ -1056,6 +1070,143 @@ slv2_instance_get_handle(SLV2Instance instance) #endif /* SLV2_INTERNAL */ +/** @} */ +/** @name Plugin UI + * @{ + */ + +/** Get a UI from @a uis by URI. + * Return value is shared (stored in @a uis) and must not be freed or + * modified by the caller in any way. + * @return NULL if no UI with @a uri is found in @a list. + */ +SLV2_API +SLV2UI +slv2_uis_get_by_uri(SLV2UIs uis, + SLV2Value uri); + +/** Get a list of all UIs available for this plugin. + * Note this returns the URI of the UI, and not the path/URI to its shared + * library, use slv2_ui_get_library_uri with the values returned + * here for that. + * + * Returned value must be freed by caller using slv2_uis_free. + */ +SLV2_API +SLV2UIs +slv2_plugin_get_uis(SLV2Plugin plugin); + +/** @name Plugin UI + * @{ + */ + +/** Get the URI of a Plugin UI. + * @param ui The Plugin UI + * @return a shared value which must not be modified or freed. + */ +SLV2_API +SLV2Value +slv2_ui_get_uri(SLV2UI ui); + +/** Get the types (URIs of RDF classes) of a Plugin UI. + * @param ui The Plugin UI + * @return a shared value which must not be modified or freed. + */ +SLV2_API +SLV2Values +slv2_ui_get_classes(SLV2UI ui); + +/** Check whether a plugin UI is a given type. + * @param ui The Plugin UI + * @param class_uri The URI of the LV2 UI type to check this UI against + */ +SLV2_API +bool +slv2_ui_is_a(SLV2UI ui, SLV2Value class_uri); + +/** Get the URI for a Plugin UI's bundle. + * @param ui The Plugin UI + * @return a shared value which must not be modified or freed. + */ +SLV2_API +SLV2Value +slv2_ui_get_bundle_uri(SLV2UI ui); + +/** Get the URI for a Plugin UI's shared library. + * @param ui The Plugin UI + * @return a shared value which must not be modified or freed. + */ +SLV2_API +SLV2Value +slv2_ui_get_binary_uri(SLV2UI ui); + +/** @} */ +/** @name Plugin UI Instance + * @{ + */ + +typedef struct _SLV2UIInstance* SLV2UIInstance; + +/** Instantiate a plugin UI. + * The returned object represents shared library objects loaded into memory, + * it must be cleaned up with slv2_ui_instance_free when no longer + * needed. + * + * @a plugin is not modified or directly referenced by the returned object + * (instances store only a copy of the plugin's URI). + * + * @a features NULL-terminated array of features the host supports. + * NULL may be passed if the host supports no additional features (unlike + * the LV2 specification - SLV2 takes care of it). + * + * @return NULL if instantiation failed. + */ +SLV2_API +SLV2UIInstance +slv2_ui_instantiate(SLV2Plugin plugin, + SLV2UI ui, + LV2UI_Write_Function write_function, + LV2UI_Controller controller, + const LV2_Feature* const* features); + +/** Free a plugin UI instance. + * @a instance is invalid after this call. + * It is the caller's responsibility to ensure all references to the UI + * instance (including any returned widgets) are cut before calling + * this function. + */ +SLV2_API +void +slv2_ui_instance_free(SLV2UIInstance instance); + +/** Get the widget for the UI instance. + */ +SLV2_API +LV2UI_Widget +slv2_ui_instance_get_widget(SLV2UIInstance instance); + +/** DEPRECATED: Get the LV2UI_Descriptor of the plugin UI instance. + * Normally hosts should not need to access the LV2UI_Descriptor directly, + * use the slv2_ui_instance_* functions. + * + * The returned descriptor is shared and must not be deleted. + */ +SLV2_DEPRECATED +SLV2_API +const LV2UI_Descriptor* +slv2_ui_instance_get_descriptor(SLV2UIInstance instance); + +/** DEPRECATED: Get the LV2UI_Handle of the plugin UI instance. + * Normally hosts should not need to access the LV2UI_Handle directly, + * use the slv2_ui_instance_* functions. + * + * The returned handle is shared and must not be deleted. + */ +SLV2_DEPRECATED +SLV2_API +LV2UI_Handle +slv2_ui_instance_get_handle(SLV2UIInstance instance); + /** @} */ /** @} */ diff --git a/slv2/ui.h b/slv2/ui.h deleted file mode 100644 index 84cb96a..0000000 --- a/slv2/ui.h +++ /dev/null @@ -1,175 +0,0 @@ -/* SLV2 - * Copyright (C) 2007-2011 David Robillard - * - * This library is free software; you can redistribute it and/or modify it - * under the terms of the GNU General Public License as published by the Free - * Software Foundation; either version 2 of the License, or (at your option) - * any later version. - * - * This library is distributed in the hope that it will be useful, but WITHOUT - * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or - * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License - * for more details. - * - * You should have received a copy of the GNU General Public License along - * with this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. - */ - -#ifndef SLV2_SLV2_UI_H__ -#define SLV2_SLV2_UI_H__ - -#include "slv2/slv2.h" -#include "lv2/lv2plug.in/ns/extensions/ui/ui.h" - -#ifdef __cplusplus -extern "C" { -#endif - -/** @addtogroup slv2 - * @{ - */ - -typedef struct _SLV2UI* SLV2UI; /**< Plugin UI. */ -typedef void* SLV2UIs; /**< set. */ - -SLV2_COLLECTION(SLV2UIs, SLV2UI, slv2_uis) - -/** Get a UI from @a uis by URI. - * Return value is shared (stored in @a uis) and must not be freed or - * modified by the caller in any way. - * @return NULL if no UI with @a uri is found in @a list. - */ -SLV2_API -SLV2UI -slv2_uis_get_by_uri(SLV2UIs uis, - SLV2Value uri); - -/** Get a list of all UIs available for this plugin. - * Note this returns the URI of the UI, and not the path/URI to its shared - * library, use slv2_ui_get_library_uri with the values returned - * here for that. - * - * Returned value must be freed by caller using slv2_uis_free. - */ -SLV2_API -SLV2UIs -slv2_plugin_get_uis(SLV2Plugin plugin); - -/** @name Plugin UI - * @{ - */ - -/** Get the URI of a Plugin UI. - * @param ui The Plugin UI - * @return a shared value which must not be modified or freed. - */ -SLV2_API -SLV2Value -slv2_ui_get_uri(SLV2UI ui); - -/** Get the types (URIs of RDF classes) of a Plugin UI. - * @param ui The Plugin UI - * @return a shared value which must not be modified or freed. - */ -SLV2_API -SLV2Values -slv2_ui_get_classes(SLV2UI ui); - -/** Check whether a plugin UI is a given type. - * @param ui The Plugin UI - * @param class_uri The URI of the LV2 UI type to check this UI against - */ -SLV2_API -bool -slv2_ui_is_a(SLV2UI ui, SLV2Value class_uri); - -/** Get the URI for a Plugin UI's bundle. - * @param ui The Plugin UI - * @return a shared value which must not be modified or freed. - */ -SLV2_API -SLV2Value -slv2_ui_get_bundle_uri(SLV2UI ui); - -/** Get the URI for a Plugin UI's shared library. - * @param ui The Plugin UI - * @return a shared value which must not be modified or freed. - */ -SLV2_API -SLV2Value -slv2_ui_get_binary_uri(SLV2UI ui); - -/** @} */ -/** @name Plugin UI Instance - * @{ - */ - -typedef struct _SLV2UIInstance* SLV2UIInstance; - -/** Instantiate a plugin UI. - * The returned object represents shared library objects loaded into memory, - * it must be cleaned up with slv2_ui_instance_free when no longer - * needed. - * - * @a plugin is not modified or directly referenced by the returned object - * (instances store only a copy of the plugin's URI). - * - * @a features NULL-terminated array of features the host supports. - * NULL may be passed if the host supports no additional features (unlike - * the LV2 specification - SLV2 takes care of it). - * - * @return NULL if instantiation failed. - */ -SLV2_API -SLV2UIInstance -slv2_ui_instantiate(SLV2Plugin plugin, - SLV2UI ui, - LV2UI_Write_Function write_function, - LV2UI_Controller controller, - const LV2_Feature* const* features); - -/** Free a plugin UI instance. - * @a instance is invalid after this call. - * It is the caller's responsibility to ensure all references to the UI - * instance (including any returned widgets) are cut before calling - * this function. - */ -SLV2_API -void -slv2_ui_instance_free(SLV2UIInstance instance); - -/** Get the widget for the UI instance. - */ -SLV2_API -LV2UI_Widget -slv2_ui_instance_get_widget(SLV2UIInstance instance); - -/** Get the LV2UI_Descriptor of the plugin UI instance. - * Normally hosts should not need to access the LV2UI_Descriptor directly, - * use the slv2_ui_instance_* functions. - * - * The returned descriptor is shared and must not be deleted. - */ -SLV2_API -const LV2UI_Descriptor* -slv2_ui_instance_get_descriptor(SLV2UIInstance instance); - -/** Get the LV2UI_Handle of the plugin UI instance. - * Normally hosts should not need to access the LV2UI_Handle directly, - * use the slv2_ui_instance_* functions. - * - * The returned handle is shared and must not be deleted. - */ -SLV2_API -LV2UI_Handle -slv2_ui_instance_get_handle(SLV2UIInstance instance); - -/** @} */ -/** @} */ - -#ifdef __cplusplus -} /* extern "C" */ -#endif - -#endif /* SLV2_SLV2_UI_H__ */ diff --git a/src/collections.c b/src/collections.c index 6ec004e..9ad2ddc 100644 --- a/src/collections.c +++ b/src/collections.c @@ -108,10 +108,8 @@ prefix ## _get_by_uri(CollType coll, SLV2Value uri) \ SLV2_SEQUENCE_IMPL(SLV2PluginClasses, SLV2PluginClass, slv2_plugin_classes, &slv2_plugin_class_free) -#ifdef SLV2_WITH_UI SLV2_SEQUENCE_IMPL(SLV2UIs, SLV2UI, slv2_uis, &slv2_ui_free) -#endif /* VALUES */ diff --git a/src/plugin.c b/src/plugin.c index 2987371..f06cfd7 100644 --- a/src/plugin.c +++ b/src/plugin.c @@ -760,7 +760,6 @@ slv2_plugin_get_author_homepage(SLV2Plugin plugin) return NULL; } -#ifdef SLV2_WITH_UI SLV2_API SLV2UIs slv2_plugin_get_uis(SLV2Plugin p) @@ -811,4 +810,3 @@ slv2_plugin_get_uis(SLV2Plugin p) return NULL; } } -#endif /* SLV2_WITH_UI */ diff --git a/src/slv2_internal.h b/src/slv2_internal.h index a52f62e..4314a73 100644 --- a/src/slv2_internal.h +++ b/src/slv2_internal.h @@ -42,9 +42,6 @@ extern "C" { #endif #include "slv2/slv2.h" -#ifdef SLV2_WITH_UI -#include "slv2/ui.h" -#endif #define SLV2_NS_DOAP (const uint8_t*)"http://usefulinc.com/ns/doap#" #define SLV2_NS_RDFS (const uint8_t*)"http://www.w3.org/2000/01/rdf-schema#" @@ -153,14 +150,12 @@ struct _SLV2InstanceImpl { /* ********* UI Instance ********* */ -#ifdef SLV2_WITH_UI struct _SLV2UIInstance { void* lib_handle; const LV2UI_Descriptor* lv2ui_descriptor; LV2UI_Handle lv2ui_handle; LV2UI_Widget widget; }; -#endif /* ********* Plugin Class ********* */ @@ -234,8 +229,6 @@ slv2_world_load_file(SLV2World world, const char* file_uri); /* ********* Plugin UI ********* */ -#ifdef SLV2_WITH_UI - struct _SLV2UI { struct _SLV2World* world; SLV2Value uri; @@ -253,8 +246,6 @@ slv2_ui_new(SLV2World world, void slv2_ui_free(SLV2UI ui); -#endif /* SLV2_WITH_UI */ - /* ********* Value ********* */ typedef enum _SLV2ValueType { diff --git a/test/slv2_test.c b/test/slv2_test.c index eca38ee..665448a 100644 --- a/test/slv2_test.c +++ b/test/slv2_test.c @@ -30,9 +30,6 @@ #include #include "slv2/slv2.h" -#ifdef SLV2_WITH_UI -#include "slv2/ui.h" -#endif #define TEST_PATH_MAX 1024 @@ -710,11 +707,9 @@ test_plugin() TEST_ASSERT(slv2_value_is_string(thing_name)); TEST_ASSERT(!strcmp(slv2_value_as_string(thing_name), "Something else")); - #ifdef SLV2_WITH_UI SLV2UIs uis = slv2_plugin_get_uis(plug); TEST_ASSERT(slv2_uis_size(uis) == 0); slv2_uis_free(uis); - #endif slv2_values_free(thing_names); slv2_value_free(thing_uri); @@ -918,7 +913,6 @@ test_port() /*****************************************************************************/ -#ifdef SLV2_WITH_UI int test_ui() { @@ -1019,7 +1013,6 @@ test_ui() cleanup_uris(); return 1; } -#endif /* SLV2_WITH_UI */ /*****************************************************************************/ @@ -1035,9 +1028,7 @@ static struct TestCase tests[] = { TEST_CASE(plugin), TEST_CASE(port), TEST_CASE(plugin), -#ifdef SLV2_WITH_UI TEST_CASE(ui), -#endif { NULL, NULL } }; diff --git a/utils/lv2_inspect.c b/utils/lv2_inspect.c index 7484d96..5a405ce 100644 --- a/utils/lv2_inspect.c +++ b/utils/lv2_inspect.c @@ -23,9 +23,6 @@ #include #include "slv2/slv2.h" -#ifdef SLV2_WITH_UI -#include "slv2/ui.h" -#endif #include "slv2-config.h" @@ -174,10 +171,9 @@ print_plugin(SLV2Plugin p) printf("\tBundle: %s\n", slv2_value_as_uri(slv2_plugin_get_bundle_uri(p))); printf("\tBinary: %s\n", slv2_value_as_uri(slv2_plugin_get_library_uri(p))); -#ifdef SLV2_WITH_UI SLV2UIs uis = slv2_plugin_get_uis(p); if (slv2_values_size(uis) > 0) { - printf("\tGUI: "); + printf("\tUI: "); for (unsigned i = 0; i < slv2_uis_size(uis); ++i) { SLV2UI ui = slv2_uis_get_at(uis, i); printf("%s\n", slv2_value_as_uri(slv2_ui_get_uri(ui))); @@ -198,7 +194,6 @@ print_plugin(SLV2Plugin p) } } slv2_uis_free(uis); -#endif printf("\tData URIs: "); SLV2Values data_uris = slv2_plugin_get_data_uris(p); diff --git a/wscript b/wscript index ada28d9..b2073d2 100644 --- a/wscript +++ b/wscript @@ -92,11 +92,7 @@ def configure(conf): atleast_version='0.120.0', mandatory=False) autowaf.check_header(conf, 'lv2/lv2plug.in/ns/lv2core/lv2.h') - autowaf.check_header(conf, 'lv2/lv2plug.in/ns/extensions/ui/ui.h', - 'HAVE_UI_H') - - if conf.env['HAVE_UI_H']: - autowaf.define(conf, 'SLV2_WITH_UI', 1) + autowaf.check_header(conf, 'lv2/lv2plug.in/ns/extensions/ui/ui.h') if not Options.options.no_jack_session: if conf.env['HAVE_NEW_JACK'] and conf.env['HAVE_GTHREAD']: @@ -165,8 +161,6 @@ def configure(conf): bool(conf.env['BUILD_TESTS'])) autowaf.display_msg(conf, "Dynamic manifest support", bool(conf.env['SLV2_DYN_MANIFEST'])) - autowaf.display_msg(conf, "UI support", - bool(conf.env['SLV2_WITH_UI'])) autowaf.display_msg(conf, "Python bindings", bool(conf.env['SLV2_SWIG'])) print @@ -185,6 +179,8 @@ def build(bld): src/pluginclass.c src/plugininstance.c src/plugins.c + src/pluginui.c + src/pluginuiinstance.c src/port.c src/query.c src/scalepoint.c @@ -193,12 +189,6 @@ def build(bld): src/world.c '''.split() - if bld.env['SLV2_WITH_UI']: - lib_source += ''' - src/pluginui.c - src/pluginuiinstance.c - '''.split() - # Library obj = bld(features = 'c cshlib') obj.export_includes = ['.'] -- cgit v1.2.1