[MPlayer-dev-eng] [GUI][PATCH] Import Playtree into Gui-PL;

[Arpi]

Hi,

> +      
> +      if (!enqueue)
> +      {
> +	gtkSet(gtkDelPl,0,0);
> +      }
> +      // Import playtree into gui
> +      import_playtree_into_gui(playtree, mconfig, 0); // Just add the items to end of list

why not pass 'enqueue' to import_playtree_into_gui as a parameter and let it
to do gtkSet(gtkDelPl,0,0); instead of messing in mplayer.c with gtk???

>    }
> +
> + if (WinID>0)
> + {
> +   appMPlayer.subWindow.Parent=WinID;
> +   appMPlayer.mainWindow.Parent=WinID;
> +   appMPlayer.sub.x=0;
> +   appMPlayer.sub.y=0;
> + }
>   
>   wsCreateWindow( &appMPlayer.subWindow,
>    appMPlayer.sub.x,appMPlayer.sub.y,appMPlayer.sub.width,appMPlayer.sub.height,
> @@ -509,6 +517,8 @@
>  	   }
>  	  guiIntfStruct.MovieWidth=vo_dwidth;
>  	  guiIntfStruct.MovieHeight=vo_dheight;
> +	  if (WinID>0)
> +	    wsMoveWindow( &appMPlayer.mainWindow,0,0, vo_dheight);
>           }

i have serious doubts that these are related to playtree...

>  	 }
> -	if ( guiIntfStruct.StreamType != STREAMTYPE_PLAYLIST )
> +	//if ( guiIntfStruct.StreamType != STREAMTYPE_PLAYLIST ) // Does not make problems anymore!

why?


A'rpi / Astral & ESP-team

--
Developer of MPlayer, the Movie Player for Linux - http://www.MPlayerHQ.hu
    "However, many people beg for its inclusion in Debian. Why?" - Gabucino
  "Because having new software in Debian is good." - Josselin Mouette
"Because having good software in Debian is new." - Gabucino