[PATCH v2 3/4] sapi: Implement ISpVoice::Set/GetRate.

[Shaun Ren]

From: Shaun Ren

---
 dlls/sapi/tests/tts.c | 30 ++++++++++++++++++++++++++++++
 dlls/sapi/tts.c       | 26 ++++++++++++++++++++------
 2 files changed, 50 insertions(+), 6 deletions(-)

diff --git a/dlls/sapi/tests/tts.c b/dlls/sapi/tests/tts.c
index e7fe5a96285..c8af2e3d809 100644
--- a/dlls/sapi/tests/tts.c
+++ b/dlls/sapi/tests/tts.c
@@ -100,6 +100,7 @@ static void test_spvoice(void)
     ISpObjectTokenCategory *token_cat;
     ISpObjectToken *token;
     WCHAR *token_id = NULL, *default_token_id = NULL;
+    LONG rate;
     HRESULT hr;

if (waveOutGetNumDevs() == 0) {
@@ -146,6 +147,35 @@ static void test_spvoice(void)
     ISpObjectToken_Release(token);
     ISpObjectTokenCategory_Release(token_cat);

+    rate = 0xdeadbeef;
+    hr = ISpVoice_GetRate(voice, &rate);
+    ok(hr == S_OK, "got %#lx.\n", hr);
+    ok(rate == 0, "rate = %ld\n", rate);
+
+    hr = ISpVoice_SetRate(voice, 1);
+    ok(hr == S_OK, "got %#lx.\n", hr);
+
+    rate = 0xdeadbeef;
+    hr = ISpVoice_GetRate(voice, &rate);
+    ok(hr == S_OK, "got %#lx.\n", hr);
+    ok(rate == 1, "rate = %ld\n", rate);
+
+    hr = ISpVoice_SetRate(voice, -1000);
+    ok(hr == S_OK, "got %#lx.\n", hr);
+
+    rate = 0xdeadbeef;
+    hr = ISpVoice_GetRate(voice, &rate);
+    ok(hr == S_OK, "got %#lx.\n", hr);
+    ok(rate == -1000, "rate = %ld\n", rate);
+
+    hr = ISpVoice_SetRate(voice, 1000);
+    ok(hr == S_OK, "got %#lx.\n", hr);
+
+    rate = 0xdeadbeef;
+    hr = ISpVoice_GetRate(voice, &rate);
+    ok(hr == S_OK, "got %#lx.\n", hr);
+    ok(rate == 1000, "rate = %ld\n", rate);
+
     ISpVoice_Release(voice);
     ISpMMSysAudio_Release(audio_out);
 }
diff --git a/dlls/sapi/tts.c b/dlls/sapi/tts.c
index b7af24790e0..32d3aa31335 100644
--- a/dlls/sapi/tts.c
+++ b/dlls/sapi/tts.c
@@ -43,6 +43,7 @@ struct speech_voice

ISpStreamFormat *output;
     ISpTTSEngine *engine;
+    LONG rate;
     CRITICAL_SECTION cs;
 };

@@ -750,18 +751,30 @@ static HRESULT WINAPI spvoice_GetAlertBoundary(ISpVoice *iface, SPEVENTENUM *bou
     return E_NOTIMPL;
 }

-static HRESULT WINAPI spvoice_SetRate(ISpVoice *iface, LONG adjust)
+static HRESULT WINAPI spvoice_SetRate(ISpVoice *iface, LONG rate)
 {
-    FIXME("(%p, %ld): stub.\n", iface, adjust);
+    struct speech_voice *This = impl_from_ISpVoice(iface);

-    return E_NOTIMPL;
+    TRACE("(%p, %ld).\n", iface, rate);
+
+    EnterCriticalSection(&This->cs);
+    This->rate = rate;
+    LeaveCriticalSection(&This->cs);
+
+    return S_OK;
 }

-static HRESULT WINAPI spvoice_GetRate(ISpVoice *iface, LONG *adjust)
+static HRESULT WINAPI spvoice_GetRate(ISpVoice *iface, LONG *rate)
 {
-    FIXME("(%p, %p): stub.\n", iface, adjust);
+    struct speech_voice *This = impl_from_ISpVoice(iface);

-    return E_NOTIMPL;
+    TRACE("(%p, %p).\n", iface, rate);
+
+    EnterCriticalSection(&This->cs);
+    *rate = This->rate;
+    LeaveCriticalSection(&This->cs);
+
+    return S_OK;
 }

static HRESULT WINAPI spvoice_SetVolume(ISpVoice *iface, USHORT volume)
@@ -930,6 +943,7 @@ HRESULT speech_voice_create(IUnknown *outer, REFIID iid, void **obj)

This->output = NULL;
     This->engine = NULL;
+    This->rate = 0;

InitializeCriticalSection(&This->cs);