[MPlayer-cvslog] r24444 - in trunk: input/input.c libmpdemux/demux_ts.c libvo/mga_common.c playtreeparser.c

[diego]

Author: diego
Date: Thu Sep 13 15:16:30 2007
New Revision: 24444

Log:
warning fixes:
input.c: In function 'mp_input_set_section':
input.c:1640: warning: suggest parentheses around assignment used as truth value
input.c:1643: warning: suggest parentheses around assignment used as truth value
mga_common.c: In function 'mga_init':
mga_common.c:394: warning: suggest parentheses around assignment used as truth value
playtreeparser.c: In function 'parse_smil':
playtreeparser.c:523: warning: suggest parentheses around assignment used as truth value
libmpdemux/demux_ts.c: In function 'ts_parse':
libmpdemux/demux_ts.c:2795: warning: suggest parentheses around assignment used as truth value
libmpdemux/demux_ts.c: In function 'demux_open_ts':
libmpdemux/demux_ts.c:591: warning: 'frame_length' may be used uninitialized in this function


Modified:
   trunk/input/input.c
   trunk/libmpdemux/demux_ts.c
   trunk/libvo/mga_common.c
   trunk/playtreeparser.c

Modified: trunk/input/input.c
==============================================================================
--- trunk/input/input.c	(original)
+++ trunk/input/input.c	Thu Sep 13 15:16:30 2007
@@ -1637,10 +1637,10 @@ mp_input_set_section(char *name) {
   cmd_binds_default=NULL;
   if(section) free(section);
   if(name) section=strdup(name); else section=strdup("default");
-  if(bind_section=mp_input_get_bind_section(section))
+  if((bind_section=mp_input_get_bind_section(section)))
     cmd_binds=bind_section->cmd_binds;
   if(strcmp(section,"default")==0) return;
-  if(bind_section=mp_input_get_bind_section(NULL))
+  if((bind_section=mp_input_get_bind_section(NULL)))
     cmd_binds_default=bind_section->cmd_binds;
 }
 

Modified: trunk/libmpdemux/demux_ts.c
==============================================================================
--- trunk/libmpdemux/demux_ts.c	(original)
+++ trunk/libmpdemux/demux_ts.c	Thu Sep 13 15:16:30 2007
@@ -2792,7 +2792,7 @@ static int ts_parse(demuxer_t *demuxer ,
 
 						if(dvdsub_lang)
 						{
-							if(lang = pid_lang_from_pmt(priv, pid))
+							if ((lang = pid_lang_from_pmt(priv, pid)))
 								asgn = (strncmp(lang, dvdsub_lang, 3) == 0);
 						}
 						else		//no language specified with -slang

Modified: trunk/libvo/mga_common.c
==============================================================================
--- trunk/libvo/mga_common.c	(original)
+++ trunk/libvo/mga_common.c	Thu Sep 13 15:16:30 2007
@@ -391,7 +391,7 @@ static int mga_init(int width,int height
 	{
 		// try whether we have a G550
 		int ret;
-		if(ret = ioctl(f,MGA_VID_CONFIG,&mga_vid_config))
+		if ((ret = ioctl(f,MGA_VID_CONFIG,&mga_vid_config)))
 		{
 			if(mga_vid_config.card_type != MGA_G550)
 			{

Modified: trunk/playtreeparser.c
==============================================================================
--- trunk/playtreeparser.c	(original)
+++ trunk/playtreeparser.c	Thu Sep 13 15:16:30 2007
@@ -520,7 +520,7 @@ parse_smil(play_tree_parser_t* p) {
     pos = line;
    while (pos) {
     if (!entrymode) { // all entries filled so far 
-     while (pos=strchr(pos, '<')) {
+     while ((pos=strchr(pos, '<'))) {
       if (strncasecmp(pos,"<video",6)==0  || strncasecmp(pos,"<audio",6)==0 || strncasecmp(pos,"<media",6)==0) {
           entrymode=1;
           break; // Got a valid tag, exit '<' search loop