Shaun Ren : sapi: Implement ISpeechVoice::{get/putref}_Voice.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 43220a6063328bc445a346e5f6849731aafcaf88
URL:    https://gitlab.winehq.org/wine/wine/-/commit/43220a6063328bc445a346e5f684973...

Author: Shaun Ren 
Date:   Fri Mar  1 20:44:43 2024 -0500

sapi: Implement ISpeechVoice::{get/putref}_Voice.

---

dlls/sapi/tests/tts.c | 22 +++++++++++++++++++++-
 dlls/sapi/tts.c       | 26 ++++++++++++++++++++++----
 2 files changed, 43 insertions(+), 5 deletions(-)

diff --git a/dlls/sapi/tests/tts.c b/dlls/sapi/tests/tts.c
index a5c950aa80b..38c69e6144a 100644
--- a/dlls/sapi/tests/tts.c
+++ b/dlls/sapi/tests/tts.c
@@ -424,7 +424,7 @@ static void test_spvoice(void)
     IUnknown *dummy;
     ISpMMSysAudio *audio_out;
     ISpObjectTokenCategory *token_cat;
-    ISpObjectToken *token;
+    ISpObjectToken *token, *token2;
     WCHAR *token_id = NULL, *default_token_id = NULL;
     ISpDataKey *attrs_key;
     LONG rate;
@@ -435,6 +435,7 @@ static void test_spvoice(void)
     ISpeechVoice *speech_voice;
     ISpeechObjectTokens *speech_tokens;
     LONG count, volume_long;
+    ISpeechObjectToken *speech_token;
     BSTR req = NULL, opt = NULL;
     UINT info_count;
     ITypeInfo *typeinfo;
@@ -724,6 +725,25 @@ static void test_spvoice(void)
     ok(hr == S_OK, "got %#lx.\n", hr);
     ok(volume_long == 80, "got %ld.\n", volume_long);

+    hr = ISpObjectToken_QueryInterface(token, &IID_ISpeechObjectToken, (void **)&speech_token);
+    ok(hr == S_OK, "got %#lx.\n", hr);
+    hr = ISpeechVoice_putref_Voice(speech_voice, speech_token);
+    ok(hr == S_OK, "got %#lx.\n", hr);
+    ISpeechObjectToken_Release(speech_token);
+
+    speech_token = (ISpeechObjectToken *)0xdeadbeef;
+    hr = ISpeechVoice_get_Voice(speech_voice, &speech_token);
+    ok(hr == S_OK, "got %#lx.\n", hr);
+    ok(speech_token && speech_token != (ISpeechObjectToken *)0xdeadbeef, "got %p.\n", speech_token);
+    hr = ISpeechObjectToken_QueryInterface(speech_token, &IID_ISpObjectToken, (void **)&token2);
+    ok(hr == S_OK, "got %#lx.\n", hr);
+    token_id = NULL;
+    hr = ISpObjectToken_GetId(token2, &token_id);
+    ok(hr == S_OK, "got %#lx.\n", hr);
+    ok(!wcscmp(token_id, test_token_id), "got %s.\n", wine_dbgstr_w(token_id));
+    CoTaskMemFree(token_id);
+    ISpObjectToken_Release(token2);
+
     hr = ISpeechVoice_Speak(speech_voice, NULL, SVSFPurgeBeforeSpeak, NULL);
     ok(hr == S_OK, "got %#lx.\n", hr);

diff --git a/dlls/sapi/tts.c b/dlls/sapi/tts.c
index 325edaf04fb..c474a118981 100644
--- a/dlls/sapi/tts.c
+++ b/dlls/sapi/tts.c
@@ -235,16 +235,34 @@ static HRESULT WINAPI speech_voice_get_Status(ISpeechVoice *iface, ISpeechVoiceS

static HRESULT WINAPI speech_voice_get_Voice(ISpeechVoice *iface, ISpeechObjectToken **voice)
 {
-    FIXME("(%p, %p): stub.\n", iface, voice);
+    struct speech_voice *This = impl_from_ISpeechVoice(iface);
+    ISpObjectToken *token;
+    HRESULT hr;

-    return E_NOTIMPL;
+    TRACE("(%p, %p).\n", iface, voice);
+
+    if (!voice) return E_POINTER;
+    if (FAILED(hr = ISpVoice_GetVoice(&This->ISpVoice_iface, &token)))
+        return hr;
+    hr = ISpObjectToken_QueryInterface(token, &IID_ISpeechObjectToken, (void **)voice);
+    ISpObjectToken_Release(token);
+    return hr;
 }

static HRESULT WINAPI speech_voice_putref_Voice(ISpeechVoice *iface, ISpeechObjectToken *voice)
 {
-    FIXME("(%p, %p): stub.\n", iface, voice);
+    struct speech_voice *This = impl_from_ISpeechVoice(iface);
+    ISpObjectToken *token;
+    HRESULT hr;

-    return E_NOTIMPL;
+    TRACE("(%p, %p).\n", iface, voice);
+
+    if (!voice) return E_INVALIDARG;
+    if (FAILED(hr = ISpeechObjectToken_QueryInterface(voice, &IID_ISpObjectToken, (void **)&token)))
+        return hr;
+    hr = ISpVoice_SetVoice(&This->ISpVoice_iface, token);
+    ISpObjectToken_Release(token);
+    return hr;
 }

static HRESULT WINAPI speech_voice_get_AudioOutput(ISpeechVoice *iface, ISpeechObjectToken **output)