thecodrr / BreadPlayer

Bread Player, a free and open source music player powered by UWP and C#/.NET with a sleek and polished design built for, and by, the people seeking a better alternative to Groove and Windows Media Player by Microsoft.
https://theweavrs.github.io/BreadPlayer/
GNU General Public License v3.0
300 stars 84 forks source link

Crash when audio device disabled. #232

Closed JoshGoolnik closed 6 years ago

JoshGoolnik commented 6 years ago

Actual Behaviour:

What Happened?

When the current audio device is disabled, Bread Player crashes.

How it happened?

I was experimenting with changing the audio device during running. I disabled my current device. BreadPlayer crashed.

Expected Behaviour:

What were you expecting?

Some kind of safe handling of this, and some feedback to the user.

Steps to reproduce the bug:

  1. Open BreadPlayer
  2. Open "Manage Audio Devices" dialog via Windows > Start
  3. Disable current audio device from list, by right-clicking and choosing "Disable"

General Information:

  1. Bread Player Version:2.7.8
  2. Device Type: Laptop
  3. OS Version: Windows 10
  4. Device Model:HP Pavillion

Additional Comments:

I have a fix for this bug which I am about to push, in the branch "add-audio-device-disabled-handling"