Merge lp://qastaging/~barcc/repeat-one-song/one-second-issue into lp://qastaging/repeat-one-song/trunk

First of all, thanks for you effort to make this plugin better. My first approach when I started it was to call the do_previous with the same kind of "state machine" you did in the callback method on_song_change. Consequently, the erroneous behavior which happens when the user changes a song by himself happened to me, the same one that you pointed out -- and I totally agree with you about that there should be a way to differentiate its sources.

The last second cut off I guess is not a big deal, but I agree it is a problem. The play count can be fixed using rhythmdb, and I'd not realized until the moment, thanks again.

One awesome thing that the do_previous approach (I guess it) has, is the possibility to fix the crossfader problem reported by another user. I was wondering that last night.

Anyway, I'm really restricted about the time, and I will check everything out to answer your merge proposal.

Thanks B Clausius

I found out, how to connect to the end-of-stream signal. I think this fixes all issues mentioned above.

You're right, B Clausius. Anyway, over here, the on_gst_player_eos callback is not feeding the method with 4 parameters, but with 3. I defined 0 to the "early" parameter in order to it works. Works like a charm, congratulations.

Thanks for your efforts. I will change the "early" parameter as I commented and pack it as version 0.0.6.

There is a bug in my code, if the song is the last song in the list, it does not work. To do a "do_previous" there must have been a next song of course. I will look at it.

Nice workaround.

I will pack it in the 0.0.6 and generate a Stable version with only the basic function.