diff options
author | Dave Davenport <qball@gmpclient.org> | 2016-03-19 13:29:04 +0100 |
---|---|---|
committer | Dave Davenport <qball@gmpclient.org> | 2016-03-19 13:29:04 +0100 |
commit | a2869ef39d1e78c70877df27dc9a6ebc66962842 (patch) | |
tree | 98715f56cd4792a9c8b68a9933765a113fd37d41 /source/dialogs/dmenu.c | |
parent | 1f8db0fc5a1da29346119937da5fe559469af2bc (diff) |
Make clang static code analyzer happy (300+ -> 5)
Diffstat (limited to 'source/dialogs/dmenu.c')
-rw-r--r-- | source/dialogs/dmenu.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/source/dialogs/dmenu.c b/source/dialogs/dmenu.c index 9add4934..89352bba 100644 --- a/source/dialogs/dmenu.c +++ b/source/dialogs/dmenu.c @@ -123,7 +123,8 @@ static unsigned int dmenu_mode_get_num_entries ( const Mode *sw ) static void parse_pair ( char *input, struct range_pair *item ) { int index = 0; - for ( char *token = strsep ( &input, "-" ); token != NULL; token = strsep ( &input, "-" ) ) { + const char * const sep = "-"; + for ( char *token = strsep ( &input, sep ); token != NULL; token = strsep ( &input, sep ) ) { if ( index == 0 ) { item->start = item->stop = (unsigned int) strtoul ( token, NULL, 10 ); index++; @@ -145,7 +146,8 @@ static void parse_ranges ( char *input, struct range_pair **list, unsigned int * if ( input == NULL ) { return; } - for ( char *token = strtok_r ( input, ",", &endp ); token != NULL; token = strtok_r ( NULL, ",", &endp ) ) { + const char *const sep =","; + for ( char *token = strtok_r ( input, sep, &endp ); token != NULL; token = strtok_r ( NULL, sep, &endp ) ) { // Make space. *list = g_realloc ( ( *list ), ( ( *length ) + 1 ) * sizeof ( struct range_pair ) ); // Parse a single pair. |