Re: patch to append files to playlist

Morten Nilsen <[email protected]>
Newsgroups gmane.comp.multimedia.xmms.devel
Message-ID <[email protected]>
Chris Hellberg wrote:
> Fair enough. Attached is the rehash without the whitespace.
> 
> --- xmms-1.2.10-orig/xmms/main.c	Thu Dec  2 09:39:24 2004
> +++ xmms-1.2.10/xmms/main.c	Fri Dec  3 08:31:11 2004
> @@ -2415,9 +2420,14 @@
>  		break;
>  	case MAINWIN_OPT_NPA:
>  		cfg.no_playlist_advance = GTK_CHECK_MENU_ITEM(gtk_item_factory_get_widget(mainwin_options_menu, "/No Playlist Advance"))->active;
> +		break;
> +	case MAINWIN_OPT_ATP:
> +		cfg.append_to_playlist = GTK_CHECK_MENU_ITEM(gtk_item_factory_get_widget(mainwin_options_menu, "/Append to playlist"))->active;
> +		cfg.append_to_playlist = TRUE;

I don't know the xmms source, but this seems weird to me..
won't setting cfg.append_to_playlist to TRUE explicitly force this
option on?
you said the option won't work until reloading config, could this be the
reason?

>  	}
>  }
>  
> +

missed one ;)

>  void mainwin_vis_menu_callback(gpointer cb_data, guint action, GtkWidget * w)
>  {
>  	switch (action)

Cheers,
-- 
Morten
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.