[PATCH v2 6/7] dmusic: Parse instrument name from INFO list.

[Rémi Bernon]

From: Rémi Bernon

---
 dlls/dmusic/collection.c     | 11 +++--------
 dlls/dmusic/dmusic_private.h |  5 +----
 dlls/dmusic/instrument.c     | 13 +++++++------
 3 files changed, 11 insertions(+), 18 deletions(-)

diff --git a/dlls/dmusic/collection.c b/dlls/dmusic/collection.c
index c3093d10a77..aaeb62e2cd9 100644
--- a/dlls/dmusic/collection.c
+++ b/dlls/dmusic/collection.c
@@ -25,6 +25,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(dmusic);
 struct instrument_entry
 {
     struct list entry;
+    DMUS_OBJECTDESC desc;
     IDirectMusicInstrument *instrument;
 };

@@ -158,13 +159,7 @@ static HRESULT WINAPI collection_EnumInstrument(IDirectMusicCollection *iface,
     {
         if (index--) continue;
         if (FAILED(hr = IDirectMusicInstrument_GetPatch(entry->instrument, patch))) return hr;
-        if (name)
-        {
-            struct instrument *instrument = impl_from_IDirectMusicInstrument(entry->instrument);
-            DWORD length = min(lstrlenW(instrument->wszName), name_length - 1);
-            memcpy(name, instrument->wszName, length * sizeof(WCHAR));
-            name[length] = '\0';
-        }
+        if (name) lstrcpynW(name, entry->desc.wszName, name_length);
         return S_OK;
     }

@@ -192,7 +187,7 @@ static HRESULT parse_lins_list(struct collection *This, IStream *stream, struct
         {
         case MAKE_IDTYPE(FOURCC_LIST, FOURCC_INS):
             if (!(entry = malloc(sizeof(*entry)))) return E_OUTOFMEMORY;
-            hr = instrument_create_from_chunk(stream, &chunk, &entry->instrument);
+            hr = instrument_create_from_chunk(stream, &chunk, &entry->desc, &entry->instrument);
             if (SUCCEEDED(hr)) list_add_tail(&This->instruments, &entry->entry);
             else free(entry);
             break;
diff --git a/dlls/dmusic/dmusic_private.h b/dlls/dmusic/dmusic_private.h
index d200d3f0e15..31ff63a954d 100644
--- a/dlls/dmusic/dmusic_private.h
+++ b/dlls/dmusic/dmusic_private.h
@@ -98,7 +98,7 @@ extern HRESULT DMUSIC_CreateReferenceClockImpl (LPCGUID lpcGUID, LPVOID* ppobj,
 extern HRESULT download_create(DWORD size, IDirectMusicDownload **ret_iface);

extern HRESULT instrument_create_from_chunk(IStream *stream, struct chunk_entry *parent,
-        IDirectMusicInstrument **ret_iface);
+        DMUS_OBJECTDESC *desc, IDirectMusicInstrument **ret_iface);

/*****************************************************************************
  * IDirectMusic8Impl implementation structure
@@ -173,10 +173,7 @@ struct instrument
     IDirectMusicInstrument IDirectMusicInstrument_iface;
     LONG ref;

-    GUID id;
     INSTHEADER header;
-    WCHAR wszName[DMUS_MAX_NAME];
-    /* instrument data */

struct list articulations;
     struct list regions;
diff --git a/dlls/dmusic/instrument.c b/dlls/dmusic/instrument.c
index aacbe6e8c88..761c2e22822 100644
--- a/dlls/dmusic/instrument.c
+++ b/dlls/dmusic/instrument.c
@@ -266,7 +266,8 @@ static HRESULT parse_lrgn_list(struct instrument *This, IStream *stream, struct
     return hr;
 }

-static HRESULT parse_ins_chunk(struct instrument *This, IStream *stream, struct chunk_entry *parent)
+static HRESULT parse_ins_chunk(struct instrument *This, IStream *stream, struct chunk_entry *parent,
+        DMUS_OBJECTDESC *desc)
 {
     struct chunk_entry chunk = {.parent = parent};
     HRESULT hr;
@@ -280,7 +281,8 @@ static HRESULT parse_ins_chunk(struct instrument *This, IStream *stream, struct
             break;

case FOURCC_DLID:
-            hr = stream_chunk_get_data(stream, &chunk, &This->id, sizeof(This->id));
+        case MAKE_IDTYPE(FOURCC_LIST, DMUS_FOURCC_INFO_LIST):
+            hr = stream_chunk_parse_desc(stream, &chunk, desc);
             break;

case MAKE_IDTYPE(FOURCC_LIST, FOURCC_LRGN):
@@ -303,7 +305,7 @@ static HRESULT parse_ins_chunk(struct instrument *This, IStream *stream, struct
 }

HRESULT instrument_create_from_chunk(IStream *stream, struct chunk_entry *parent,
-        IDirectMusicInstrument **ret_iface)
+        DMUS_OBJECTDESC *desc, IDirectMusicInstrument **ret_iface)
 {
     IDirectMusicInstrument *iface;
     struct instrument *This;
@@ -314,7 +316,7 @@ HRESULT instrument_create_from_chunk(IStream *stream, struct chunk_entry *parent
     if (FAILED(hr = instrument_create(&iface))) return hr;
     This = impl_from_IDirectMusicInstrument(iface);

-    if (FAILED(hr = parse_ins_chunk(This, stream, parent)))
+    if (FAILED(hr = parse_ins_chunk(This, stream, parent, desc)))
     {
         IDirectMusicInstrument_Release(iface);
         return DMUS_E_UNSUPPORTED_STREAM;
@@ -323,14 +325,13 @@ HRESULT instrument_create_from_chunk(IStream *stream, struct chunk_entry *parent
     if (TRACE_ON(dmusic))
     {
         TRACE("Created DirectMusicInstrument (%p) ***\n", This);
-        if (!IsEqualGUID(&This->id, &GUID_NULL))
-            TRACE(" - GUID = %s\n", debugstr_dmguid(&This->id));
         TRACE(" - Instrument header:\n");
         TRACE("    - cRegions: %ld\n", This->header.cRegions);
         TRACE("    - Locale:\n");
         TRACE("       - ulBank: %ld\n", This->header.Locale.ulBank);
         TRACE("       - ulInstrument: %ld\n", This->header.Locale.ulInstrument);
         TRACE("       => dwPatch: %ld\n", MIDILOCALE2Patch(&This->header.Locale));
+        if (desc->dwValidData & DMUS_OBJ_OBJECT) TRACE(" - guid: %s\n", debugstr_dmguid(&desc->guidObject));
     }

*ret_iface = iface;