lastfm / lastfm-windows-desktop

Last.fm Desktop app for Windows (3.0)
GNU General Public License v3.0
10 stars 1 forks source link

WMP: Now Playing notification is shown for last and new track #66

Closed lauranolte closed 6 years ago

lauranolte commented 6 years ago

Setup:

Steps:

Actual behaviour:

Expexted:

Example:

171011_scrobbler_wmp-np
VorTechS commented 6 years ago

This one was weird. Throwing breakpoints in the project was no help at all, as the problem never surfaced under that condition, which pointed to a race condition. I added some conditional compilation arguments into the source to tidy up the debug output and it clearly shows two 'started' notifications being raised.

There was no consistent pattern to it all. It would occur on the first run through of a playlist, on the second run-through but was easily re-producible.

In the end, I modified the WMP timing to 2 seconds instead of one and fixed other bugs that I spotted along the way, and I was able to play a playlist 4 times through without this bug appearing again.

So, I'm declaring this fixed, until you prove me wrong! ;)