summaryrefslogtreecommitdiff
path: root/libpeas
diff options
context:
space:
mode:
authorChristian Hergert <chergert@redhat.com>2023-03-17 14:34:15 -0700
committerChristian Hergert <chergert@redhat.com>2023-03-22 16:44:35 -0700
commitea348fc01180d135c4d3d59eede52e48ea906a43 (patch)
treed0af259c4bc6c5882042b9d1ce6eabc3d054cdcb /libpeas
parentc40bf394e6fd1cc7c077e79a2a8ba97cf3a1c608 (diff)
downloadlibpeas-ea348fc01180d135c4d3d59eede52e48ea906a43.tar.gz
plugin-info: fix usage of char including constness
We use `const char * const *` for read-only char** arrays. Additionally, stop using gchar and instead use char directly.
Diffstat (limited to 'libpeas')
-rw-r--r--libpeas/peas-engine.c6
-rw-r--r--libpeas/peas-plugin-info.c58
-rw-r--r--libpeas/peas-plugin-info.h52
3 files changed, 56 insertions, 60 deletions
diff --git a/libpeas/peas-engine.c b/libpeas/peas-engine.c
index da04ad3..d48b247 100644
--- a/libpeas/peas-engine.c
+++ b/libpeas/peas-engine.c
@@ -127,7 +127,7 @@ plugin_info_add_sorted (GQueue *plugin_list,
{
guint i;
GList *furthest_dep = NULL;
- const gchar **dependencies;
+ const gchar * const *dependencies;
dependencies = peas_plugin_info_get_dependencies (info);
@@ -173,7 +173,7 @@ plugin_info_add_sorted (GQueue *plugin_list,
for (iter = plugin_list->head; iter; iter = iter->next)
{
const PeasPluginInfo *other = iter->data;
- const char **other_dependencies = peas_plugin_info_get_dependencies (other);
+ const char * const *other_dependencies = peas_plugin_info_get_dependencies (other);
for (i = 0; other_dependencies[i] != NULL; i++)
{
@@ -1127,7 +1127,7 @@ static void
peas_engine_load_plugin_real (PeasEngine *engine,
PeasPluginInfo *info)
{
- const gchar **dependencies;
+ const gchar * const *dependencies;
PeasPluginInfo *dep_info;
guint i;
PeasPluginLoader *loader;
diff --git a/libpeas/peas-plugin-info.c b/libpeas/peas-plugin-info.c
index 9907957..fbf4db9 100644
--- a/libpeas/peas-plugin-info.c
+++ b/libpeas/peas-plugin-info.c
@@ -116,14 +116,14 @@ peas_plugin_info_init (PeasPluginInfo *info)
* Returns: a newly created #PeasPluginInfo.
*/
PeasPluginInfo *
-_peas_plugin_info_new (const gchar *filename,
- const gchar *module_dir,
- const gchar *data_dir)
+_peas_plugin_info_new (const char *filename,
+ const char *module_dir,
+ const char *data_dir)
{
gsize i;
gboolean is_resource;
- gchar *loader = NULL;
- gchar **strv, **keys;
+ char *loader = NULL;
+ char **strv, **keys;
PeasPluginInfo *info;
GKeyFile *plugin_file;
GBytes *bytes = NULL;
@@ -145,7 +145,7 @@ _peas_plugin_info_new (const gchar *filename,
}
else
{
- gchar *content;
+ char *content;
gsize length;
if (g_file_get_contents (filename, &content, &length, &error))
@@ -233,7 +233,7 @@ _peas_plugin_info_new (const gchar *filename,
"Plugin",
"Depends", NULL, NULL);
if (info->dependencies == NULL)
- info->dependencies = g_new0 (gchar *, 1);
+ info->dependencies = g_new0 (char *, 1);
/* Get Description */
info->desc = g_key_file_get_locale_string (plugin_file, "Plugin",
@@ -247,7 +247,7 @@ _peas_plugin_info_new (const gchar *filename,
info->authors = g_key_file_get_string_list (plugin_file, "Plugin",
"Authors", NULL, NULL);
if (info->authors == NULL)
- info->authors = g_new0 (gchar *, 1);
+ info->authors = g_new0 (char *, 1);
/* Get Copyright */
strv = g_key_file_get_string_list (plugin_file, "Plugin",
@@ -437,7 +437,7 @@ peas_plugin_info_is_hidden (const PeasPluginInfo *info)
*
* Returns: the module name.
*/
-const gchar *
+const char *
peas_plugin_info_get_module_name (const PeasPluginInfo *info)
{
g_return_val_if_fail (info != NULL, NULL);
@@ -457,7 +457,7 @@ peas_plugin_info_get_module_name (const PeasPluginInfo *info)
*
* Returns: the module directory.
*/
-const gchar *
+const char *
peas_plugin_info_get_module_dir (const PeasPluginInfo *info)
{
g_return_val_if_fail (info != NULL, NULL);
@@ -478,7 +478,7 @@ peas_plugin_info_get_module_dir (const PeasPluginInfo *info)
*
* Returns: the plugin's data dir.
*/
-const gchar *
+const char *
peas_plugin_info_get_data_dir (const PeasPluginInfo *info)
{
g_return_val_if_fail (info != NULL, NULL);
@@ -501,7 +501,7 @@ peas_plugin_info_get_data_dir (const PeasPluginInfo *info)
*/
GSettings *
peas_plugin_info_get_settings (const PeasPluginInfo *info,
- const gchar *schema_id)
+ const char *schema_id)
{
GSettingsSchema *schema;
GSettings *settings;
@@ -520,14 +520,14 @@ peas_plugin_info_get_settings (const PeasPluginInfo *info,
if (!g_file_query_exists (gschema_compiled, NULL))
{
- const gchar *argv[] = {
+ const char *argv[] = {
"glib-compile-schemas",
"--targetdir", info->module_dir,
info->module_dir,
NULL
};
- g_spawn_sync (NULL, (gchar **) argv, NULL, G_SPAWN_SEARCH_PATH,
+ g_spawn_sync (NULL, (char **) argv, NULL, G_SPAWN_SEARCH_PATH,
NULL, NULL, NULL, NULL, NULL, NULL);
}
@@ -576,12 +576,12 @@ peas_plugin_info_get_settings (const PeasPluginInfo *info,
*
* Returns: (transfer none): the plugin's dependencies.
*/
-const gchar **
+const char * const *
peas_plugin_info_get_dependencies (const PeasPluginInfo *info)
{
g_return_val_if_fail (info != NULL, NULL);
- return (const gchar **) info->dependencies;
+ return (const char * const *) info->dependencies;
}
/**
@@ -595,7 +595,7 @@ peas_plugin_info_get_dependencies (const PeasPluginInfo *info)
*/
gboolean
peas_plugin_info_has_dependency (const PeasPluginInfo *info,
- const gchar *module_name)
+ const char *module_name)
{
guint i;
@@ -624,7 +624,7 @@ peas_plugin_info_has_dependency (const PeasPluginInfo *info,
*
* Returns: the plugin's name.
*/
-const gchar *
+const char *
peas_plugin_info_get_name (const PeasPluginInfo *info)
{
g_return_val_if_fail (info != NULL, NULL);
@@ -645,7 +645,7 @@ peas_plugin_info_get_name (const PeasPluginInfo *info)
*
* Returns: the plugin's description.
*/
-const gchar *
+const char *
peas_plugin_info_get_description (const PeasPluginInfo *info)
{
g_return_val_if_fail (info != NULL, NULL);
@@ -666,7 +666,7 @@ peas_plugin_info_get_description (const PeasPluginInfo *info)
*
* Returns: the plugin's icon name.
*/
-const gchar *
+const char *
peas_plugin_info_get_icon_name (const PeasPluginInfo *info)
{
g_return_val_if_fail (info != NULL, NULL);
@@ -687,12 +687,12 @@ peas_plugin_info_get_icon_name (const PeasPluginInfo *info)
*
* Returns: (transfer none) (array zero-terminated=1): the plugin's author list.
*/
-const gchar **
+const char * const *
peas_plugin_info_get_authors (const PeasPluginInfo *info)
{
- g_return_val_if_fail (info != NULL, (const gchar **) NULL);
+ g_return_val_if_fail (info != NULL, (const char **) NULL);
- return (const gchar **) info->authors;
+ return (const char * const *) info->authors;
}
/**
@@ -705,7 +705,7 @@ peas_plugin_info_get_authors (const PeasPluginInfo *info)
*
* Returns: the plugin's associated website.
*/
-const gchar *
+const char *
peas_plugin_info_get_website (const PeasPluginInfo *info)
{
g_return_val_if_fail (info != NULL, NULL);
@@ -723,7 +723,7 @@ peas_plugin_info_get_website (const PeasPluginInfo *info)
*
* Returns: the plugin's copyright information.
*/
-const gchar *
+const char *
peas_plugin_info_get_copyright (const PeasPluginInfo *info)
{
g_return_val_if_fail (info != NULL, NULL);
@@ -741,7 +741,7 @@ peas_plugin_info_get_copyright (const PeasPluginInfo *info)
*
* Returns: the plugin's version.
*/
-const gchar *
+const char *
peas_plugin_info_get_version (const PeasPluginInfo *info)
{
g_return_val_if_fail (info != NULL, NULL);
@@ -765,7 +765,7 @@ peas_plugin_info_get_version (const PeasPluginInfo *info)
*
* Returns: the plugin's help URI.
*/
-const gchar *
+const char *
peas_plugin_info_get_help_uri (const PeasPluginInfo *info)
{
g_return_val_if_fail (info != NULL, NULL);
@@ -791,9 +791,9 @@ peas_plugin_info_get_help_uri (const PeasPluginInfo *info)
*
* Since: 1.6
*/
-const gchar *
+const char *
peas_plugin_info_get_external_data (const PeasPluginInfo *info,
- const gchar *key)
+ const char *key)
{
g_return_val_if_fail (info != NULL, NULL);
g_return_val_if_fail (key != NULL, NULL);
diff --git a/libpeas/peas-plugin-info.h b/libpeas/peas-plugin-info.h
index f7b675a..f5c4288 100644
--- a/libpeas/peas-plugin-info.h
+++ b/libpeas/peas-plugin-info.h
@@ -70,52 +70,48 @@ PEAS_AVAILABLE_IN_ALL
G_DECLARE_FINAL_TYPE (PeasPluginInfo, peas_plugin_info, PEAS, PLUGIN_INFO, GObject)
PEAS_AVAILABLE_IN_ALL
-GQuark peas_plugin_info_error_quark (void);
-
+GQuark peas_plugin_info_error_quark (void);
PEAS_AVAILABLE_IN_ALL
-gboolean peas_plugin_info_is_loaded (const PeasPluginInfo *info);
+gboolean peas_plugin_info_is_loaded (const PeasPluginInfo *info);
PEAS_AVAILABLE_IN_ALL
-gboolean peas_plugin_info_is_available (const PeasPluginInfo *info,
- GError **error);
+gboolean peas_plugin_info_is_available (const PeasPluginInfo *info,
+ GError **error);
PEAS_AVAILABLE_IN_ALL
-gboolean peas_plugin_info_is_builtin (const PeasPluginInfo *info);
+gboolean peas_plugin_info_is_builtin (const PeasPluginInfo *info);
PEAS_AVAILABLE_IN_ALL
-gboolean peas_plugin_info_is_hidden (const PeasPluginInfo *info);
-
+gboolean peas_plugin_info_is_hidden (const PeasPluginInfo *info);
PEAS_AVAILABLE_IN_ALL
-const gchar *peas_plugin_info_get_module_name (const PeasPluginInfo *info);
+const char *peas_plugin_info_get_module_name (const PeasPluginInfo *info);
PEAS_AVAILABLE_IN_ALL
-const gchar *peas_plugin_info_get_module_dir (const PeasPluginInfo *info);
+const char *peas_plugin_info_get_module_dir (const PeasPluginInfo *info);
PEAS_AVAILABLE_IN_ALL
-const gchar *peas_plugin_info_get_data_dir (const PeasPluginInfo *info);
+const char *peas_plugin_info_get_data_dir (const PeasPluginInfo *info);
PEAS_AVAILABLE_IN_ALL
-GSettings *peas_plugin_info_get_settings (const PeasPluginInfo *info,
- const gchar *schema_id);
+GSettings *peas_plugin_info_get_settings (const PeasPluginInfo *info,
+ const char *schema_id);
PEAS_AVAILABLE_IN_ALL
-const gchar **peas_plugin_info_get_dependencies (const PeasPluginInfo *info);
+const char * const *peas_plugin_info_get_dependencies (const PeasPluginInfo *info);
PEAS_AVAILABLE_IN_ALL
-gboolean peas_plugin_info_has_dependency (const PeasPluginInfo *info,
- const gchar *module_name);
-
+gboolean peas_plugin_info_has_dependency (const PeasPluginInfo *info,
+ const char *module_name);
PEAS_AVAILABLE_IN_ALL
-const gchar *peas_plugin_info_get_name (const PeasPluginInfo *info);
+const char *peas_plugin_info_get_name (const PeasPluginInfo *info);
PEAS_AVAILABLE_IN_ALL
-const gchar *peas_plugin_info_get_description (const PeasPluginInfo *info);
+const char *peas_plugin_info_get_description (const PeasPluginInfo *info);
PEAS_AVAILABLE_IN_ALL
-const gchar *peas_plugin_info_get_icon_name (const PeasPluginInfo *info);
+const char *peas_plugin_info_get_icon_name (const PeasPluginInfo *info);
PEAS_AVAILABLE_IN_ALL
-const gchar **peas_plugin_info_get_authors (const PeasPluginInfo *info);
+const char * const *peas_plugin_info_get_authors (const PeasPluginInfo *info);
PEAS_AVAILABLE_IN_ALL
-const gchar *peas_plugin_info_get_website (const PeasPluginInfo *info);
+const char *peas_plugin_info_get_website (const PeasPluginInfo *info);
PEAS_AVAILABLE_IN_ALL
-const gchar *peas_plugin_info_get_copyright (const PeasPluginInfo *info);
+const char *peas_plugin_info_get_copyright (const PeasPluginInfo *info);
PEAS_AVAILABLE_IN_ALL
-const gchar *peas_plugin_info_get_version (const PeasPluginInfo *info);
+const char *peas_plugin_info_get_version (const PeasPluginInfo *info);
PEAS_AVAILABLE_IN_ALL
-const gchar *peas_plugin_info_get_help_uri (const PeasPluginInfo *info);
-
+const char *peas_plugin_info_get_help_uri (const PeasPluginInfo *info);
PEAS_AVAILABLE_IN_ALL
-const gchar *peas_plugin_info_get_external_data (const PeasPluginInfo *info,
- const gchar *key);
+const char *peas_plugin_info_get_external_data (const PeasPluginInfo *info,
+ const char *key);
G_END_DECLS