reactos/dll/directx/wine/d3d9/vertexdeclaration.c

   1 /*
   2  * IDirect3DVertexDeclaration9 implementation
   3  *
   4  * Copyright 2002-2003 Raphael Junqueira
   5  *                     Jason Edmeades
   6  *
   7  * This library is free software; you can redistribute it and/or
   8  * modify it under the terms of the GNU Lesser General Public
   9  * License as published by the Free Software Foundation; either
  10  * version 2.1 of the License, or (at your option) any later version.
  11  *
  12  * This library is distributed in the hope that it will be useful,
  13  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  14  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  15  * Lesser General Public License for more details.
  16  *
  17  * You should have received a copy of the GNU Lesser General Public
  18  * License along with this library; if not, write to the Free Software
  19  * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA
  20  */
  21
  22 #include "config.h"
  23 #include "d3d9_private.h"
  24
  25 WINE_DEFAULT_DEBUG_CHANNEL(d3d9);
  26
  27 typedef struct _D3DDECLTYPE_INFO {
  28     D3DDECLTYPE d3dType;
  29     WINED3DFORMAT format;
  30     int         size;
  31     int         typesize;
  32 } D3DDECLTYPE_INFO;
  33
  34 static D3DDECLTYPE_INFO const d3d_dtype_lookup[D3DDECLTYPE_UNUSED] = {
  35    {D3DDECLTYPE_FLOAT1,    WINED3DFMT_R32_FLOAT,          1, sizeof(float)},
  36    {D3DDECLTYPE_FLOAT2,    WINED3DFMT_R32G32_FLOAT,       2, sizeof(float)},
  37    {D3DDECLTYPE_FLOAT3,    WINED3DFMT_R32G32B32_FLOAT,    3, sizeof(float)},
  38    {D3DDECLTYPE_FLOAT4,    WINED3DFMT_R32G32B32A32_FLOAT, 4, sizeof(float)},
  39    {D3DDECLTYPE_D3DCOLOR,  WINED3DFMT_A8R8G8B8,           4, sizeof(BYTE)},
  40    {D3DDECLTYPE_UBYTE4,    WINED3DFMT_R8G8B8A8_UINT,      4, sizeof(BYTE)},
  41    {D3DDECLTYPE_SHORT2,    WINED3DFMT_R16G16_SINT,        2, sizeof(short int)},
  42    {D3DDECLTYPE_SHORT4,    WINED3DFMT_R16G16B16A16_SINT,  4, sizeof(short int)},
  43    {D3DDECLTYPE_UBYTE4N,   WINED3DFMT_R8G8B8A8_UNORM,     4, sizeof(BYTE)},
  44    {D3DDECLTYPE_SHORT2N,   WINED3DFMT_R16G16_SNORM,       2, sizeof(short int)},
  45    {D3DDECLTYPE_SHORT4N,   WINED3DFMT_R16G16B16A16_SNORM, 4, sizeof(short int)},
  46    {D3DDECLTYPE_USHORT2N,  WINED3DFMT_R16G16_UNORM,       2, sizeof(short int)},
  47    {D3DDECLTYPE_USHORT4N,  WINED3DFMT_R16G16B16A16_UNORM, 4, sizeof(short int)},
  48    {D3DDECLTYPE_UDEC3,     WINED3DFMT_R10G10B10A2_UINT,   3, sizeof(short int)},
  49    {D3DDECLTYPE_DEC3N,     WINED3DFMT_R10G10B10A2_SNORM,  3, sizeof(short int)},
  50    {D3DDECLTYPE_FLOAT16_2, WINED3DFMT_R16G16_FLOAT,       2, sizeof(short int)},
  51    {D3DDECLTYPE_FLOAT16_4, WINED3DFMT_R16G16B16A16_FLOAT, 4, sizeof(short int)}};
  52
  53 #define D3D_DECL_SIZE(type)          d3d_dtype_lookup[type].size
  54 #define D3D_DECL_TYPESIZE(type)      d3d_dtype_lookup[type].typesize
  55
  56 HRESULT vdecl_convert_fvf(
  57     DWORD fvf,
  58     D3DVERTEXELEMENT9** ppVertexElements) {
  59
  60     unsigned int idx, idx2;
  61     unsigned int offset;
  62     BOOL has_pos = (fvf & D3DFVF_POSITION_MASK) != 0;
  63     BOOL has_blend = (fvf & D3DFVF_XYZB5) > D3DFVF_XYZRHW;
  64     BOOL has_blend_idx = has_blend &&
  65        (((fvf & D3DFVF_XYZB5) == D3DFVF_XYZB5) ||
  66         (fvf & D3DFVF_LASTBETA_D3DCOLOR) ||
  67         (fvf & D3DFVF_LASTBETA_UBYTE4));
  68     BOOL has_normal = (fvf & D3DFVF_NORMAL) != 0;
  69     BOOL has_psize = (fvf & D3DFVF_PSIZE) != 0;
  70
  71     BOOL has_diffuse = (fvf & D3DFVF_DIFFUSE) != 0;
  72     BOOL has_specular = (fvf & D3DFVF_SPECULAR) !=0;
  73
  74     DWORD num_textures = (fvf & D3DFVF_TEXCOUNT_MASK) >> D3DFVF_TEXCOUNT_SHIFT;
  75     DWORD texcoords = (fvf & 0xFFFF0000) >> 16;
  76
  77     D3DVERTEXELEMENT9 end_element = D3DDECL_END();
  78     D3DVERTEXELEMENT9 *elements = NULL;
  79
  80     unsigned int size;
  81     DWORD num_blends = 1 + (((fvf & D3DFVF_XYZB5) - D3DFVF_XYZB1) >> 1);
  82     if (has_blend_idx) num_blends--;
  83
  84     /* Compute declaration size */
  85     size = has_pos + (has_blend && num_blends > 0) + has_blend_idx + has_normal +
  86            has_psize + has_diffuse + has_specular + num_textures + 1;
  87
  88     /* convert the declaration */
  89     elements = HeapAlloc(GetProcessHeap(), 0, size * sizeof(D3DVERTEXELEMENT9));
  90     if (!elements) return D3DERR_OUTOFVIDEOMEMORY;
  91
  92     elements[size-1] = end_element;
  93     idx = 0;
  94     if (has_pos) {
  95         if (!has_blend && (fvf & D3DFVF_XYZRHW)) {
  96             elements[idx].Type = D3DDECLTYPE_FLOAT4;
  97             elements[idx].Usage = D3DDECLUSAGE_POSITIONT;
  98         }
  99         else if (!has_blend && (fvf & D3DFVF_XYZW) == D3DFVF_XYZW) {
 100             elements[idx].Type = D3DDECLTYPE_FLOAT4;
 101             elements[idx].Usage = D3DDECLUSAGE_POSITION;
 102         }
 103         else {
 104             elements[idx].Type = D3DDECLTYPE_FLOAT3;
 105             elements[idx].Usage = D3DDECLUSAGE_POSITION;
 106         }
 107         elements[idx].UsageIndex = 0;
 108         idx++;
 109     }
 110     if (has_blend && (num_blends > 0)) {
 111         if (((fvf & D3DFVF_XYZB5) == D3DFVF_XYZB2) && (fvf & D3DFVF_LASTBETA_D3DCOLOR))
 112             elements[idx].Type = D3DDECLTYPE_D3DCOLOR;
 113         else {
 114             switch(num_blends) {
 115                 case 1: elements[idx].Type = D3DDECLTYPE_FLOAT1; break;
 116                 case 2: elements[idx].Type = D3DDECLTYPE_FLOAT2; break;
 117                 case 3: elements[idx].Type = D3DDECLTYPE_FLOAT3; break;
 118                 case 4: elements[idx].Type = D3DDECLTYPE_FLOAT4; break;
 119                 default:
 120                     ERR("Unexpected amount of blend values: %u\n", num_blends);
 121             }
 122         }
 123         elements[idx].Usage = D3DDECLUSAGE_BLENDWEIGHT;
 124         elements[idx].UsageIndex = 0;
 125         idx++;
 126     }
 127     if (has_blend_idx) {
 128         if (fvf & D3DFVF_LASTBETA_UBYTE4 ||
 129             (((fvf & D3DFVF_XYZB5) == D3DFVF_XYZB2) && (fvf & D3DFVF_LASTBETA_D3DCOLOR)))
 130             elements[idx].Type = D3DDECLTYPE_UBYTE4;
 131         else if (fvf & D3DFVF_LASTBETA_D3DCOLOR)
 132             elements[idx].Type = D3DDECLTYPE_D3DCOLOR;
 133         else
 134             elements[idx].Type = D3DDECLTYPE_FLOAT1;
 135         elements[idx].Usage = D3DDECLUSAGE_BLENDINDICES;
 136         elements[idx].UsageIndex = 0;
 137         idx++;
 138     }
 139     if (has_normal) {
 140         elements[idx].Type = D3DDECLTYPE_FLOAT3;
 141         elements[idx].Usage = D3DDECLUSAGE_NORMAL;
 142         elements[idx].UsageIndex = 0;
 143         idx++;
 144     }
 145     if (has_psize) {
 146         elements[idx].Type = D3DDECLTYPE_FLOAT1;
 147         elements[idx].Usage = D3DDECLUSAGE_PSIZE;
 148         elements[idx].UsageIndex = 0;
 149         idx++;
 150     }
 151     if (has_diffuse) {
 152         elements[idx].Type = D3DDECLTYPE_D3DCOLOR;
 153         elements[idx].Usage = D3DDECLUSAGE_COLOR;
 154         elements[idx].UsageIndex = 0;
 155         idx++;
 156     }
 157     if (has_specular) {
 158         elements[idx].Type = D3DDECLTYPE_D3DCOLOR;
 159         elements[idx].Usage = D3DDECLUSAGE_COLOR;
 160         elements[idx].UsageIndex = 1;
 161         idx++;
 162     }
 163     for (idx2 = 0; idx2 < num_textures; idx2++) {
 164         unsigned int numcoords = (texcoords >> (idx2*2)) & 0x03;
 165         switch (numcoords) {
 166             case D3DFVF_TEXTUREFORMAT1:
 167                 elements[idx].Type = D3DDECLTYPE_FLOAT1;
 168                 break;
 169             case D3DFVF_TEXTUREFORMAT2:
 170                 elements[idx].Type = D3DDECLTYPE_FLOAT2;
 171                 break;
 172             case D3DFVF_TEXTUREFORMAT3:
 173                 elements[idx].Type = D3DDECLTYPE_FLOAT3;
 174                 break;
 175             case D3DFVF_TEXTUREFORMAT4:
 176                 elements[idx].Type = D3DDECLTYPE_FLOAT4;
 177                 break;
 178         }
 179         elements[idx].Usage = D3DDECLUSAGE_TEXCOORD;
 180         elements[idx].UsageIndex = idx2;
 181         idx++;
 182     }
 183
 184     /* Now compute offsets, and initialize the rest of the fields */
 185     for (idx = 0, offset = 0; idx < size-1; idx++) {
 186         elements[idx].Stream = 0;
 187         elements[idx].Method = D3DDECLMETHOD_DEFAULT;
 188         elements[idx].Offset = offset;
 189         offset += D3D_DECL_SIZE(elements[idx].Type) * D3D_DECL_TYPESIZE(elements[idx].Type);
 190     }
 191
 192     *ppVertexElements = elements;
 193     return D3D_OK;
 194 }
 195
 196 /* IDirect3DVertexDeclaration9 IUnknown parts follow: */
 197 static HRESULT WINAPI IDirect3DVertexDeclaration9Impl_QueryInterface(LPDIRECT3DVERTEXDECLARATION9 iface, REFIID riid, LPVOID* ppobj) {
 198     IDirect3DVertexDeclaration9Impl *This = (IDirect3DVertexDeclaration9Impl *)iface;
 199
 200     if (IsEqualGUID(riid, &IID_IUnknown)
 201         || IsEqualGUID(riid, &IID_IDirect3DVertexDeclaration9)) {
 202         IDirect3DVertexDeclaration9_AddRef(iface);
 203         *ppobj = This;
 204         return S_OK;
 205     }
 206
 207     WARN("(%p)->(%s,%p),not found\n", This, debugstr_guid(riid), ppobj);
 208     *ppobj = NULL;
 209     return E_NOINTERFACE;
 210 }
 211
 212 static ULONG WINAPI IDirect3DVertexDeclaration9Impl_AddRef(LPDIRECT3DVERTEXDECLARATION9 iface) {
 213     IDirect3DVertexDeclaration9Impl *This = (IDirect3DVertexDeclaration9Impl *)iface;
 214     ULONG ref = InterlockedIncrement(&This->ref);
 215
 216     TRACE("(%p) : AddRef from %d\n", This, ref - 1);
 217
 218     if(ref == 1) {
 219         IDirect3DDevice9Ex_AddRef(This->parentDevice);
 220     }
 221
 222     return ref;
 223 }
 224
 225 void IDirect3DVertexDeclaration9Impl_Destroy(LPDIRECT3DVERTEXDECLARATION9 iface) {
 226     IDirect3DVertexDeclaration9Impl *This = (IDirect3DVertexDeclaration9Impl *)iface;
 227
 228     if(This->ref != 0) {
 229         /* Should not happen unless wine has a bug or the application releases references it does not own */
 230         ERR("Destroying vdecl with ref != 0\n");
 231     }
 232
 233     wined3d_mutex_lock();
 234     IWineD3DVertexDeclaration_Release(This->wineD3DVertexDeclaration);
 235     wined3d_mutex_unlock();
 236
 237     HeapFree(GetProcessHeap(), 0, This->elements);
 238     HeapFree(GetProcessHeap(), 0, This);
 239 }
 240
 241 static ULONG WINAPI IDirect3DVertexDeclaration9Impl_Release(LPDIRECT3DVERTEXDECLARATION9 iface) {
 242     IDirect3DVertexDeclaration9Impl *This = (IDirect3DVertexDeclaration9Impl *)iface;
 243     ULONG ref = InterlockedDecrement(&This->ref);
 244
 245     TRACE("(%p) : ReleaseRef to %d\n", This, ref);
 246
 247     if (ref == 0) {
 248         IDirect3DDevice9Ex *parentDevice = This->parentDevice;
 249
 250         if(!This->convFVF) {
 251             IDirect3DVertexDeclaration9Impl_Release(iface);
 252         }
 253         IDirect3DDevice9Ex_Release(parentDevice);
 254     }
 255     return ref;
 256 }
 257
 258 /* IDirect3DVertexDeclaration9 Interface follow: */
 259 static HRESULT WINAPI IDirect3DVertexDeclaration9Impl_GetDevice(LPDIRECT3DVERTEXDECLARATION9 iface, IDirect3DDevice9** ppDevice) {
 260     IDirect3DVertexDeclaration9Impl *This = (IDirect3DVertexDeclaration9Impl *)iface;
 261     IWineD3DDevice *myDevice = NULL;
 262     HRESULT hr = D3D_OK;
 263
 264     TRACE("(%p) : Relay\n", iface);
 265
 266     wined3d_mutex_lock();
 267     hr = IWineD3DVertexDeclaration_GetDevice(This->wineD3DVertexDeclaration, &myDevice);
 268     if (hr == D3D_OK && myDevice != NULL) {
 269         hr = IWineD3DDevice_GetParent(myDevice, (IUnknown **)ppDevice);
 270         IWineD3DDevice_Release(myDevice);
 271     }
 272     wined3d_mutex_unlock();
 273
 274     return hr;
 275 }
 276
 277 static HRESULT WINAPI IDirect3DVertexDeclaration9Impl_GetDeclaration(LPDIRECT3DVERTEXDECLARATION9 iface, D3DVERTEXELEMENT9* pDecl, UINT* pNumElements) {
 278     IDirect3DVertexDeclaration9Impl *This = (IDirect3DVertexDeclaration9Impl *)iface;
 279
 280     TRACE("(%p) : pDecl %p, pNumElements %p)\n", This, pDecl, pNumElements);
 281
 282     *pNumElements = This->element_count;
 283
 284     /* Passing a NULL pDecl is used to just retrieve the number of elements */
 285     if (!pDecl) {
 286         TRACE("NULL pDecl passed. Returning D3D_OK.\n");
 287         return D3D_OK;
 288     }
 289
 290     TRACE("Copying %p to %p\n", This->elements, pDecl);
 291     CopyMemory(pDecl, This->elements, This->element_count * sizeof(D3DVERTEXELEMENT9));
 292
 293     return D3D_OK;
 294 }
 295
 296 static const IDirect3DVertexDeclaration9Vtbl Direct3DVertexDeclaration9_Vtbl =
 297 {
 298     /* IUnknown */
 299     IDirect3DVertexDeclaration9Impl_QueryInterface,
 300     IDirect3DVertexDeclaration9Impl_AddRef,
 301     IDirect3DVertexDeclaration9Impl_Release,
 302     /* IDirect3DVertexDeclaration9 */
 303     IDirect3DVertexDeclaration9Impl_GetDevice,
 304     IDirect3DVertexDeclaration9Impl_GetDeclaration
 305 };
 306
 307 static HRESULT convert_to_wined3d_declaration(const D3DVERTEXELEMENT9* d3d9_elements,
 308         WINED3DVERTEXELEMENT **wined3d_elements, UINT *element_count)
 309 {
 310     const D3DVERTEXELEMENT9* element;
 311     UINT count = 1;
 312     UINT i;
 313
 314     TRACE("d3d9_elements %p, wined3d_elements %p\n", d3d9_elements, wined3d_elements);
 315
 316     element = d3d9_elements;
 317     while (element++->Stream != 0xff && count++ < 128);
 318
 319     if (count == 128) return E_FAIL;
 320
 321     /* Skip the END element */
 322     --count;
 323
 324     *wined3d_elements = HeapAlloc(GetProcessHeap(), 0, count * sizeof(WINED3DVERTEXELEMENT));
 325     if (!*wined3d_elements) {
 326         FIXME("Memory allocation failed\n");
 327         return D3DERR_OUTOFVIDEOMEMORY;
 328     }
 329
 330     for (i = 0; i < count; ++i)
 331     {
 332         if (d3d9_elements[i].Type >= (sizeof(d3d_dtype_lookup) / sizeof(*d3d_dtype_lookup)))
 333         {
 334             WARN("Invalid element type %#x.\n", d3d9_elements[i].Type);
 335             HeapFree(GetProcessHeap(), 0, *wined3d_elements);
 336             return E_FAIL;
 337         }
 338         (*wined3d_elements)[i].format = d3d_dtype_lookup[d3d9_elements[i].Type].format;
 339         (*wined3d_elements)[i].input_slot = d3d9_elements[i].Stream;
 340         (*wined3d_elements)[i].offset = d3d9_elements[i].Offset;
 341         (*wined3d_elements)[i].output_slot = ~0U;
 342         (*wined3d_elements)[i].method = d3d9_elements[i].Method;
 343         (*wined3d_elements)[i].usage = d3d9_elements[i].Usage;
 344         (*wined3d_elements)[i].usage_idx = d3d9_elements[i].UsageIndex;
 345     }
 346
 347     *element_count = count;
 348
 349     return D3D_OK;
 350 }
 351
 352 /* IDirect3DDevice9 IDirect3DVertexDeclaration9 Methods follow: */
 353 HRESULT WINAPI IDirect3DDevice9Impl_CreateVertexDeclaration(IDirect3DDevice9Ex *iface,
 354         const D3DVERTEXELEMENT9 *pVertexElements, IDirect3DVertexDeclaration9 **ppDecl)
 355 {
 356     IDirect3DDevice9Impl *This = (IDirect3DDevice9Impl *)iface;
 357     IDirect3DVertexDeclaration9Impl *object = NULL;
 358     WINED3DVERTEXELEMENT* wined3d_elements;
 359     UINT wined3d_element_count;
 360     UINT element_count;
 361     HRESULT hr;
 362
 363     TRACE("(%p) : Relay\n", iface);
 364     if (NULL == ppDecl) {
 365         WARN("(%p) : Caller passed NULL As ppDecl, returning D3DERR_INVALIDCALL\n",This);
 366         return D3DERR_INVALIDCALL;
 367     }
 368
 369     hr = convert_to_wined3d_declaration(pVertexElements, &wined3d_elements, &wined3d_element_count);
 370     if (FAILED(hr))
 371     {
 372         WARN("(%p) : Error parsing vertex declaration\n", This);
 373         return hr;
 374     }
 375
 376     /* Allocate the storage for the device */
 377     object = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(IDirect3DVertexDeclaration9Impl));
 378     if (NULL == object) {
 379         HeapFree(GetProcessHeap(), 0, wined3d_elements);
 380         FIXME("Allocation of memory failed, returning D3DERR_OUTOFVIDEOMEMORY\n");
 381         return D3DERR_OUTOFVIDEOMEMORY;
 382     }
 383
 384     object->lpVtbl = &Direct3DVertexDeclaration9_Vtbl;
 385     object->ref = 0;
 386
 387     element_count = wined3d_element_count + 1;
 388     object->elements = HeapAlloc(GetProcessHeap(), 0, element_count * sizeof(D3DVERTEXELEMENT9));
 389     if (!object->elements) {
 390         HeapFree(GetProcessHeap(), 0, wined3d_elements);
 391         HeapFree(GetProcessHeap(), 0, object);
 392         ERR("Memory allocation failed\n");
 393         return D3DERR_OUTOFVIDEOMEMORY;
 394     }
 395     CopyMemory(object->elements, pVertexElements, element_count * sizeof(D3DVERTEXELEMENT9));
 396     object->element_count = element_count;
 397
 398     wined3d_mutex_lock();
 399     hr = IWineD3DDevice_CreateVertexDeclaration(This->WineD3DDevice, &object->wineD3DVertexDeclaration,
 400             (IUnknown *)object, wined3d_elements, wined3d_element_count);
 401     wined3d_mutex_unlock();
 402
 403     HeapFree(GetProcessHeap(), 0, wined3d_elements);
 404
 405     if (FAILED(hr)) {
 406
 407         /* free up object */
 408         WARN("(%p) call to IWineD3DDevice_CreateVertexDeclaration failed\n", This);
 409         HeapFree(GetProcessHeap(), 0, object->elements);
 410         HeapFree(GetProcessHeap(), 0, object);
 411     } else {
 412         object->parentDevice = iface;
 413         *ppDecl = (LPDIRECT3DVERTEXDECLARATION9) object;
 414         IDirect3DVertexDeclaration9_AddRef(*ppDecl);
 415          TRACE("(%p) : Created vertex declaration %p\n", This, object);
 416     }
 417     return hr;
 418 }
 419
 420 HRESULT  WINAPI  IDirect3DDevice9Impl_SetVertexDeclaration(LPDIRECT3DDEVICE9EX iface, IDirect3DVertexDeclaration9* pDecl) {
 421     IDirect3DDevice9Impl *This = (IDirect3DDevice9Impl *)iface;
 422     IDirect3DVertexDeclaration9Impl *pDeclImpl = (IDirect3DVertexDeclaration9Impl *)pDecl;
 423     HRESULT hr = D3D_OK;
 424
 425     TRACE("(%p) : Relay\n", iface);
 426
 427     wined3d_mutex_lock();
 428     hr = IWineD3DDevice_SetVertexDeclaration(This->WineD3DDevice, pDeclImpl == NULL ? NULL : pDeclImpl->wineD3DVertexDeclaration);
 429     wined3d_mutex_unlock();
 430
 431     return hr;
 432 }
 433
 434 HRESULT  WINAPI  IDirect3DDevice9Impl_GetVertexDeclaration(LPDIRECT3DDEVICE9EX iface, IDirect3DVertexDeclaration9** ppDecl) {
 435     IDirect3DDevice9Impl* This = (IDirect3DDevice9Impl*) iface;
 436     IWineD3DVertexDeclaration* pTest = NULL;
 437     HRESULT hr = D3D_OK;
 438
 439     TRACE("(%p) : Relay+\n", iface);
 440
 441     if (NULL == ppDecl) {
 442       return D3DERR_INVALIDCALL;
 443     }
 444
 445     *ppDecl = NULL;
 446
 447     wined3d_mutex_lock();
 448     hr = IWineD3DDevice_GetVertexDeclaration(This->WineD3DDevice, &pTest);
 449     if (hr == D3D_OK && NULL != pTest) {
 450         IWineD3DVertexDeclaration_GetParent(pTest, (IUnknown **)ppDecl);
 451         IWineD3DVertexDeclaration_Release(pTest);
 452     } else {
 453         *ppDecl = NULL;
 454     }
 455     wined3d_mutex_unlock();
 456
 457     TRACE("(%p) : returning %p\n", This, *ppDecl);
 458     return hr;
 459 }