View Single Post
Posts: 2,154 | Thanked: 8,464 times | Joined on May 2010
#1226
Originally Posted by gidzzz View Post
I'm OK with removing the hacky code, but all the time I have a feeling that you don't realize one thing. updateWiredHeadset() is indeed responsible for setting jackbias with amixer, but it also takes care of pausing music when you pull the headphone plug out. Removing this means that Disconnect headset to pause and Reconnect headset to resume will work only for Bluetooth headsets.

Aren't QProcess::startDetached("amixer", args << "on"); and its "off" counterpart the only things that really have to be removed?
Sorry, I totally forgot that updateWiredHeadset also handle pause on disconnect. My fault...

I updated merge request, see: https://gitorious.org/qt-mediaplayer...rge_requests/7
 

The Following 6 Users Say Thank You to pali For This Useful Post: