On 2/11/2020 6:03 AM, Richard Cochran wrote:
> The two methods, config_create_interface and config_read, never modify the
> strings passed in.  This patch adds the const keyword to ensure these
> functions stay that way.
> 


Makes sense. Using const in more places like this is great! It helps
make behavior explicit.

> Signed-off-by: Richard Cochran <richardcoch...@gmail.com>

Reviewed-by: Jacob Keller <jacob.e.kel...@intel.com>

> ---
>  config.c | 4 ++--
>  config.h | 4 ++--
>  2 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/config.c b/config.c
> index 12eb1f9..65afa70 100644
> --- a/config.c
> +++ b/config.c
> @@ -710,7 +710,7 @@ static struct option *config_alloc_longopts(void)
>       return opts;
>  }
>  
> -int config_read(char *name, struct config *cfg)
> +int config_read(const char *name, struct config *cfg)
>  {
>       enum config_section current_section = UNKNOWN_SECTION;
>       enum parser_result parser_res;
> @@ -817,7 +817,7 @@ parse_error:
>       return -2;
>  }
>  
> -struct interface *config_create_interface(char *name, struct config *cfg)
> +struct interface *config_create_interface(const char *name, struct config 
> *cfg)
>  {
>       struct interface *iface;
>  
> diff --git a/config.h b/config.h
> index f237fb2..e27d3e2 100644
> --- a/config.h
> +++ b/config.h
> @@ -60,8 +60,8 @@ struct config {
>       STAILQ_HEAD(ucmtab_head, unicast_master_table) unicast_master_tables;
>  };
>  
> -int config_read(char *name, struct config *cfg);
> -struct interface *config_create_interface(char *name, struct config *cfg);
> +int config_read(const char *name, struct config *cfg);
> +struct interface *config_create_interface(const char *name, struct config 
> *cfg);
>  void config_destroy(struct config *cfg);
>  
>  /* New, hash table based methods: */
> 


_______________________________________________
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel

Reply via email to