[MPlayer-dev-eng] [PATCH] GUI: Enhancement to r35555

Hans-Dieter Kosch hdkosch at kabelbw.de
Mon Dec 10 18:36:40 CET 2012


Ingo Brückl wrote:
> Hans-Dieter Kosch wrote on Sun, 09 Dec 2012 21:48:24 +0100:
> 
>> Ingo Brückl wrote:
>>> Hans-Dieter Kosch wrote on Sat, 08 Dec 2012 23:31:27 +0100:
>>>
>>>> If the current file is still present in the list, playback continues with
>>>> adjusted position.
>>> I've considered this in r35555, but we don't know anything about the files
>>> before the present one. These might have changed, too, so it's best to
>>> restart playback, because I don't think it's worth to check the list of file
>>> before it. (The reason for r35555 was to allow adding to the playlist while
>>> playing, not to change it at this stage.)
>>>
>> This was also my concern at first. However, since the playlist is
>> completely recreated upon selecting 'OK', we are always in sync, and it
>> works flawlessly. You can edit anything in the list, no issue will arise.
>> Or did I miss something?
> 
> No, there is no problem with the list or on-the-fly editing.
> 
> It's just that we don't know what the user meant with the editing if the
> position has changed (inserting an item, or delete one and/or exchanging some
> before the current one). To avoid extensive analysis that probably won't help
> us anyway, we simply restart. It's not worth the effort.
> 
Yes, the user is always the unknown being to us programmers ;-)

But I don't see any problem. I think it's just intuitive and logical; a user 
should be smart enough (and it's only those two lines of code).

The behaviour would be exactly the same like editing the program list of a TV 
receiver: The current program continues playing with adjusted program number.

And we could do even more: If the item at the current track number has changed, 
restart playing of this track number with the new item, just like TV receivers 
do; only go back to track 1 if current track number does no longer exist (i.e. 
we're beyond end of list; if track 1 does not exist, list is empty, but this 
case is handled anyway). Such a change should take only few lines of code.

Hans-Dieter


More information about the MPlayer-dev-eng mailing list