Re: patch to append files to playlist

Cal Peake wrote:
> On Fri, 3 Dec 2004, Chris Hellberg wrote:
> 
>> I don't think this is the reason it's failing, since even after 
>> selecting the option when XMMS is running, it doesn't append files to 
>> the playlist.
> 
> Originally you mentioned that it doesn't work for the first session, it's 
> worst than that. It seems you have to exit and restart if you want a state 
> change in the checkbox to take effect. You prolly need a callback of some 
> such to reflect in the cfg.append_to_playlist variable the current state 
> of the checkbox.

I now know why this is happening (after thinking for a bit)

the config isn't saved until xmms exits.

when the new binary is started, it reads the config and sees the
original state, thus, it assumes not to append, and tells the old
process to replace it's playlist with the specified file(s)

I see 3 solutions;

1) force xmms to save config when checkbox changes

2) move the option away from the dropdown (into prefs) and put a notice
that changes require restart

3) change the architecture such that the enqueue/replace decision is
made in the running xmms, not the new instance.

Cheers,
-- 
Morten