Nikolay Sivov : mf: Simplify clock notification handling.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: afe4b134cf0b6c4930c6138cc76888f74e94389a
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=afe4b134cf0b6c4930c6138cc...

Author: Nikolay Sivov <nsivov(a)codeweavers.com>
Date:   Mon Jan 27 08:51:59 2020 +0300

mf: Simplify clock notification handling.

Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard(a)winehq.org>

---

 dlls/mf/session.c | 30 ++++++++++++------------------
 1 file changed, 12 insertions(+), 18 deletions(-)

diff --git a/dlls/mf/session.c b/dlls/mf/session.c
index 7a49401d83..1f790a6ed3 100644
--- a/dlls/mf/session.c
+++ b/dlls/mf/session.c
@@ -1976,6 +1976,13 @@ static HRESULT clock_change_state(struct presentation_clock *clock, enum clock_c
         /* CLOCK_CMD_PAUSE    */ MFCLOCK_STATE_PAUSED,
         /* CLOCK_CMD_SET_RATE */ 0, /* Unused */
     };
+    static const enum clock_notification notifications[CLOCK_CMD_MAX] =
+    {
+        /* CLOCK_CMD_START    */ CLOCK_NOTIFY_START,
+        /* CLOCK_CMD_STOP     */ CLOCK_NOTIFY_STOP,
+        /* CLOCK_CMD_PAUSE    */ CLOCK_NOTIFY_PAUSE,
+        /* CLOCK_CMD_SET_RATE */ CLOCK_NOTIFY_SET_RATE,
+    };
     enum clock_notification notification;
     struct clock_sink *sink;
     IUnknown *notify_object;
@@ -1994,26 +2001,13 @@ static HRESULT clock_change_state(struct presentation_clock *clock, enum clock_c
 
     system_time = MFGetSystemTime();
 
-    switch (command)
+    if (command == CLOCK_CMD_START && clock->state == MFCLOCK_STATE_PAUSED &&
+            param.u.offset == PRESENTATION_CURRENT_POSITION)
     {
-        case CLOCK_CMD_START:
-            if (clock->state == MFCLOCK_STATE_PAUSED && param.u.offset == PRESENTATION_CURRENT_POSITION)
-                notification = CLOCK_NOTIFY_RESTART;
-            else
-                notification = CLOCK_NOTIFY_START;
-            break;
-        case CLOCK_CMD_STOP:
-            notification = CLOCK_NOTIFY_STOP;
-            break;
-        case CLOCK_CMD_PAUSE:
-            notification = CLOCK_NOTIFY_PAUSE;
-            break;
-        case CLOCK_CMD_SET_RATE:
-            notification = CLOCK_NOTIFY_SET_RATE;
-            break;
-        default:
-            ;
+        notification = CLOCK_NOTIFY_RESTART;
     }
+    else
+        notification = notifications[command];
 
     if (FAILED(hr = clock_call_state_change(system_time, param, notification, clock->time_source_sink)))
         return hr;