[MPlayer-dev-eng] [patch] added const for strings

Rich Felker dalias at aerifal.cx
Thu Jun 22 04:55:43 CEST 2006


On Wed, Jun 21, 2006 at 08:00:03PM +0200, Stefan Huehner wrote:
> Hi,
> 
> attached patch adds const to string-constants in:
> mp3lib/sr1.c
> libvo/font_load.c
> postproc/swscale.c
> 
> and adds the const modifier to read-only string parameters of functions
> in:
> libvo/vo_caca.c
> libvo/vo_aa.c
> 
> Regards,
> Stefan
> 

> Index: mp3lib/sr1.c
> ===================================================================
> --- mp3lib/sr1.c	(revision 18777)
> +++ mp3lib/sr1.c	(working copy)
> @@ -531,8 +531,8 @@
>  
>  // Prints last frame header in ascii.
>  void MP3_PrintHeader(void){
> -        static char *modes[4] = { "Stereo", "Joint-Stereo", "Dual-Channel", "Single-Channel" };
> -        static char *layers[4] = { "???" , "I", "II", "III" };
> +        static const char *modes[4] = { "Stereo", "Joint-Stereo", "Dual-Channel", "Single-Channel" };
> +        static const char *layers[4] = { "???" , "I", "II", "III" };
>  
>          mp_msg(MSGT_DECAUDIO,MSGL_V,"\rMPEG %s, Layer %s, %ld Hz %d kbit %s, BPF: %ld\n",
>                  fr.mpeg25 ? "2.5" : (fr.lsf ? "2.0" : "1.0"),
> Index: libvo/font_load.c
> ===================================================================
> --- libvo/font_load.c	(revision 18777)
> +++ libvo/font_load.c	(working copy)
> @@ -161,7 +161,7 @@
>    } else    
>  
>    if(strcmp(section,"[files]")==0){
> -      char *default_dir=MPLAYER_DATADIR "/font";
> +      const char *default_dir=MPLAYER_DATADIR "/font";

Things like this should be:

static const char default_dir[] = MPLAYER_DATADIR "/font";

It saves the wasted space for storing a pointer to a constant address.
Yes I'm being pedantic but your patch is pedantic. So if you want
pedantry, do it right. :)

Rich




More information about the MPlayer-dev-eng mailing list