ukanth / monitores

Monitor Energy Saving App for Windows
Apache License 2.0
91 stars 11 forks source link

after unlock, volume should restore previous state, not unmute #36

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Set volume to XX
2. Lock
3. Unlock

[works; volume is restore to XX]

1. Mute volume
2. Lock
3. Unlock

What is the expected output? What do you see instead?
Volume remains muted, as that's how it was before the lock. However, the volume 
is unmuted.

What version of the product are you using? On what operating system?
1.0b, Win7 32bit

Please provide any additional information below.

Original issue reported on code.google.com by mikeage on 9 Mar 2011 at 6:28

GoogleCodeExporter commented 9 years ago
Thanks for reporting. Able to reproduce.I'll fix it.

Original comment by cumakt on 9 Mar 2011 at 6:58

GoogleCodeExporter commented 9 years ago
thanks for the prompt response!

BTW, is the source available? What I've seen here seems to be a bit outdated; 
it seemed like an easy enough fix, and I wanted to be able to send you a patch, 
not just a report.

Original comment by mikeage on 9 Mar 2011 at 7:03

GoogleCodeExporter commented 9 years ago
I almost forgot.I'll update the code today. Thanks for your help.

Original comment by cumakt on 9 Mar 2011 at 11:36

GoogleCodeExporter commented 9 years ago
thanks for the code update, although I think you may have missed resource.h

Original comment by mikeage on 13 Mar 2011 at 6:32

GoogleCodeExporter commented 9 years ago
Oops, my mistake.Updated code.Please let me know, If you would like to join, I 
can add you as a contributor :) 

Original comment by cumakt on 13 Mar 2011 at 7:53

GoogleCodeExporter commented 9 years ago
and log.h? ;)

Original comment by mikeage on 13 Mar 2011 at 10:09

GoogleCodeExporter commented 9 years ago
You know what, I committed the latest version(dev version) which includes all 
tnhose. You don't need log.h(just remove it from functions.cpp)

Original comment by cumakt on 13 Mar 2011 at 10:35

GoogleCodeExporter commented 9 years ago
Just noticed that. Also volume.h is not required. I'll probably be sending you 
a patch soon.

Original comment by mikeage on 13 Mar 2011 at 10:37