October 2012 - Wine-Commits

Dmitry Timoshkov : user32: Make sure that window' s normal_rect is initialized at window creation time.
by Alexandre Julliard Oct. 15, 2012

Oct. 15, 2012

Module: wine Branch: master Commit: e27801e93b098b054ad6668a4c6c4189dd669bb2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e27801e93b098b054ad6668a4… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Mon Oct 15 16:35:19 2012 +0900 user32: Make sure that window's normal_rect is initialized at window creation time. --- dlls/user32/win.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/user32/win.c b/dlls/user32/win.c index 66eaf09..645d563 100644 --- a/dlls/user32/win.c +++ b/dlls/user32/win.c @@ -1467,6 +1467,7 @@ HWND WIN_CreateWindowEx( CREATESTRUCTW *cs, LPCWSTR className, HINSTANCE module, wndPtr->min_pos.x = wndPtr->min_pos.y = -1; wndPtr->max_pos.x = wndPtr->max_pos.y = -1; + SetRect( &wndPtr->normal_rect, cs->x, cs->y, cs->x + cs->cx, cs->y + cs->cy ); if (wndPtr->dwStyle & WS_SYSMENU) SetSystemMenu( hwnd, 0 );

1 0

Thomas Faber : widl: Use 'class' for coclass typedefs in C++.
by Alexandre Julliard Oct. 15, 2012

Oct. 15, 2012

Module: wine Branch: master Commit: 52607ba66bad5f477209362f03ea2ddf8ca524de URL: http://source.winehq.org/git/wine.git/?a=commit;h=52607ba66bad5f477209362f0… Author: Thomas Faber <thfabba(a)gmx.de> Date: Sun Oct 14 18:35:14 2012 +0200 widl: Use 'class' for coclass typedefs in C++. --- tools/widl/header.c | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/tools/widl/header.c b/tools/widl/header.c index 02e3b60..2f275c7 100644 --- a/tools/widl/header.c +++ b/tools/widl/header.c @@ -1301,7 +1301,11 @@ static void write_coclass_forward(FILE *header, type_t *cocl) { fprintf(header, "#ifndef __%s_FWD_DEFINED__\n", cocl->name); fprintf(header, "#define __%s_FWD_DEFINED__\n", cocl->name); + fprintf(header, "#ifdef __cplusplus\n"); + fprintf(header, "typedef class %s %s;\n", cocl->name, cocl->name); + fprintf(header, "#else\n"); fprintf(header, "typedef struct %s %s;\n", cocl->name, cocl->name); + fprintf(header, "#endif /* defined __cplusplus */\n"); fprintf(header, "#endif /* defined __%s_FWD_DEFINED__ */\n\n", cocl->name ); }

1 0

Christian Costa : amstream: Get rid of (IBaseFilter*) casts.
by Alexandre Julliard Oct. 15, 2012

Oct. 15, 2012

Module: wine Branch: master Commit: 568c88ccb943df016acc9b4b16593eb23161a036 URL: http://source.winehq.org/git/wine.git/?a=commit;h=568c88ccb943df016acc9b4b1… Author: Christian Costa <titan.costa(a)gmail.com> Date: Fri Oct 12 23:07:07 2012 +0200 amstream: Get rid of (IBaseFilter*) casts. --- dlls/amstream/mediastreamfilter.c | 52 +++++++++++++++++++++--------------- 1 files changed, 30 insertions(+), 22 deletions(-) diff --git a/dlls/amstream/mediastreamfilter.c b/dlls/amstream/mediastreamfilter.c index 9d38b72..194914e 100644 --- a/dlls/amstream/mediastreamfilter.c +++ b/dlls/amstream/mediastreamfilter.c @@ -164,74 +164,82 @@ static ULONG WINAPI MediaStreamFilterImpl_Release(IMediaStreamFilter *iface) /*** IPersist methods ***/ -static HRESULT WINAPI MediaStreamFilterImpl_GetClassID(IMediaStreamFilter * iface, CLSID * pClsid) +static HRESULT WINAPI MediaStreamFilterImpl_GetClassID(IMediaStreamFilter *iface, CLSID *clsid) { - return BaseFilterImpl_GetClassID((IBaseFilter*)iface, pClsid); + IMediaStreamFilterImpl *This = impl_from_IMediaStreamFilter(iface); + return BaseFilterImpl_GetClassID(&This->filter.IBaseFilter_iface, clsid); } /*** IBaseFilter methods ***/ -static HRESULT WINAPI MediaStreamFilterImpl_Stop(IMediaStreamFilter * iface) +static HRESULT WINAPI MediaStreamFilterImpl_Stop(IMediaStreamFilter *iface) { FIXME("(%p)->(): Stub!\n", iface); return E_NOTIMPL; } -static HRESULT WINAPI MediaStreamFilterImpl_Pause(IMediaStreamFilter * iface) +static HRESULT WINAPI MediaStreamFilterImpl_Pause(IMediaStreamFilter *iface) { FIXME("(%p)->(): Stub!\n", iface); return E_NOTIMPL; } -static HRESULT WINAPI MediaStreamFilterImpl_Run(IMediaStreamFilter * iface, REFERENCE_TIME tStart) +static HRESULT WINAPI MediaStreamFilterImpl_Run(IMediaStreamFilter *iface, REFERENCE_TIME start) { - FIXME("(%p)->(%s): Stub!\n", iface, wine_dbgstr_longlong(tStart)); + FIXME("(%p)->(%s): Stub!\n", iface, wine_dbgstr_longlong(start)); return E_NOTIMPL; } -static HRESULT WINAPI MediaStreamFilterImpl_GetState(IMediaStreamFilter * iface, DWORD dwMilliSecsTimeout, FILTER_STATE *pState) +static HRESULT WINAPI MediaStreamFilterImpl_GetState(IMediaStreamFilter *iface, DWORD ms_timeout, FILTER_STATE *state) { - return BaseFilterImpl_GetState((IBaseFilter*)iface, dwMilliSecsTimeout, pState); + IMediaStreamFilterImpl *This = impl_from_IMediaStreamFilter(iface); + return BaseFilterImpl_GetState(&This->filter.IBaseFilter_iface, ms_timeout, state); } -static HRESULT WINAPI MediaStreamFilterImpl_SetSyncSource(IMediaStreamFilter * iface, IReferenceClock *pClock) +static HRESULT WINAPI MediaStreamFilterImpl_SetSyncSource(IMediaStreamFilter *iface, IReferenceClock *clock) { - return BaseFilterImpl_SetSyncSource((IBaseFilter*)iface, pClock); + IMediaStreamFilterImpl *This = impl_from_IMediaStreamFilter(iface); + return BaseFilterImpl_SetSyncSource(&This->filter.IBaseFilter_iface, clock); } -static HRESULT WINAPI MediaStreamFilterImpl_GetSyncSource(IMediaStreamFilter * iface, IReferenceClock **ppClock) +static HRESULT WINAPI MediaStreamFilterImpl_GetSyncSource(IMediaStreamFilter *iface, IReferenceClock **clock) { - return BaseFilterImpl_GetSyncSource((IBaseFilter*)iface, ppClock); + IMediaStreamFilterImpl *This = impl_from_IMediaStreamFilter(iface); + return BaseFilterImpl_GetSyncSource(&This->filter.IBaseFilter_iface, clock); } -static HRESULT WINAPI MediaStreamFilterImpl_EnumPins(IMediaStreamFilter * iface, IEnumPins **ppEnum) +static HRESULT WINAPI MediaStreamFilterImpl_EnumPins(IMediaStreamFilter *iface, IEnumPins **enum_pins) { - return BaseFilterImpl_EnumPins((IBaseFilter*)iface, ppEnum); + IMediaStreamFilterImpl *This = impl_from_IMediaStreamFilter(iface); + return BaseFilterImpl_EnumPins(&This->filter.IBaseFilter_iface, enum_pins); } -static HRESULT WINAPI MediaStreamFilterImpl_FindPin(IMediaStreamFilter * iface, LPCWSTR Id, IPin **ppPin) +static HRESULT WINAPI MediaStreamFilterImpl_FindPin(IMediaStreamFilter *iface, LPCWSTR id, IPin **pin) { - FIXME("(%p)->(%s,%p): Stub!\n", iface, debugstr_w(Id), ppPin); + FIXME("(%p)->(%s,%p): Stub!\n", iface, debugstr_w(id), pin); return E_NOTIMPL; } -static HRESULT WINAPI MediaStreamFilterImpl_QueryFilterInfo(IMediaStreamFilter * iface, FILTER_INFO *pInfo) +static HRESULT WINAPI MediaStreamFilterImpl_QueryFilterInfo(IMediaStreamFilter *iface, FILTER_INFO *info) { - return BaseFilterImpl_QueryFilterInfo((IBaseFilter*)iface, pInfo); + IMediaStreamFilterImpl *This = impl_from_IMediaStreamFilter(iface); + return BaseFilterImpl_QueryFilterInfo(&This->filter.IBaseFilter_iface, info); } -static HRESULT WINAPI MediaStreamFilterImpl_JoinFilterGraph(IMediaStreamFilter * iface, IFilterGraph *pGraph, LPCWSTR pName) +static HRESULT WINAPI MediaStreamFilterImpl_JoinFilterGraph(IMediaStreamFilter *iface, IFilterGraph *graph, LPCWSTR name) { - return BaseFilterImpl_JoinFilterGraph((IBaseFilter*)iface, pGraph, pName); + IMediaStreamFilterImpl *This = impl_from_IMediaStreamFilter(iface); + return BaseFilterImpl_JoinFilterGraph(&This->filter.IBaseFilter_iface, graph, name); } -static HRESULT WINAPI MediaStreamFilterImpl_QueryVendorInfo(IMediaStreamFilter * iface, LPWSTR *pVendorInfo) +static HRESULT WINAPI MediaStreamFilterImpl_QueryVendorInfo(IMediaStreamFilter *iface, LPWSTR *vendor_info) { - return BaseFilterImpl_QueryVendorInfo((IBaseFilter*)iface, pVendorInfo); + IMediaStreamFilterImpl *This = impl_from_IMediaStreamFilter(iface); + return BaseFilterImpl_QueryVendorInfo(&This->filter.IBaseFilter_iface, vendor_info); } /*** IMediaStreamFilter methods ***/

1 0

Marcus Meissner : winefile: Check index before buffer overread (Coverity).
by Alexandre Julliard Oct. 15, 2012

Oct. 15, 2012

Module: wine Branch: master Commit: 8af406fb00054cd611de8c8bf54c926524eb0f0a URL: http://source.winehq.org/git/wine.git/?a=commit;h=8af406fb00054cd611de8c8bf… Author: Marcus Meissner <marcus(a)jet.franken.de> Date: Sat Oct 13 16:58:05 2012 +0200 winefile: Check index before buffer overread (Coverity). --- programs/winefile/winefile.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/programs/winefile/winefile.c b/programs/winefile/winefile.c index 366aa9c..23388c1 100644 --- a/programs/winefile/winefile.c +++ b/programs/winefile/winefile.c @@ -3163,7 +3163,7 @@ static void set_header(Pane* pane) item.mask = HDI_WIDTH; item.cxy = 0; - for(; x+pane->widths[i]<scroll_pos && i<COLUMNS; i++) { + for(; (i < COLUMNS) && (x+pane->widths[i] < scroll_pos); i++) { x += pane->widths[i]; SendMessageW(pane->hwndHeader, HDM_SETITEMW, i, (LPARAM)&item); } @@ -3173,7 +3173,7 @@ static void set_header(Pane* pane) item.cxy = x - scroll_pos; SendMessageW(pane->hwndHeader, HDM_SETITEMW, i++, (LPARAM)&item); - for(; i<COLUMNS; i++) { + for(; i < COLUMNS; i++) { item.cxy = pane->widths[i]; x += pane->widths[i]; SendMessageW(pane->hwndHeader, HDM_SETITEMW, i, (LPARAM)&item);

1 0

Henri Verbeet : wined3d: Introduce a structure for shader register indices.
by Alexandre Julliard Oct. 15, 2012

Oct. 15, 2012

Module: wine Branch: master Commit: 5626e165ffbf543a77729ea0480dadd37b878e79 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5626e165ffbf543a77729ea04… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Sun Oct 14 18:40:36 2012 +0200 wined3d: Introduce a structure for shader register indices. --- dlls/wined3d/arb_program_shader.c | 258 ++++++++++++++++++------------------- dlls/wined3d/glsl_shader.c | 140 +++++++++++--------- dlls/wined3d/shader.c | 118 +++++++++-------- dlls/wined3d/shader_sm1.c | 21 ++-- dlls/wined3d/shader_sm4.c | 25 +--- dlls/wined3d/wined3d_private.h | 16 ++- 6 files changed, 295 insertions(+), 283 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=5626e165ffbf543a77729…

1 0

Henri Verbeet : wined3d: Recognise SM4 scalar swizzles.
by Alexandre Julliard Oct. 15, 2012

Oct. 15, 2012

Module: wine Branch: master Commit: 3baeaa2b886eff05c4589df60c9cc8e8bd76ed41 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3baeaa2b886eff05c4589df60… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Sun Oct 14 18:40:34 2012 +0200 wined3d: Recognise SM4 scalar swizzles. --- dlls/wined3d/shader_sm4.c | 28 +++++++++++++++++++++++++++- 1 files changed, 27 insertions(+), 1 deletions(-) diff --git a/dlls/wined3d/shader_sm4.c b/dlls/wined3d/shader_sm4.c index 3def9d8..9b52bd7 100644 --- a/dlls/wined3d/shader_sm4.c +++ b/dlls/wined3d/shader_sm4.c @@ -51,6 +51,9 @@ WINE_DECLARE_DEBUG_CHANNEL(d3d_bytecode); #define WINED3D_SM4_REGISTER_TYPE_SHIFT 12 #define WINED3D_SM4_REGISTER_TYPE_MASK (0xf << WINED3D_SM4_REGISTER_TYPE_SHIFT) +#define WINED3D_SM4_SWIZZLE_TYPE_SHIFT 2 +#define WINED3D_SM4_SWIZZLE_TYPE_MASK (0x3 << WINED3D_SM4_SWIZZLE_TYPE_SHIFT) + #define WINED3D_SM4_IMMCONST_TYPE_SHIFT 0 #define WINED3D_SM4_IMMCONST_TYPE_MASK (0x3 << WINED3D_SM4_IMMCONST_TYPE_SHIFT) @@ -147,6 +150,12 @@ enum wined3d_sm4_input_primitive_type WINED3D_SM4_INPUT_PT_TRIANGLEADJ = 0x7, }; +enum wined3d_sm4_swizzle_type +{ + WINED3D_SM4_SWIZZLE_VEC4 = 0x1, + WINED3D_SM4_SWIZZLE_SCALAR = 0x2, +}; + enum wined3d_sm4_immconst_type { WINED3D_SM4_IMMCONST_SCALAR = 0x1, @@ -609,7 +618,24 @@ static BOOL shader_sm4_read_src_param(struct wined3d_sm4_data *priv, const DWORD } else { - src_param->swizzle = (token & WINED3D_SM4_SWIZZLE_MASK) >> WINED3D_SM4_SWIZZLE_SHIFT; + enum wined3d_sm4_swizzle_type swizzle_type = + (token & WINED3D_SM4_SWIZZLE_TYPE_MASK) >> WINED3D_SM4_SWIZZLE_TYPE_SHIFT; + + switch (swizzle_type) + { + case WINED3D_SM4_SWIZZLE_SCALAR: + src_param->swizzle = (token & WINED3D_SM4_SWIZZLE_MASK) >> WINED3D_SM4_SWIZZLE_SHIFT; + src_param->swizzle = (src_param->swizzle & 0x3) * 0x55; + break; + + case WINED3D_SM4_SWIZZLE_VEC4: + src_param->swizzle = (token & WINED3D_SM4_SWIZZLE_MASK) >> WINED3D_SM4_SWIZZLE_SHIFT; + break; + + default: + FIXME("Unhandled swizzle type %#x.\n", swizzle_type); + break; + } } map_register(priv, &src_param->reg);

1 0

Christian Costa : amstream: Implement BasePinImpl_CheckMediaType in media stream filter.
by Alexandre Julliard Oct. 15, 2012

Oct. 15, 2012

Module: wine Branch: master Commit: 604fb218b77eec6ee6e204e43fda237316c30b96 URL: http://source.winehq.org/git/wine.git/?a=commit;h=604fb218b77eec6ee6e204e43… Author: Christian Costa <titan.costa(a)gmail.com> Date: Fri Oct 12 23:07:24 2012 +0200 amstream: Implement BasePinImpl_CheckMediaType in media stream filter. --- dlls/amstream/mediastreamfilter.c | 42 +++++++++++++++++++++++++++++++++++++ 1 files changed, 42 insertions(+), 0 deletions(-) diff --git a/dlls/amstream/mediastreamfilter.c b/dlls/amstream/mediastreamfilter.c index 8423cfa..a2851ca 100644 --- a/dlls/amstream/mediastreamfilter.c +++ b/dlls/amstream/mediastreamfilter.c @@ -76,6 +76,48 @@ static inline IMediaStreamFilterImpl *impl_from_IMediaStreamFilter(IMediaStreamF static HRESULT WINAPI BasePinImpl_CheckMediaType(BasePin *This, const AM_MEDIA_TYPE *pmt) { + IMediaStreamFilterImpl *filter = impl_from_IMediaStreamFilter((IMediaStreamFilter*)This->pinInfo.pFilter); + MSPID purpose_id; + int i; + + TRACE("Checking media type %s - %s\n", debugstr_guid(&pmt->majortype), debugstr_guid(&pmt->subtype)); + + /* Find which stream is associated with the pin */ + for (i = 0; i < filter->nb_streams; i++) + if (&This->IPin_iface == filter->pins[i]) + break; + + if (i == filter->nb_streams) + return S_FALSE; + + if (FAILED(IMediaStream_GetInformation(filter->streams[i], &purpose_id, NULL))) + return S_FALSE; + + TRACE("Checking stream with purpose id %s\n", debugstr_guid(&purpose_id)); + + if (IsEqualGUID(&purpose_id, &MSPID_PrimaryVideo) && IsEqualGUID(&pmt->majortype, &MEDIATYPE_Video)) + { + if (IsEqualGUID(&pmt->subtype, &MEDIASUBTYPE_RGB1) || + IsEqualGUID(&pmt->subtype, &MEDIASUBTYPE_RGB4) || + IsEqualGUID(&pmt->subtype, &MEDIASUBTYPE_RGB8) || + IsEqualGUID(&pmt->subtype, &MEDIASUBTYPE_RGB565) || + IsEqualGUID(&pmt->subtype, &MEDIASUBTYPE_RGB555) || + IsEqualGUID(&pmt->subtype, &MEDIASUBTYPE_RGB24) || + IsEqualGUID(&pmt->subtype, &MEDIASUBTYPE_RGB32)) + { + TRACE("Video sub-type %s matches\n", debugstr_guid(&pmt->subtype)); + return S_OK; + } + } + else if (IsEqualGUID(&purpose_id, &MSPID_PrimaryAudio) && IsEqualGUID(&pmt->majortype, &MEDIATYPE_Audio)) + { + if (IsEqualGUID(&pmt->subtype, &MEDIASUBTYPE_PCM)) + { + TRACE("Audio sub-type %s matches\n", debugstr_guid(&pmt->subtype)); + return S_OK; + } + } + return S_FALSE; }

1 0

Henri Verbeet : wined3d: Use a common function for parsing SM4 source and destination parameters.
by Alexandre Julliard Oct. 15, 2012

Oct. 15, 2012

Module: wine Branch: master Commit: a92d7a12de29b8f5ece629d4ecab6fbe5743115d URL: http://source.winehq.org/git/wine.git/?a=commit;h=a92d7a12de29b8f5ece629d4e… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Sun Oct 14 18:40:35 2012 +0200 wined3d: Use a common function for parsing SM4 source and destination parameters. --- dlls/wined3d/shader_sm4.c | 113 +++++++++++++++++++++++--------------------- 1 files changed, 59 insertions(+), 54 deletions(-) diff --git a/dlls/wined3d/shader_sm4.c b/dlls/wined3d/shader_sm4.c index 9b52bd7..dceb120 100644 --- a/dlls/wined3d/shader_sm4.c +++ b/dlls/wined3d/shader_sm4.c @@ -511,58 +511,59 @@ static BOOL shader_sm4_read_reg_idx(struct wined3d_sm4_data *priv, const DWORD * return TRUE; } -static BOOL shader_sm4_read_src_param(struct wined3d_sm4_data *priv, const DWORD **ptr, - enum wined3d_data_type data_type, struct wined3d_shader_src_param *src_param) +static BOOL shader_sm4_read_param(struct wined3d_sm4_data *priv, const DWORD **ptr, + enum wined3d_data_type data_type, struct wined3d_shader_register *param, + enum wined3d_shader_src_modifier *modifier) { - DWORD token = *(*ptr)++; enum wined3d_sm4_register_type register_type; struct wined3d_sm4_reg_idx reg_idx; + DWORD token = *(*ptr)++; DWORD order; register_type = (token & WINED3D_SM4_REGISTER_TYPE_MASK) >> WINED3D_SM4_REGISTER_TYPE_SHIFT; if (register_type >= sizeof(register_type_table) / sizeof(*register_type_table)) { - FIXME("Unhandled register type %#x\n", register_type); - src_param->reg.type = WINED3DSPR_TEMP; + FIXME("Unhandled register type %#x.\n", register_type); + param->type = WINED3DSPR_TEMP; } else { - src_param->reg.type = register_type_table[register_type]; + param->type = register_type_table[register_type]; } - src_param->reg.data_type = data_type; + param->data_type = data_type; if (token & WINED3D_SM4_REGISTER_MODIFIER) { - DWORD modifier = *(*ptr)++; + DWORD m = *(*ptr)++; /* FIXME: This will probably break down at some point. The SM4 * modifiers look like flags, while wined3d currently has an enum * with possible combinations, e.g. WINED3DSPSM_ABSNEG. */ - switch (modifier) + switch (m) { case 0x41: - src_param->modifiers = WINED3DSPSM_NEG; + *modifier = WINED3DSPSM_NEG; break; case 0x81: - src_param->modifiers = WINED3DSPSM_ABS; + *modifier = WINED3DSPSM_ABS; break; default: - FIXME("Skipping modifier 0x%08x.\n", modifier); - src_param->modifiers = WINED3DSPSM_NONE; + FIXME("Skipping modifier 0x%08x.\n", m); + *modifier = WINED3DSPSM_NONE; break; } } else { - src_param->modifiers = WINED3DSPSM_NONE; + *modifier = WINED3DSPSM_NONE; } order = (token & WINED3D_SM4_REGISTER_ORDER_MASK) >> WINED3D_SM4_REGISTER_ORDER_SHIFT; if (order < 1) - src_param->reg.idx = ~0U; + param->idx = ~0U; else { DWORD addressing = (token & WINED3D_SM4_ADDRESSING_MASK0) >> WINED3D_SM4_ADDRESSING_SHIFT0; @@ -571,12 +572,12 @@ static BOOL shader_sm4_read_src_param(struct wined3d_sm4_data *priv, const DWORD ERR("Failed to read register index.\n"); return FALSE; } - src_param->reg.rel_addr = reg_idx.rel_addr; - src_param->reg.idx = reg_idx.offset; + param->rel_addr = reg_idx.rel_addr; + param->idx = reg_idx.offset; } if (order < 2) - src_param->reg.array_idx = ~0U; + param->array_idx = ~0U; else { DWORD addressing = (token & WINED3D_SM4_ADDRESSING_MASK1) >> WINED3D_SM4_ADDRESSING_SHIFT1; @@ -585,37 +586,58 @@ static BOOL shader_sm4_read_src_param(struct wined3d_sm4_data *priv, const DWORD ERR("Failed to read register index.\n"); return FALSE; } - src_param->reg.array_rel_addr = reg_idx.rel_addr; - src_param->reg.array_idx = reg_idx.offset; + param->array_rel_addr = reg_idx.rel_addr; + param->array_idx = reg_idx.offset; } - if (order > 2) FIXME("Unhandled order %u.\n", order); + if (order > 2) + FIXME("Unhandled order %u.\n", order); if (register_type == WINED3D_SM4_RT_IMMCONST) { enum wined3d_sm4_immconst_type immconst_type = (token & WINED3D_SM4_IMMCONST_TYPE_MASK) >> WINED3D_SM4_IMMCONST_TYPE_SHIFT; - src_param->swizzle = WINED3DSP_NOSWIZZLE; - switch(immconst_type) + switch (immconst_type) { case WINED3D_SM4_IMMCONST_SCALAR: - src_param->reg.immconst_type = WINED3D_IMMCONST_SCALAR; - memcpy(src_param->reg.immconst_data, *ptr, 1 * sizeof(DWORD)); + param->immconst_type = WINED3D_IMMCONST_SCALAR; + memcpy(param->immconst_data, *ptr, 1 * sizeof(DWORD)); *ptr += 1; break; case WINED3D_SM4_IMMCONST_VEC4: - src_param->reg.immconst_type = WINED3D_IMMCONST_VEC4; - memcpy(src_param->reg.immconst_data, *ptr, 4 * sizeof(DWORD)); + param->immconst_type = WINED3D_IMMCONST_VEC4; + memcpy(param->immconst_data, *ptr, 4 * sizeof(DWORD)); *ptr += 4; break; default: - FIXME("Unhandled immediate constant type %#x\n", immconst_type); + FIXME("Unhandled immediate constant type %#x.\n", immconst_type); break; } } + + map_register(priv, param); + + return TRUE; +} + +static BOOL shader_sm4_read_src_param(struct wined3d_sm4_data *priv, const DWORD **ptr, + enum wined3d_data_type data_type, struct wined3d_shader_src_param *src_param) +{ + DWORD token = **ptr; + + if (!shader_sm4_read_param(priv, ptr, data_type, &src_param->reg, &src_param->modifiers)) + { + ERR("Failed to read parameter.\n"); + return FALSE; + } + + if (src_param->reg.type == WINED3DSPR_IMMCONST) + { + src_param->swizzle = WINED3DSP_NOSWIZZLE; + } else { enum wined3d_sm4_swizzle_type swizzle_type = @@ -638,47 +660,30 @@ static BOOL shader_sm4_read_src_param(struct wined3d_sm4_data *priv, const DWORD } } - map_register(priv, &src_param->reg); - return TRUE; } static BOOL shader_sm4_read_dst_param(struct wined3d_sm4_data *priv, const DWORD **ptr, enum wined3d_data_type data_type, struct wined3d_shader_dst_param *dst_param) { - DWORD token = *(*ptr)++; - enum wined3d_sm4_register_type register_type; - DWORD order; + enum wined3d_shader_src_modifier modifier; + DWORD token = **ptr; - register_type = (token & WINED3D_SM4_REGISTER_TYPE_MASK) >> WINED3D_SM4_REGISTER_TYPE_SHIFT; - if (register_type >= sizeof(register_type_table) / sizeof(*register_type_table)) + if (!shader_sm4_read_param(priv, ptr, data_type, &dst_param->reg, &modifier)) { - FIXME("Unhandled register type %#x\n", register_type); - dst_param->reg.type = WINED3DSPR_TEMP; + ERR("Failed to read parameter.\n"); + return FALSE; } - else + + if (modifier != WINED3DSPSM_NONE) { - dst_param->reg.type = register_type_table[register_type]; + ERR("Invalid source modifier %#x on destination register.\n", modifier); + return FALSE; } - dst_param->reg.data_type = data_type; - - order = (token & WINED3D_SM4_REGISTER_ORDER_MASK) >> WINED3D_SM4_REGISTER_ORDER_SHIFT; - - if (order < 1) dst_param->reg.idx = ~0U; - else dst_param->reg.idx = *(*ptr)++; - - if (order < 2) dst_param->reg.array_idx = ~0U; - else dst_param->reg.array_idx = *(*ptr)++; - - if (order > 2) FIXME("Unhandled order %u.\n", order); dst_param->write_mask = (token & WINED3D_SM4_WRITEMASK_MASK) >> WINED3D_SM4_WRITEMASK_SHIFT; dst_param->modifiers = 0; dst_param->shift = 0; - dst_param->reg.rel_addr = NULL; - dst_param->reg.array_rel_addr = NULL; - - map_register(priv, &dst_param->reg); return TRUE; }

1 0

Hans Leidekker : wbemprox: Implement StdRegProv.GetStringValue.
by Alexandre Julliard Oct. 15, 2012

Oct. 15, 2012

Module: wine Branch: master Commit: 60db122133011036a732fc3875a9ff9b0fc0a3bb URL: http://source.winehq.org/git/wine.git/?a=commit;h=60db122133011036a732fc387… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Mon Oct 15 10:46:09 2012 +0200 wbemprox: Implement StdRegProv.GetStringValue. --- dlls/wbemprox/builtin.c | 15 +++++-- dlls/wbemprox/reg.c | 84 +++++++++++++++++++++++++++++++++++--- dlls/wbemprox/tests/query.c | 56 +++++++++++++++++++++++++- dlls/wbemprox/wbemprox_private.h | 8 +++- 4 files changed, 150 insertions(+), 13 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=60db122133011036a732f…

1 0

Henri Verbeet : wined3d: Handle SM1 comments in the frontend.
by Alexandre Julliard Oct. 15, 2012

Oct. 15, 2012

Module: wine Branch: master Commit: 311a1e014843eb2357ac71d83e6cf871574363b4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=311a1e014843eb2357ac71d83… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Sun Oct 14 18:40:37 2012 +0200 wined3d: Handle SM1 comments in the frontend. --- dlls/wined3d/shader.c | 43 --------------------------- dlls/wined3d/shader_sm1.c | 62 ++++++++++++++++++++++++++++----------- dlls/wined3d/shader_sm4.c | 7 ---- dlls/wined3d/wined3d_private.h | 1 - 4 files changed, 44 insertions(+), 69 deletions(-) diff --git a/dlls/wined3d/shader.c b/dlls/wined3d/shader.c index 12d5f98..9c70f08 100644 --- a/dlls/wined3d/shader.c +++ b/dlls/wined3d/shader.c @@ -494,12 +494,6 @@ static HRESULT shader_get_registers_used(struct wined3d_shader *shader, const st while (!fe->shader_is_end(fe_data, &ptr)) { struct wined3d_shader_instruction ins; - const char *comment; - UINT comment_size; - - /* Skip comments. */ - fe->shader_read_comment(&ptr, &comment, &comment_size); - if (comment) continue; /* Fetch opcode. */ fe->shader_read_instruction(fe_data, &ptr, &ins); @@ -1214,13 +1208,6 @@ void shader_generate_main(const struct wined3d_shader *shader, struct wined3d_sh while (!fe->shader_is_end(fe_data, &ptr)) { - const char *comment; - UINT comment_size; - - /* Skip comment tokens. */ - fe->shader_read_comment(&ptr, &comment, &comment_size); - if (comment) continue; - /* Read opcode. */ fe->shader_read_instruction(fe_data, &ptr, &ins); @@ -1342,36 +1329,6 @@ static void shader_trace_init(const struct wined3d_shader_frontend *fe, void *fe while (!fe->shader_is_end(fe_data, &ptr)) { struct wined3d_shader_instruction ins; - const char *comment; - UINT comment_size; - - /* comment */ - fe->shader_read_comment(&ptr, &comment, &comment_size); - if (comment) - { - if (comment_size > 4 && *(const DWORD *)comment == WINEMAKEFOURCC('T', 'E', 'X', 'T')) - { - const char *end = comment + comment_size; - const char *ptr = comment + 4; - const char *line = ptr; - - TRACE("// TEXT\n"); - while (ptr != end) - { - if (*ptr == '\n') - { - UINT len = ptr - line; - if (len && *(ptr - 1) == '\r') --len; - TRACE("// %s\n", debugstr_an(line, len)); - line = ++ptr; - } - else ++ptr; - } - if (line != ptr) TRACE("// %s\n", debugstr_an(line, ptr - line)); - } - else TRACE("// %s\n", debugstr_an(comment, comment_size)); - continue; - } fe->shader_read_instruction(fe_data, &ptr, &ins); if (ins.handler_idx == WINED3DSIH_TABLE_SIZE) diff --git a/dlls/wined3d/shader_sm1.c b/dlls/wined3d/shader_sm1.c index ae7e3a9..18885f5 100644 --- a/dlls/wined3d/shader_sm1.c +++ b/dlls/wined3d/shader_sm1.c @@ -642,6 +642,48 @@ static void shader_sm1_read_immconst(const DWORD **ptr, struct wined3d_shader_sr *ptr += count; } +static void shader_sm1_read_comment(const DWORD **ptr) +{ + DWORD token = **ptr; + const char *comment; + UINT size; + + while ((token & WINED3DSI_OPCODE_MASK) == WINED3D_SM1_OP_COMMENT) + { + size = (token & WINED3DSI_COMMENTSIZE_MASK) >> WINED3DSI_COMMENTSIZE_SHIFT; + comment = (const char *)++(*ptr); + *ptr += size; + + if (size > 1 && *(const DWORD *)comment == WINEMAKEFOURCC('T', 'E', 'X', 'T')) + { + const char *end = comment + size * sizeof(token); + const char *p = comment + sizeof(token); + const char *line = p; + + TRACE("// TEXT\n"); + while (p != end) + { + if (*p == '\n') + { + UINT len = p - line; + if (len && *(p - 1) == '\r') --len; + TRACE("// %s\n", debugstr_an(line, len)); + line = ++p; + } + else ++p; + } + if (line != p) + TRACE("// %s\n", debugstr_an(line, p - line)); + } + else if (size) + TRACE("// %s\n", debugstr_an(comment, size * sizeof(token))); + else + break; + + token = **ptr; + } +} + static void shader_sm1_read_instruction(void *data, const DWORD **ptr, struct wined3d_shader_instruction *ins) { const struct wined3d_sm1_opcode_info *opcode_info; @@ -650,6 +692,8 @@ static void shader_sm1_read_instruction(void *data, const DWORD **ptr, struct wi unsigned int i; const DWORD *p; + shader_sm1_read_comment(ptr); + opcode_token = *(*ptr)++; if (!(opcode_info = shader_get_opcode(priv, opcode_token))) { @@ -708,23 +752,6 @@ static void shader_sm1_read_instruction(void *data, const DWORD **ptr, struct wi } } -static void shader_sm1_read_comment(const DWORD **ptr, const char **comment, UINT *comment_size) -{ - DWORD token = **ptr; - UINT size; - - if ((token & WINED3DSI_OPCODE_MASK) != WINED3D_SM1_OP_COMMENT) - { - *comment = NULL; - return; - } - - size = (token & WINED3DSI_COMMENTSIZE_MASK) >> WINED3DSI_COMMENTSIZE_SHIFT; - *comment = (const char *)++(*ptr); - *comment_size = size * sizeof(DWORD); - *ptr += size; -} - static BOOL shader_sm1_is_end(void *data, const DWORD **ptr) { if (**ptr == WINED3DSP_END) @@ -742,6 +769,5 @@ const struct wined3d_shader_frontend sm1_shader_frontend = shader_sm1_free, shader_sm1_read_header, shader_sm1_read_instruction, - shader_sm1_read_comment, shader_sm1_is_end, }; diff --git a/dlls/wined3d/shader_sm4.c b/dlls/wined3d/shader_sm4.c index f7a73fe..b69ba82 100644 --- a/dlls/wined3d/shader_sm4.c +++ b/dlls/wined3d/shader_sm4.c @@ -789,12 +789,6 @@ static void shader_sm4_read_instruction(void *data, const DWORD **ptr, struct wi } } -static void shader_sm4_read_comment(const DWORD **ptr, const char **comment, UINT *comment_size) -{ - FIXME("ptr %p, comment %p, comment_size %p stub!\n", ptr, comment, comment_size); - *comment = NULL; -} - static BOOL shader_sm4_is_end(void *data, const DWORD **ptr) { struct wined3d_sm4_data *priv = data; @@ -807,6 +801,5 @@ const struct wined3d_shader_frontend sm4_shader_frontend = shader_sm4_free, shader_sm4_read_header, shader_sm4_read_instruction, - shader_sm4_read_comment, shader_sm4_is_end, }; diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 41dd737..56bdbad 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -702,7 +702,6 @@ struct wined3d_shader_frontend void (*shader_free)(void *data); void (*shader_read_header)(void *data, const DWORD **ptr, struct wined3d_shader_version *shader_version); void (*shader_read_instruction)(void *data, const DWORD **ptr, struct wined3d_shader_instruction *ins); - void (*shader_read_comment)(const DWORD **ptr, const char **comment, UINT *comment_size); BOOL (*shader_is_end)(void *data, const DWORD **ptr); };

1 0