Skip to content

Commit

Permalink
[clean-up] Remove more const stuff
Browse files Browse the repository at this point in the history
Avoid unneeded casting.

Signed-off-by: Philippe De Swert <philippe.deswert@jollamobile.com>
  • Loading branch information
philippedeswert committed Nov 7, 2014
1 parent c2fe327 commit 0cb508b
Show file tree
Hide file tree
Showing 5 changed files with 20 additions and 20 deletions.
4 changes: 2 additions & 2 deletions src/usb_moded-config-private.h
Expand Up @@ -23,7 +23,7 @@

/*============================================================================= */

const char * get_mode_setting(void);
char * get_mode_setting(void);
int set_mode_setting(const char *mode);
int set_network_setting(const char *config, const char *setting);
const char *get_network_setting(const char *config);
char *get_network_setting(const char *config);
24 changes: 12 additions & 12 deletions src/usb_moded-config.c
Expand Up @@ -85,12 +85,12 @@ const char * find_alt_mount(void)
return(get_conf_string(ALT_MOUNT_ENTRY, ALT_MOUNT_KEY));
}

const char * find_udev_path(void)
char * find_udev_path(void)
{
return(get_conf_string(UDEV_PATH_ENTRY, UDEV_PATH_KEY));
}

const char * find_udev_subsystem(void)
char * find_udev_subsystem(void)
{
return(get_conf_string(UDEV_PATH_ENTRY, UDEV_SUBSYSTEM_KEY));
}
Expand Down Expand Up @@ -333,9 +333,9 @@ static char * get_kcmdline_string(const char *entry)
return(ret);
}

const char * get_mode_setting(void)
char * get_mode_setting(void)
{
const char * mode = get_kcmdline_string(MODE_SETTING_KEY);
char * mode = get_kcmdline_string(MODE_SETTING_KEY);
if (mode != NULL)
return(mode);

Expand Down Expand Up @@ -418,9 +418,9 @@ int set_network_setting(const char *config, const char *setting)
return(!ret);
}

const char * get_network_setting(const char *config)
char * get_network_setting(const char *config)
{
const char * ret = 0;
char * ret = 0;
struct mode_list_elem *data;

if(!strcmp(config, NETWORK_IP_KEY))
Expand Down Expand Up @@ -457,12 +457,12 @@ int conf_file_merge(void)
{
GDir *confdir;
struct stat fileinfo, dir;
const gchar *filename, *mode = 0, *ip = 0, *gateway = 0;
char *mode = 0, *ip = 0, *gateway = 0, *udev = 0;
gchar *filename_full;
const gchar *filename;
GString *keyfile_string = NULL;
GKeyFile *settingsfile;
int ret = 0, test = 0, conffile_created = 0;
const gchar *udev = 0;

confdir = g_dir_open(CONFIG_FILE_DIR, 0, NULL);
if(!confdir)
Expand Down Expand Up @@ -559,13 +559,13 @@ int conf_file_merge(void)
ret = 1;
cleanup:
if(mode)
free((void *)mode);
free(mode);
if(udev)
free((void *)udev);
free(udev);
if(ip)
free((void *)ip);
free(ip);
if(gateway)
free((void *)gateway);
free(gateway);

end:
g_dir_close(confdir);
Expand Down
4 changes: 2 additions & 2 deletions src/usb_moded-config.h
Expand Up @@ -60,8 +60,8 @@ const char * find_mounts(void);
int find_sync(void);
const char * find_alt_mount(void);

const char * find_udev_path(void);
const char * find_udev_subsystem(void);
char * find_udev_path(void);
char * find_udev_subsystem(void);

#ifdef NOKIA
const char * find_cdrom_path(void);
Expand Down
6 changes: 3 additions & 3 deletions src/usb_moded-dbus.c
Expand Up @@ -155,7 +155,7 @@ static DBusHandlerResult msg_handler(DBusConnection *const connection, DBusMessa
else if(!strcmp(member, USB_MODE_NETWORK_GET))
{
char *config = 0;
const char *setting = 0;
char *setting = 0;
DBusError err = DBUS_ERROR_INIT;

if(!dbus_message_get_args(msg, &err, DBUS_TYPE_STRING, &config, DBUS_TYPE_INVALID))
Expand All @@ -177,11 +177,11 @@ static DBusHandlerResult msg_handler(DBusConnection *const connection, DBusMessa
}
else if(!strcmp(member, USB_MODE_CONFIG_GET))
{
const char *config = get_mode_setting();
char *config = get_mode_setting();

if((reply = dbus_message_new_method_return(msg)))
dbus_message_append_args (reply, DBUS_TYPE_STRING, &config, DBUS_TYPE_INVALID);
free((void *)config);
free(config);
}
else if(!strcmp(member, USB_MODE_LIST))
{
Expand Down
2 changes: 1 addition & 1 deletion src/usb_moded-udev.c
Expand Up @@ -63,7 +63,7 @@ static void notify_issue (gpointer data)

gboolean hwal_init(void)
{
const gchar *udev_path = NULL, *udev_subsystem = NULL;
char *udev_path = NULL, *udev_subsystem = NULL;
struct udev_device *dev;
int ret = 0;

Expand Down

0 comments on commit 0cb508b

Please sign in to comment.