From 33d8bae70bc2dd2d674675b7cb3b2d132887e373 Mon Sep 17 00:00:00 2001
From: lx-s <lx-s@users.noreply.github.com>
Date: Thu, 17 Jul 2014 13:50:46 +0200
Subject: [PATCH] Fixed muting behaviour with Windows' erratic screenshot
 behaviour

---
 Build/WinMute.nuspec            |  2 +-
 Build/bin/changelog.txt         |  5 ++++
 Build/bin/liesmich.html         |  2 +-
 Build/bin/readme.html           |  2 +-
 WinMute/ScreensaverNotifier.cpp | 51 ++++++++++++++++++++++++---------
 WinMute/ScreensaverNotifier.h   |  4 +++
 WinMute/res.rc                  | 22 +++++++-------
 7 files changed, 61 insertions(+), 27 deletions(-)

diff --git a/Build/WinMute.nuspec b/Build/WinMute.nuspec
index 4dffc8d..08154c3 100644
--- a/Build/WinMute.nuspec
+++ b/Build/WinMute.nuspec
@@ -4,7 +4,7 @@
   <metadata>
     <id>winmute</id>
     <title>WinMute</title>
-    <version>1.4.0.20140710</version>
+    <version>1.4.1.20140717</version>
     <authors>Alexander Steinhöfer</authors>
     <owners>Alexander Steinhöfer</owners>
     <summary>Automatic sound muter</summary>
diff --git a/Build/bin/changelog.txt b/Build/bin/changelog.txt
index 9a27f75..fb6c57d 100644
--- a/Build/bin/changelog.txt
+++ b/Build/bin/changelog.txt
@@ -1,3 +1,8 @@
+Version 1.4.1 (2014-07-17)
+
+ * Fixed screenshot muting: It sometimes happened that WinMute 
+   muted Windows Audio for no apparent reason and "forgot" to
+   unmute. This is now fixed.
 
 Version 1.4 (2014-07-10)
 
diff --git a/Build/bin/liesmich.html b/Build/bin/liesmich.html
index 6232b39..9b96bb4 100644
--- a/Build/bin/liesmich.html
+++ b/Build/bin/liesmich.html
@@ -2,7 +2,7 @@
 <html lang="de">
 <head>
    <meta charset="utf-8">
-   <title>WinMute 1.4 - Readme</title>
+   <title>WinMute 1.4.1 - Readme</title>
 
    <style type="text/css">
    <!--
diff --git a/Build/bin/readme.html b/Build/bin/readme.html
index 7cb5b0c..9e1acbe 100644
--- a/Build/bin/readme.html
+++ b/Build/bin/readme.html
@@ -2,7 +2,7 @@
 <html lang="en">
 <head>
    <meta charset="utf-8">
-   <title>WinMute 1.4 - Readme</title>
+   <title>WinMute 1.4.1 - Readme</title>
 
    <style type="text/css">
    <!--
diff --git a/WinMute/ScreensaverNotifier.cpp b/WinMute/ScreensaverNotifier.cpp
index 0adee9c..5bfa77d 100644
--- a/WinMute/ScreensaverNotifier.cpp
+++ b/WinMute/ScreensaverNotifier.cpp
@@ -76,6 +76,7 @@ ScreensaverNotifier::ScreensaverNotifier() :
    hookDll_(nullptr),
    regHook_(nullptr),
    unregHook_(nullptr),
+   alreadyNotified_(false),
    hookWndMsg_(0)
 { }
 
@@ -204,6 +205,23 @@ void ScreensaverNotifier::StartScreensaverPollTimer(bool start)
    }
 }
 
+/* ========================================================================== */
+/*    Other                                                                   */
+/* ========================================================================== */
+
+bool ScreensaverNotifier::IsScreensaverRunning()
+{
+   BOOL isRunning = FALSE;
+   if (SystemParametersInfo(SPI_GETSCREENSAVERRUNNING,
+                            0,
+                            &isRunning,
+                            FALSE) == NULL) {
+      PrintWindowsError(_T("SystemParametersInfo"));
+      return false;
+   }
+   return isRunning != 0;
+}
+
 /* ========================================================================== */
 /*    Window Proc                                                             */
 /* ========================================================================== */
@@ -220,25 +238,32 @@ LRESULT CALLBACK ScreensaverNotifier::WindowProc(HWND hWnd,
    }
    case WM_TIMER: {
       if (wParam == SCRSV_TIMER_ID) {
-         BOOL isRunning = FALSE;
-         if (SystemParametersInfo(SPI_GETSCREENSAVERRUNNING,
-                                  0,
-                                  &isRunning,
-                                  FALSE) == NULL) {
-            PrintWindowsError(_T("SystemParametersInfo"));
-         } else {
-            if (!isRunning) {
-               StartScreensaverPollTimer(false);
-               SendMessage(hNotifyWnd_, WM_SCRNSAVE_CHANGE, SCRNSAVE_STOP, 0);
-            }
+         if (!IsScreensaverRunning()) {
+            StartScreensaverPollTimer(false);
+            SendMessage(hNotifyWnd_, WM_SCRNSAVE_CHANGE, SCRNSAVE_STOP, 0);
+            alreadyNotified_ = false;
          }
       }
       return 0;
    }
    default:
       if (hookWndMsg_ != 0 && msg == hookWndMsg_) {
-         SendMessage(hNotifyWnd_, WM_SCRNSAVE_CHANGE, SCRNSAVE_START, 0);
-         StartScreensaverPollTimer();
+         /* Windows sends SC_SCREENSAVE when it tries to start a screensaver.
+            If screensaver is set to "none" or if another application cancels
+            this event, we would falsely mute windows.
+            To prevent that, we wait for a few milliseconds to and then check
+            if the screensaver has actually started. If yes, then send the
+            notification.
+         */
+         Sleep(500);
+         if (IsScreensaverRunning()) {
+            /* SC_SCREENSAVE might be fired multiple times */
+            if (!alreadyNotified_) {
+               alreadyNotified_ = true;
+               SendMessage(hNotifyWnd_, WM_SCRNSAVE_CHANGE, SCRNSAVE_START, 0);
+               StartScreensaverPollTimer();
+            }
+         }
       }
       break;
    }
diff --git a/WinMute/ScreensaverNotifier.h b/WinMute/ScreensaverNotifier.h
index ef8ffb9..0a37f96 100644
--- a/WinMute/ScreensaverNotifier.h
+++ b/WinMute/ScreensaverNotifier.h
@@ -60,6 +60,8 @@ private:
    typedef int(*RegisterHook)(HWND, UINT);
    typedef void(*UnregisterHook)(void);
 
+   bool alreadyNotified_;
+
 
    HINSTANCE hookDll_;
    HWND hWnd_, hNotifyWnd_;
@@ -70,6 +72,8 @@ private:
 
    void StartScreensaverPollTimer(bool start = true);
 
+   bool IsScreensaverRunning();
+
    bool InitHookDll();
    bool RegisterWindowClass();
    bool InitWindow();
diff --git a/WinMute/res.rc b/WinMute/res.rc
index d1d92aa..e7e73d7 100644
--- a/WinMute/res.rc
+++ b/WinMute/res.rc
@@ -80,7 +80,7 @@ CAPTION "About WinMute"
 FONT 8, "MS Shell Dlg", 400, 0, 0x1
 BEGIN
     DEFPUSHBUTTON   "OK",IDOK,194,7,50,14
-    LTEXT           "WinMute v1.4",IDC_STATIC,43,9,67,11
+    LTEXT           "WinMute v1.4.1",IDC_STATIC,43,9,67,11
     CONTROL         "<a href=""http://lx-s.de"">lx-systems - lx-s.de</a>",IDC_LINK_HOMEPAGE,
                     "SysLink",WS_TABSTOP,43,31,69,13
     LTEXT           "Copyright © 2014 Alexander Steinhöfer",IDC_STATIC,43,20,148,12
@@ -114,8 +114,8 @@ END
 //
 
 VS_VERSION_INFO VERSIONINFO
- FILEVERSION 1,4,0,0
- PRODUCTVERSION 1,4,0,0
+ FILEVERSION 1,4,1,0
+ PRODUCTVERSION 1,4,1,0
  FILEFLAGSMASK 0x17L
 #ifdef _DEBUG
  FILEFLAGS 0x1L
@@ -130,25 +130,25 @@ BEGIN
     BEGIN
         BLOCK "000004b0"
         BEGIN
-            VALUE "CompanyName", "lx-s"
+            VALUE "CompanyName", "lx-systems"
             VALUE "FileDescription", "Automatic sound muter."
-            VALUE "FileVersion", "1.4.0.0"
+            VALUE "FileVersion", "1.4.1.0"
             VALUE "InternalName", "WinMute"
-            VALUE "LegalCopyright", "Copyright (C) 2013 Alexander Steinhöfer"
+            VALUE "LegalCopyright", "Copyright (C) 2014, Alexander Steinhöfer"
             VALUE "OriginalFilename", "WinMute.exe"
             VALUE "ProductName", "WinMute"
-            VALUE "ProductVersion", "1.4.0.0"
+            VALUE "ProductVersion", "1.4.1.0"
         END
         BLOCK "040704b0"
         BEGIN
-            VALUE "CompanyName", "lx-s"
+            VALUE "CompanyName", "lx-systems"
             VALUE "FileDescription", "Automatisches Stummschalten bei PC Sperre."
-            VALUE "FileVersion", "1.4.0.0"
+            VALUE "FileVersion", "1.4.1.0"
             VALUE "InternalName", "WinMute"
-            VALUE "LegalCopyright", "Copyright (C) 2013 Alexander Steinhöfer"
+            VALUE "LegalCopyright", "Copyright (C) 2014, Alexander Steinhöfer"
             VALUE "OriginalFilename", "WinMute.exe"
             VALUE "ProductName", "WinMute"
-            VALUE "ProductVersion", "1.4.0.0"
+            VALUE "ProductVersion", "1.4.1.0"
         END
     END
     BLOCK "VarFileInfo"
-- 
2.43.0