Thread: [ANNOUNCE] Alpha release of Open Media Player
View Single Post
gidzzz's Avatar
gidzzz is offline gidzzz
2012-08-16 , 17:49
Posts: 282 | Thanked: 2,387 times | Joined on Sep 2011
#1221
Originally Posted by pali View Post
I really suggest to remove old hack code. In merge request is only deleted function updateWiredHeadset which call amixer, nothing more.
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?



Originally Posted by romanianusa View Post
The Entertainment View is the ONLY view that attract me to this prog...otherwise i am going back to Instinctiv. Thx for your time.
I consider Entertainment View merely a small addition to OMP, so it doesn't get updates frequently. I think you should go back to Instinctiv, but you can keep checking now and then if anything has changed.
__________________
My Thumb mini-repository: http://gidzzz.mooo.com/maemo/.
 

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