[MPlayer-dev-eng] [PATCH] Fix "initialization discards qualifiers" warnings in cfg-*.h

Diego Biurrun diego at biurrun.de
Mon Sep 27 11:37:28 CEST 2010


On Sat, Sep 11, 2010 at 03:49:51PM +0200, Clément Bœsch wrote:
> > The previous patch fixes the cfg-*.h warnings, but creates a few other in
> > the config parser. So here is a new patch adding fixing the related
> > constness issues in parser.
> > 
> > I also attached a diff of the warnings between the two versions (with and
> > without the patch).
> > 
> 
> Diff f*cked up in previous mail, sorry. Fixed in this mail.
> 
> --- m_config.c	(revision 32153)
> +++ m_config.c	(working copy)
> @@ -279,7 +279,7 @@
>  }
>  
>  static m_config_option_t*
> -m_config_get_co(m_config_t *config, char* arg) {
> +m_config_get_co(const m_config_t *config, char* arg) {
>    m_config_option_t *co;
>  
>    for(co = config->opts ; co ; co = co->next ) {
> @@ -295,7 +295,7 @@
>  }
>  
>  static int
> -m_config_parse_option(m_config_t *config, char* arg, char* param,int set) {
> +m_config_parse_option(const m_config_t *config, char* arg, char* param,int set) {
>    m_config_option_t *co;
>    int r = 0;
>  
> @@ -386,7 +386,7 @@
>  }
>  
>  int
> -m_config_check_option(m_config_t *config, char* arg, char* param) {
> +m_config_check_option(const m_config_t *config, char* arg, char* param) {
>    int r;
>    mp_msg(MSGT_CFGPARSER, MSGL_DBG2,"Checking %s=%s\n",arg,param);
>    r=m_config_parse_option(config,arg,param,0);
> @@ -399,7 +399,7 @@
>  
>  
>  const m_option_t*
> -m_config_get_option(m_config_t *config, char* arg) {
> +m_config_get_option(const m_config_t *config, char* arg) {
>    m_config_option_t *co;
>  
>  #ifdef MP_DEBUG
> @@ -417,7 +417,7 @@
>  
>  
>  void
> -m_config_print_option_list(m_config_t *config) {
> +m_config_print_option_list(const m_config_t *config) {
>    char min[50],max[50];
>    m_config_option_t* co;
>    int count = 0;
> @@ -450,7 +450,7 @@
>  }
>  
>  m_profile_t*
> -m_config_get_profile(m_config_t* config, char* name) {
> +m_config_get_profile(const m_config_t* config, char* name) {
>    m_profile_t* p;
>    for(p = config->profiles ; p ; p = p->next)
>      if(!strcmp(p->name,name)) return p;

This looks correct.  Have you checked whether it adds warnings when
applied without the other part?

Diego


More information about the MPlayer-dev-eng mailing list