search

cDima / Aerial

Aerial Apple TV screen saver for Windows
1.96k stars 180 forks source link

Fix repeat calls to SetNextVideo() #45

Closed alistairmcmillan closed 8 years ago

alistairmcmillan commented 8 years ago

With these lines included every time the screensaver transitions from one movie to another SetNextVideo() is called three times.

Crude debugging but, with these lines you get something like this...

17/12/2015 00:03:19 NextVideoTimer_Tick - current is wmppsPlaying
17/12/2015 00:03:19 player_PlayStateChange - current is wmppsMediaEnded, new state: wmppsMediaEnded
17/12/2015 00:03:19 player_PlayStateChange - current is wmppsTransitioning, new state: wmppsTransitioning
17/12/2015 00:03:19 player_PlayStateChange - current is wmppsStopped, new state: wmppsStopped
17/12/2015 00:03:19 SetNextVideo
17/12/2015 00:03:19 Playing from C:\Users\alistair\AppData\Local\Aerial\b1-2.mov
17/12/2015 00:03:19 player_PlayStateChange - current is wmppsTransitioning, new state: wmppsTransitioning
17/12/2015 00:03:19 player_PlayStateChange - current is wmppsReady, new state: wmppsReady
17/12/2015 00:03:19 player_PlayStateChange - current is wmppsTransitioning, new state: wmppsTransitioning
17/12/2015 00:03:20 player_PlayStateChange - current is wmppsStopped, new state: wmppsStopped
17/12/2015 00:03:20 SetNextVideo
17/12/2015 00:03:20 Playing from C:\Users\alistair\AppData\Local\Aerial\b4-2.mov
17/12/2015 00:03:20 player_PlayStateChange - current is wmppsTransitioning, new state: wmppsTransitioning
17/12/2015 00:03:20 player_PlayStateChange - current is wmppsReady, new state: wmppsReady
17/12/2015 00:03:20 player_PlayStateChange - current is wmppsTransitioning, new state: wmppsTransitioning
17/12/2015 00:03:20 player_PlayStateChange - current is wmppsReady, new state: wmppsReady
17/12/2015 00:03:20 NextVideoTimer_Tick - current is wmppsReady
17/12/2015 00:03:20 SetNextVideo
17/12/2015 00:03:20 Playing from C:\Users\alistair\AppData\Local\Aerial\b7-3.mov
17/12/2015 00:03:20 player_PlayStateChange - current is wmppsTransitioning, new state: wmppsTransitioning
17/12/2015 00:03:20 player_PlayStateChange - current is wmppsTransitioning, new state: wmppsTransitioning
17/12/2015 00:03:20 player_PlayStateChange - current is wmppsPlaying, new state: wmppsPlaying
17/12/2015 00:03:21 NextVideoTimer_Tick - current is wmppsPlaying

If you take them out, SetNextVideo is only called once each time...

17/12/2015 00:55:09 NextVideoTimer_Tick - current is wmppsPlaying
17/12/2015 00:55:09 player_PlayStateChange - current is wmppsMediaEnded, new state: wmppsMediaEnded
17/12/2015 00:55:09 player_PlayStateChange - current is wmppsTransitioning, new state: wmppsTransitioning
17/12/2015 00:55:09 player_PlayStateChange - current is wmppsStopped, new state: wmppsStopped
17/12/2015 00:55:10 NextVideoTimer_Tick - current is wmppsStopped
17/12/2015 00:55:10 SetNextVideo
17/12/2015 00:55:10 Playing from C:\Users\alistair\AppData\Local\Aerial\b5-3.mov
17/12/2015 00:55:10 player_PlayStateChange - current is wmppsTransitioning, new state: wmppsTransitioning
17/12/2015 00:55:10 player_PlayStateChange - current is wmppsReady, new state: wmppsReady
17/12/2015 00:55:10 player_PlayStateChange - current is wmppsTransitioning, new state: wmppsTransitioning
17/12/2015 00:55:10 player_PlayStateChange - current is wmppsPlaying, new state: wmppsPlaying
17/12/2015 00:55:11 NextVideoTimer_Tick - current is wmppsPlaying
mzzlaniz commented 8 years ago

aliatairmcmillan

mzzlaniz commented 8 years ago

cDima merged commit 2a896a5 into master about 11 hours ago