Commit 533af175 authored by Rémi Denis-Courmont's avatar Rémi Denis-Courmont

- pp_shortcuts is always true; do not check

- remove apparently harmful "continue" with an always true condition
parent da2a148c
...@@ -1376,15 +1376,13 @@ static void Usage( libvlc_int_t *p_this, char const *psz_module_name ) ...@@ -1376,15 +1376,13 @@ static void Usage( libvlc_int_t *p_this, char const *psz_module_name )
if( psz_module_name && strcmp( psz_module_name, if( psz_module_name && strcmp( psz_module_name,
p_parser->psz_object_name ) ) p_parser->psz_object_name ) )
{ {
const char **pp_shortcut = p_parser->pp_shortcuts; const char *const *pp_shortcut = p_parser->pp_shortcuts;
while( *pp_shortcut ) while( *pp_shortcut )
{ {
if( !strcmp( psz_module_name, *pp_shortcut ) ) if( !strcmp( psz_module_name, *pp_shortcut ) )
break; break;
pp_shortcut ++; pp_shortcut ++;
} }
if( !*pp_shortcut )
continue;
} }
/* Ignore modules without config options */ /* Ignore modules without config options */
...@@ -1770,7 +1768,7 @@ static void ListModules( libvlc_int_t *p_this, vlc_bool_t b_verbose ) ...@@ -1770,7 +1768,7 @@ static void ListModules( libvlc_int_t *p_this, vlc_bool_t b_verbose )
if( b_verbose ) if( b_verbose )
{ {
const char **pp_shortcut = p_parser->pp_shortcuts; const char **pp_shortcut = p_parser->pp_shortcuts;
while( pp_shortcut && *pp_shortcut ) while( *pp_shortcut )
{ {
if( strcmp( *pp_shortcut, p_parser->psz_object_name ) ) if( strcmp( *pp_shortcut, p_parser->psz_object_name ) )
{ {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment