FaSTTUBe/GST-Tensordecoder-ov_ep
61
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

install-plugins: constify _install_*sync() details array arguments

Browse Source
This commit is contained in:
Tim-Philipp Müller 2012-09-09 18:29:40 +01:00
parent 72edcd33b5
commit 202cd3c17c
3 changed files with 12 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
gst-libs/gst/pbutils/install-plugins.c
View File

@ -489,7 +489,7 @@ ptr_array_contains_string (GPtrArray * arr, const gchar * s)
} }
static gboolean static gboolean
gst_install_plugins_spawn_child (gchar ** details, gst_install_plugins_spawn_child (const gchar * const *details,
GstInstallPluginsContext * ctx, GPid * child_pid, gint * exit_status) GstInstallPluginsContext * ctx, GPid * child_pid, gint * exit_status)
{ {
GPtrArray *arr; GPtrArray *arr;
@ -511,7 +511,7 @@ gst_install_plugins_spawn_child (gchar ** details,
/* finally, add the detail strings, but without duplicates */ /* finally, add the detail strings, but without duplicates */
while (details != NULL && details[0] != NULL) { while (details != NULL && details[0] != NULL) {
if (!ptr_array_contains_string (arr, details[0])) if (!ptr_array_contains_string (arr, details[0]))
g_ptr_array_add (arr, details[0]); g_ptr_array_add (arr, (gpointer) details[0]);
++details; ++details;
} }
@ -618,8 +618,9 @@ gst_install_plugins_installer_exited (GPid pid, gint status, gpointer data)
*/ */
GstInstallPluginsReturn GstInstallPluginsReturn
gst_install_plugins_async (gchar ** details, GstInstallPluginsContext * ctx, gst_install_plugins_async (const gchar * const *details,
GstInstallPluginsResultFunc func, gpointer user_data) GstInstallPluginsContext * ctx, GstInstallPluginsResultFunc func,
gpointer user_data)
{ {
GstInstallPluginsAsyncHelper *helper; GstInstallPluginsAsyncHelper *helper;
GPid pid; GPid pid;
@ -665,7 +666,8 @@ gst_install_plugins_async (gchar ** details, GstInstallPluginsContext * ctx,
* Returns: the result of the installation. * Returns: the result of the installation.
*/ */
GstInstallPluginsReturn GstInstallPluginsReturn
gst_install_plugins_sync (gchar ** details, GstInstallPluginsContext * ctx) gst_install_plugins_sync (const gchar * const *details,
GstInstallPluginsContext * ctx)
{ {
gint status; gint status;

4
gst-libs/gst/pbutils/install-plugins.h
View File

@ -118,12 +118,12 @@ GType gst_install_plugins_context_get_type (void);
typedef void (*GstInstallPluginsResultFunc) (GstInstallPluginsReturn result, typedef void (*GstInstallPluginsResultFunc) (GstInstallPluginsReturn result,
gpointer user_data); gpointer user_data);
GstInstallPluginsReturn gst_install_plugins_async (gchar ** details, GstInstallPluginsReturn gst_install_plugins_async (const gchar * const * details,
GstInstallPluginsContext * ctx, GstInstallPluginsContext * ctx,
GstInstallPluginsResultFunc func, GstInstallPluginsResultFunc func,
gpointer user_data); gpointer user_data);
GstInstallPluginsReturn gst_install_plugins_sync (gchar ** details, GstInstallPluginsReturn gst_install_plugins_sync (const gchar * const * details,
GstInstallPluginsContext * ctx); GstInstallPluginsContext * ctx);
const gchar * gst_install_plugins_return_get_name (GstInstallPluginsReturn ret); const gchar * gst_install_plugins_return_get_name (GstInstallPluginsReturn ret);

7
tests/check/libs/pbutils.c
View File

@ -480,7 +480,7 @@ result_cb (GstInstallPluginsReturn result, gpointer user_data)
/* make sure our script gets called with the right parameters */ /* make sure our script gets called with the right parameters */
static void static void
test_pb_utils_install_plugins_do_callout (gchar ** details, test_pb_utils_install_plugins_do_callout (const gchar * const *details,
GstInstallPluginsContext * ctx, const gchar * script, GstInstallPluginsContext * ctx, const gchar * script,
GstInstallPluginsReturn expected_result) GstInstallPluginsReturn expected_result)
{ {
@ -548,9 +548,8 @@ GST_START_TEST (test_pb_utils_install_plugins)
{ {
GstInstallPluginsContext *ctx; GstInstallPluginsContext *ctx;
GstInstallPluginsReturn ret; GstInstallPluginsReturn ret;
gchar *details[] = { (char *) "detail1", (char *) "detail2", NULL }; const gchar *details[] = { "detail1", "detail2", NULL };
gchar *details_multi[] = const gchar *details_multi[] = { "detail1", "detail1", "detail2", NULL };
{ (char *) "detail1", (char *) "detail1", (char *) "detail2", NULL };
ctx = gst_install_plugins_context_new (); ctx = gst_install_plugins_context_new ();