Mercurial > emacs
changeset 110589:0e7fadbf6ef4
Use const for some pointer arguments.
* lwlib/lwlib.h (my_strcasecmp, safe_strcmp, name_to_widget)
(find_in_table, dialog_spec_p, lw_separator_p):
* lwlib/lwlib.c (my_strcasecmp, safe_strcmp, name_to_widget)
(find_in_table, dialog_spec_p, lw_separator_p): Use const.
| author | Dan Nicolaescu <dann@ics.uci.edu> |
|---|---|
| date | Sun, 26 Sep 2010 18:11:48 +0300 |
| parents | 0995e4af573f |
| children | 21047b3216d2 |
| files | lwlib/ChangeLog lwlib/lwlib.c lwlib/lwlib.h |
| diffstat | 3 files changed, 21 insertions(+), 13 deletions(-) [+] |
line wrap: on
line diff
--- a/lwlib/ChangeLog Sun Sep 26 18:06:21 2010 +0300 +++ b/lwlib/ChangeLog Sun Sep 26 18:11:48 2010 +0300 @@ -1,3 +1,11 @@ +2010-09-26 Dan Nicolaescu <dann@ics.uci.edu> + + Use const for some pointer arguments. + * lwlib.h (my_strcasecmp, safe_strcmp, name_to_widget) + (find_in_table, dialog_spec_p, lw_separator_p): + * lwlib.c (my_strcasecmp, safe_strcmp, name_to_widget) + (find_in_table, dialog_spec_p, lw_separator_p): Use const. + 2010-09-20 Dan Nicolaescu <dann@ics.uci.edu> * lwlib.h (lw_register_widget, lw_create_widget):
--- a/lwlib/lwlib.c Sun Sep 26 18:06:21 2010 +0300 +++ b/lwlib/lwlib.c Sun Sep 26 18:11:48 2010 +0300 @@ -75,7 +75,7 @@ widget_value *, int, int *); static void instantiate_widget_instance (widget_instance *); -static int my_strcasecmp (char *, char *); +static int my_strcasecmp (const char *, const char *); static void safe_free_str (char *); static void free_widget_value_tree (widget_value *); static widget_value *copy_widget_value_tree (widget_value *, @@ -92,14 +92,14 @@ static widget_info *get_widget_info (LWLIB_ID, Boolean); static widget_instance *get_widget_instance (Widget, Boolean); static widget_instance *find_instance (LWLIB_ID, Widget, Boolean); -static Boolean safe_strcmp (char *, char *); -static Widget name_to_widget (widget_instance *, char *); +static Boolean safe_strcmp (const char *, const char *); +static Widget name_to_widget (widget_instance *, const char *); static void set_one_value (widget_instance *, widget_value *, Boolean); static void update_one_widget_instance (widget_instance *, Boolean); static void update_all_widget_values (widget_info *, Boolean); static void initialize_widget_instance (widget_instance *); -static widget_creation_function find_in_table (char *, widget_creation_entry *); -static Boolean dialog_spec_p (char *); +static widget_creation_function find_in_table (const char *, const widget_creation_entry *); +static Boolean dialog_spec_p (const char *); static void destroy_one_instance (widget_instance *); static void lw_pop_all_widgets (LWLIB_ID, Boolean); static Boolean get_one_value (widget_instance *, widget_value *); @@ -120,7 +120,7 @@ /* Like strcmp but ignore differences in case. */ static int -my_strcasecmp (char *s1, char *s2) +my_strcasecmp (const char *s1, const char *s2) { while (1) { @@ -402,7 +402,7 @@ /* utility function for widget_value */ static Boolean -safe_strcmp (char *s1, char *s2) +safe_strcmp (const char *s1, const char *s2) { if (!!s1 ^ !!s2) return True; return (s1 && s2) ? strcmp (s1, s2) : s1 ? False : !!s2; @@ -586,7 +586,7 @@ /* modifying the widgets */ static Widget -name_to_widget (widget_instance *instance, char *name) +name_to_widget (widget_instance *instance, const char *name) { Widget widget = NULL; @@ -729,9 +729,9 @@ static widget_creation_function -find_in_table (char *type, widget_creation_entry *table) +find_in_table (const char *type, const widget_creation_entry *table) { - widget_creation_entry* cur; + const widget_creation_entry* cur; for (cur = table; cur->type; cur++) if (!my_strcasecmp (type, cur->type)) return cur->function; @@ -739,7 +739,7 @@ } static Boolean -dialog_spec_p (char *name) +dialog_spec_p (const char *name) { /* return True if name matches [EILPQeilpq][1-9][Bb] or [EILPQeilpq][1-9][Bb][Rr][1-9] */ @@ -1317,7 +1317,7 @@ to similar ones that are supported. */ int -lw_separator_p (char *label, enum menu_separator *type, int motif_p) +lw_separator_p (const char *label, enum menu_separator *type, int motif_p) { int separator_p = 0;
--- a/lwlib/lwlib.h Sun Sep 26 18:06:21 2010 +0300 +++ b/lwlib/lwlib.h Sun Sep 26 18:11:48 2010 +0300 @@ -186,7 +186,7 @@ MOTIF_P non-zero means map separator types not supported by Motif to similar ones that are supported. */ -int lw_separator_p (char *label, enum menu_separator *type, +int lw_separator_p (const char *label, enum menu_separator *type, int motif_p); #endif /* LWLIB_H */