wine-commits September 2021

wine-commits@winehq.org

2 participants
1030 discussions

Connor McAdams : oleacc: Add put_accValue function for edit client accessible object.
by Alexandre Julliard 27 Sep '21

27 Sep '21

Module: wine Branch: master Commit: b8d0118517e8a42a7987dc7cdc61e1a7a59f9578 URL: https://source.winehq.org/git/wine.git/?a=commit;h=b8d0118517e8a42a7987dc7c… Author: Connor McAdams <cmcadams(a)codeweavers.com> Date: Mon Sep 27 16:11:20 2021 +0200 oleacc: Add put_accValue function for edit client accessible object. Signed-off-by: Connor McAdams <cmcadams(a)codeweavers.com> Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/oleacc/client.c | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/dlls/oleacc/client.c b/dlls/oleacc/client.c index 1dd18240179..192a006bdba 100644 --- a/dlls/oleacc/client.c +++ b/dlls/oleacc/client.c @@ -48,6 +48,7 @@ struct win_class_vtbl { HRESULT (*get_name)(Client*, VARIANT, BSTR*); HRESULT (*get_kbd_shortcut)(Client*, VARIANT, BSTR*); HRESULT (*get_value)(Client*, VARIANT, BSTR*); + HRESULT (*put_value)(Client*, VARIANT, BSTR); }; static HRESULT win_get_name(HWND hwnd, BSTR *name) @@ -493,6 +494,9 @@ static HRESULT WINAPI Client_put_accValue(IAccessible *iface, VARIANT id, BSTR v TRACE("(%p)->(%s %p)\n", This, debugstr_variant(&id), value); + if(This->vtbl && This->vtbl->put_value) + return This->vtbl->put_value(This, id, value); + if(convert_child_id(&id) != CHILDID_SELF) return E_INVALIDARG; return S_FALSE; @@ -800,12 +804,22 @@ static HRESULT edit_get_value(Client *client, VARIANT id, BSTR *value_out) return S_OK; } +static HRESULT edit_put_value(Client *client, VARIANT id, BSTR value) +{ + if(convert_child_id(&id) != CHILDID_SELF || !IsWindow(client->hwnd)) + return E_INVALIDARG; + + SendMessageW(client->hwnd, WM_SETTEXT, 0, (LPARAM)value); + return S_OK; +} + static const win_class_vtbl edit_vtbl = { edit_init, edit_get_state, edit_get_name, edit_get_kbd_shortcut, edit_get_value, + edit_put_value, }; static const struct win_class_data classes[] = {

1 0

Piotr Caban : oleacc: Add default client IAccessible::put_accValue implementation.
by Alexandre Julliard 27 Sep '21

27 Sep '21

Module: wine Branch: master Commit: 38229c8a758f29f7cf81572f37972ce56da89cb0 URL: https://source.winehq.org/git/wine.git/?a=commit;h=38229c8a758f29f7cf81572f… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Mon Sep 27 16:11:16 2021 +0200 oleacc: Add default client IAccessible::put_accValue implementation. Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/oleacc/client.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/dlls/oleacc/client.c b/dlls/oleacc/client.c index f111a1b9af8..1dd18240179 100644 --- a/dlls/oleacc/client.c +++ b/dlls/oleacc/client.c @@ -487,11 +487,15 @@ static HRESULT WINAPI Client_put_accName(IAccessible *iface, VARIANT varID, BSTR return E_NOTIMPL; } -static HRESULT WINAPI Client_put_accValue(IAccessible *iface, VARIANT varID, BSTR pszValue) +static HRESULT WINAPI Client_put_accValue(IAccessible *iface, VARIANT id, BSTR value) { Client *This = impl_from_Client(iface); - FIXME("(%p)->(%s %s)\n", This, debugstr_variant(&varID), debugstr_w(pszValue)); - return E_NOTIMPL; + + TRACE("(%p)->(%s %p)\n", This, debugstr_variant(&id), value); + + if(convert_child_id(&id) != CHILDID_SELF) + return E_INVALIDARG; + return S_FALSE; } static const IAccessibleVtbl ClientVtbl = {

1 0

Connor McAdams : oleacc: Add get_accValue function for edit client accessible object.
by Alexandre Julliard 27 Sep '21

27 Sep '21

Module: wine Branch: master Commit: b23af7300a6e05edac67a1124213bfc919a38e20 URL: https://source.winehq.org/git/wine.git/?a=commit;h=b23af7300a6e05edac67a112… Author: Connor McAdams <cmcadams(a)codeweavers.com> Date: Mon Sep 27 16:11:13 2021 +0200 oleacc: Add get_accValue function for edit client accessible object. Signed-off-by: Connor McAdams <cmcadams(a)codeweavers.com> Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/oleacc/client.c | 35 +++++++++++++++++++++++++++++++---- 1 file changed, 31 insertions(+), 4 deletions(-) diff --git a/dlls/oleacc/client.c b/dlls/oleacc/client.c index 6419e8dec31..f111a1b9af8 100644 --- a/dlls/oleacc/client.c +++ b/dlls/oleacc/client.c @@ -47,6 +47,7 @@ struct win_class_vtbl { HRESULT (*get_state)(Client*, VARIANT, VARIANT*); HRESULT (*get_name)(Client*, VARIANT, BSTR*); HRESULT (*get_kbd_shortcut)(Client*, VARIANT, BSTR*); + HRESULT (*get_value)(Client*, VARIANT, BSTR*); }; static HRESULT win_get_name(HWND hwnd, BSTR *name) @@ -228,14 +229,17 @@ static HRESULT WINAPI Client_get_accName(IAccessible *iface, VARIANT id, BSTR *n return win_get_name(This->hwnd, name); } -static HRESULT WINAPI Client_get_accValue(IAccessible *iface, VARIANT varID, BSTR *pszValue) +static HRESULT WINAPI Client_get_accValue(IAccessible *iface, VARIANT id, BSTR *value) { Client *This = impl_from_Client(iface); - TRACE("(%p)->(%s %p)\n", This, debugstr_variant(&varID), pszValue); + TRACE("(%p)->(%s %p)\n", This, debugstr_variant(&id), value); - *pszValue = NULL; - if(convert_child_id(&varID) != CHILDID_SELF) + *value = NULL; + if(This->vtbl && This->vtbl->get_value) + return This->vtbl->get_value(This, id, value); + + if(convert_child_id(&id) != CHILDID_SELF) return E_INVALIDARG; return S_FALSE; } @@ -770,11 +774,34 @@ static HRESULT edit_get_kbd_shortcut(Client *client, VARIANT id, BSTR *shortcut) return win_get_kbd_shortcut(label, shortcut); } +static HRESULT edit_get_value(Client *client, VARIANT id, BSTR *value_out) +{ + WCHAR *buf; + UINT len; + + if(convert_child_id(&id) != CHILDID_SELF) + return E_INVALIDARG; + + if(GetWindowLongW(client->hwnd, GWL_STYLE) & ES_PASSWORD) + return E_ACCESSDENIED; + + len = SendMessageW(client->hwnd, WM_GETTEXTLENGTH, 0, 0); + buf = heap_alloc_zero((len + 1) * sizeof(*buf)); + if(!buf) + return E_OUTOFMEMORY; + + SendMessageW(client->hwnd, WM_GETTEXT, len + 1, (LPARAM)buf); + *value_out = SysAllocString(buf); + heap_free(buf); + return S_OK; +} + static const win_class_vtbl edit_vtbl = { edit_init, edit_get_state, edit_get_name, edit_get_kbd_shortcut, + edit_get_value, }; static const struct win_class_data classes[] = {

1 0

Connor McAdams : oleacc: Add get_accKeyboardShortcut function for edit client accessible object.
by Alexandre Julliard 27 Sep '21

27 Sep '21

Module: wine Branch: master Commit: dbdebac3166f79207eef7fcd1ff7d7dfcf725924 URL: https://source.winehq.org/git/wine.git/?a=commit;h=dbdebac3166f79207eef7fcd… Author: Connor McAdams <cmcadams(a)codeweavers.com> Date: Mon Sep 27 16:11:09 2021 +0200 oleacc: Add get_accKeyboardShortcut function for edit client accessible object. Signed-off-by: Connor McAdams <cmcadams(a)codeweavers.com> Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/oleacc/client.c | 67 ++++++++++++++++++++++++++++++++++++---------------- 1 file changed, 47 insertions(+), 20 deletions(-) diff --git a/dlls/oleacc/client.c b/dlls/oleacc/client.c index 032f8867d31..6419e8dec31 100644 --- a/dlls/oleacc/client.c +++ b/dlls/oleacc/client.c @@ -46,6 +46,7 @@ struct win_class_vtbl { void (*init)(Client*); HRESULT (*get_state)(Client*, VARIANT, VARIANT*); HRESULT (*get_name)(Client*, VARIANT, BSTR*); + HRESULT (*get_kbd_shortcut)(Client*, VARIANT, BSTR*); }; static HRESULT win_get_name(HWND hwnd, BSTR *name) @@ -69,6 +70,29 @@ static HRESULT win_get_name(HWND hwnd, BSTR *name) return *name ? S_OK : E_OUTOFMEMORY; } +static HRESULT win_get_kbd_shortcut(HWND hwnd, BSTR *shortcut) +{ + WCHAR buf[1024]; + UINT i, len; + + len = SendMessageW(hwnd, WM_GETTEXT, ARRAY_SIZE(buf), (LPARAM)buf); + if(!len) + return S_FALSE; + + for(i=0; i<len; i++) { + if(buf[i] == '&') + break; + } + if(i+1 >= len) + return S_FALSE; + + *shortcut = SysAllocString(L"Alt+!"); + if(!*shortcut) + return E_OUTOFMEMORY; + (*shortcut)[4] = buf[i+1]; + return S_OK; +} + static inline Client* impl_from_Client(IAccessible *iface) { return CONTAINING_RECORD(iface, Client, IAccessible_iface); @@ -304,32 +328,20 @@ static HRESULT WINAPI Client_get_accHelpTopic(IAccessible *iface, } static HRESULT WINAPI Client_get_accKeyboardShortcut(IAccessible *iface, - VARIANT varID, BSTR *pszKeyboardShortcut) + VARIANT id, BSTR *shortcut) { Client *This = impl_from_Client(iface); - WCHAR name[1024]; - UINT i, len; - - TRACE("(%p)->(%s %p)\n", This, debugstr_variant(&varID), pszKeyboardShortcut); - *pszKeyboardShortcut = NULL; - if(convert_child_id(&varID) != CHILDID_SELF) - return E_INVALIDARG; + TRACE("(%p)->(%s %p)\n", This, debugstr_variant(&id), shortcut); - len = SendMessageW(This->hwnd, WM_GETTEXT, ARRAY_SIZE(name), (LPARAM)name); - for(i=0; i<len; i++) { - if(name[i] == '&') - break; - } - if(i+1 >= len) - return S_FALSE; + *shortcut = NULL; + if(This->vtbl && This->vtbl->get_kbd_shortcut) + return This->vtbl->get_kbd_shortcut(This, id, shortcut); - *pszKeyboardShortcut = SysAllocString(L"Alt+!"); - if(!*pszKeyboardShortcut) - return E_OUTOFMEMORY; + if(convert_child_id(&id) != CHILDID_SELF) + return E_INVALIDARG; - (*pszKeyboardShortcut)[4] = name[i+1]; - return S_OK; + return win_get_kbd_shortcut(This->hwnd, shortcut); } static HRESULT WINAPI Client_get_accFocus(IAccessible *iface, VARIANT *focus) @@ -744,10 +756,25 @@ static HRESULT edit_get_name(Client *client, VARIANT id, BSTR *name) return win_get_name(label, name); } +static HRESULT edit_get_kbd_shortcut(Client *client, VARIANT id, BSTR *shortcut) +{ + HWND label; + + if(convert_child_id(&id) != CHILDID_SELF) + return E_INVALIDARG; + + label = edit_find_label(client->hwnd, TRUE); + if(!label) + return S_FALSE; + + return win_get_kbd_shortcut(label, shortcut); +} + static const win_class_vtbl edit_vtbl = { edit_init, edit_get_state, edit_get_name, + edit_get_kbd_shortcut, }; static const struct win_class_data classes[] = {

1 0

Connor McAdams : oleacc: Add get_accName function for edit client accessible object.
by Alexandre Julliard 27 Sep '21

27 Sep '21

Module: wine Branch: master Commit: 583caf555d123ad15d96b84ddd42eca4e5896292 URL: https://source.winehq.org/git/wine.git/?a=commit;h=583caf555d123ad15d96b84d… Author: Connor McAdams <cmcadams(a)codeweavers.com> Date: Mon Sep 27 16:11:05 2021 +0200 oleacc: Add get_accName function for edit client accessible object. Signed-off-by: Connor McAdams <cmcadams(a)codeweavers.com> Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/oleacc/client.c | 92 ++++++++++++++++++++++++++++++++++++++++------------ 1 file changed, 72 insertions(+), 20 deletions(-) diff --git a/dlls/oleacc/client.c b/dlls/oleacc/client.c index c94be37df0f..032f8867d31 100644 --- a/dlls/oleacc/client.c +++ b/dlls/oleacc/client.c @@ -45,8 +45,30 @@ typedef struct { struct win_class_vtbl { void (*init)(Client*); HRESULT (*get_state)(Client*, VARIANT, VARIANT*); + HRESULT (*get_name)(Client*, VARIANT, BSTR*); }; +static HRESULT win_get_name(HWND hwnd, BSTR *name) +{ + WCHAR buf[1024]; + UINT i, len; + + len = SendMessageW(hwnd, WM_GETTEXT, ARRAY_SIZE(buf), (LPARAM)buf); + if(!len) + return S_FALSE; + + for(i=0; i<len; i++) { + if(buf[i] == '&') { + len--; + memmove(buf+i, buf+i+1, (len-i)*sizeof(WCHAR)); + break; + } + } + + *name = SysAllocStringLen(buf, len); + return *name ? S_OK : E_OUTOFMEMORY; +} + static inline Client* impl_from_Client(IAccessible *iface) { return CONTAINING_RECORD(iface, Client, IAccessible_iface); @@ -166,32 +188,20 @@ static HRESULT WINAPI Client_get_accChild(IAccessible *iface, return E_INVALIDARG; } -static HRESULT WINAPI Client_get_accName(IAccessible *iface, VARIANT varID, BSTR *pszName) +static HRESULT WINAPI Client_get_accName(IAccessible *iface, VARIANT id, BSTR *name) { Client *This = impl_from_Client(iface); - WCHAR name[1024]; - UINT i, len; - - TRACE("(%p)->(%s %p)\n", This, debugstr_variant(&varID), pszName); - *pszName = NULL; - if(convert_child_id(&varID) != CHILDID_SELF || !IsWindow(This->hwnd)) - return E_INVALIDARG; + TRACE("(%p)->(%s %p)\n", This, debugstr_variant(&id), name); - len = SendMessageW(This->hwnd, WM_GETTEXT, ARRAY_SIZE(name), (LPARAM)name); - if(!len) - return S_FALSE; + *name = NULL; + if(This->vtbl && This->vtbl->get_name) + return This->vtbl->get_name(This, id, name); - for(i=0; i<len; i++) { - if(name[i] == '&') { - len--; - memmove(name+i, name+i+1, (len-i)*sizeof(WCHAR)); - break; - } - } + if(convert_child_id(&id) != CHILDID_SELF || !IsWindow(This->hwnd)) + return E_INVALIDARG; - *pszName = SysAllocStringLen(name, len); - return *pszName ? S_OK : E_OUTOFMEMORY; + return win_get_name(This->hwnd, name); } static HRESULT WINAPI Client_get_accValue(IAccessible *iface, VARIANT varID, BSTR *pszValue) @@ -693,9 +703,51 @@ static HRESULT edit_get_state(Client *client, VARIANT id, VARIANT *state) return S_OK; } +/* + * Edit control objects have their name property defined by the first static + * text control preceding them in the order of window creation. If one is not + * found, the edit has no name property. In the case of the keyboard shortcut + * property, the first preceding visible static text control is used. + */ +static HWND edit_find_label(HWND hwnd, BOOL visible) +{ + HWND cur; + + for(cur = hwnd; cur; cur = GetWindow(cur, GW_HWNDPREV)) { + WCHAR class_name[64]; + + if(!RealGetWindowClassW(cur, class_name, ARRAY_SIZE(class_name))) + continue; + + if(!wcsicmp(class_name, WC_STATICW)) { + if(visible && !(GetWindowLongW(cur, GWL_STYLE) & WS_VISIBLE)) + continue; + else + break; + } + } + + return cur; +} + +static HRESULT edit_get_name(Client *client, VARIANT id, BSTR *name) +{ + HWND label; + + if(convert_child_id(&id) != CHILDID_SELF || !IsWindow(client->hwnd)) + return E_INVALIDARG; + + label = edit_find_label(client->hwnd, FALSE); + if(!label) + return S_FALSE; + + return win_get_name(label, name); +} + static const win_class_vtbl edit_vtbl = { edit_init, edit_get_state, + edit_get_name, }; static const struct win_class_data classes[] = {

1 0

Connor McAdams : oleacc: Add get_accState function for edit client accessible object.
by Alexandre Julliard 27 Sep '21

27 Sep '21

Module: wine Branch: master Commit: 44fba34992ee4a7fbd0f4e18267304f7bb42e5d2 URL: https://source.winehq.org/git/wine.git/?a=commit;h=44fba34992ee4a7fbd0f4e18… Author: Connor McAdams <cmcadams(a)codeweavers.com> Date: Mon Sep 27 16:10:59 2021 +0200 oleacc: Add get_accState function for edit client accessible object. Signed-off-by: Connor McAdams <cmcadams(a)codeweavers.com> Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/oleacc/client.c | 60 +++++++++++++++++++++++++++++++++++++++------------- 1 file changed, 45 insertions(+), 15 deletions(-) diff --git a/dlls/oleacc/client.c b/dlls/oleacc/client.c index fe9026e5bc1..c94be37df0f 100644 --- a/dlls/oleacc/client.c +++ b/dlls/oleacc/client.c @@ -18,6 +18,7 @@ #define COBJMACROS +#include <assert.h> #include "oleacc_private.h" #include "commctrl.h" @@ -43,6 +44,7 @@ typedef struct { struct win_class_vtbl { void (*init)(Client*); + HRESULT (*get_state)(Client*, VARIANT, VARIANT*); }; static inline Client* impl_from_Client(IAccessible *iface) @@ -233,36 +235,44 @@ static HRESULT WINAPI Client_get_accRole(IAccessible *iface, VARIANT varID, VARI return S_OK; } -static HRESULT WINAPI Client_get_accState(IAccessible *iface, VARIANT varID, VARIANT *pvarState) +static HRESULT client_get_state(Client *client, VARIANT id, VARIANT *state) { - Client *This = impl_from_Client(iface); GUITHREADINFO info; LONG style; - TRACE("(%p)->(%s %p)\n", This, debugstr_variant(&varID), pvarState); - - if(convert_child_id(&varID) != CHILDID_SELF) { - V_VT(pvarState) = VT_EMPTY; + if(convert_child_id(&id) != CHILDID_SELF) { + V_VT(state) = VT_EMPTY; return E_INVALIDARG; } - V_VT(pvarState) = VT_I4; - V_I4(pvarState) = 0; + V_VT(state) = VT_I4; + V_I4(state) = 0; - style = GetWindowLongW(This->hwnd, GWL_STYLE); + style = GetWindowLongW(client->hwnd, GWL_STYLE); if(style & WS_DISABLED) - V_I4(pvarState) |= STATE_SYSTEM_UNAVAILABLE; - else if(IsWindow(This->hwnd)) - V_I4(pvarState) |= STATE_SYSTEM_FOCUSABLE; + V_I4(state) |= STATE_SYSTEM_UNAVAILABLE; + else if(IsWindow(client->hwnd)) + V_I4(state) |= STATE_SYSTEM_FOCUSABLE; info.cbSize = sizeof(info); - if(GetGUIThreadInfo(0, &info) && info.hwndFocus == This->hwnd) - V_I4(pvarState) |= STATE_SYSTEM_FOCUSED; + if(GetGUIThreadInfo(0, &info) && info.hwndFocus == client->hwnd) + V_I4(state) |= STATE_SYSTEM_FOCUSED; if(!(style & WS_VISIBLE)) - V_I4(pvarState) |= STATE_SYSTEM_INVISIBLE; + V_I4(state) |= STATE_SYSTEM_INVISIBLE; return S_OK; } +static HRESULT WINAPI Client_get_accState(IAccessible *iface, VARIANT id, VARIANT *state) +{ + Client *This = impl_from_Client(iface); + + TRACE("(%p)->(%s %p)\n", This, debugstr_variant(&id), state); + + if(This->vtbl && This->vtbl->get_state) + return This->vtbl->get_state(This, id, state); + return client_get_state(This, id, state); +} + static HRESULT WINAPI Client_get_accHelp(IAccessible *iface, VARIANT varID, BSTR *pszHelp) { Client *This = impl_from_Client(iface); @@ -664,8 +674,28 @@ static void edit_init(Client *client) client->role = ROLE_SYSTEM_TEXT; } +static HRESULT edit_get_state(Client *client, VARIANT id, VARIANT *state) +{ + HRESULT hres; + LONG style; + + hres = client_get_state(client, id, state); + if(FAILED(hres)) + return hres; + + assert(V_VT(state) == VT_I4); + + style = GetWindowLongW(client->hwnd, GWL_STYLE); + if(style & ES_READONLY) + V_I4(state) |= STATE_SYSTEM_READONLY; + if(style & ES_PASSWORD) + V_I4(state) |= STATE_SYSTEM_PROTECTED; + return S_OK; +} + static const win_class_vtbl edit_vtbl = { edit_init, + edit_get_state, }; static const struct win_class_data classes[] = {

1 0

Connor McAdams : oleacc: Add get_accRole implementation for edit client accessible object.
by Alexandre Julliard 27 Sep '21

27 Sep '21

Module: wine Branch: master Commit: 8db8c473e978f75daf5235d087137e6787fd3373 URL: https://source.winehq.org/git/wine.git/?a=commit;h=8db8c473e978f75daf5235d0… Author: Connor McAdams <cmcadams(a)codeweavers.com> Date: Mon Sep 27 16:10:55 2021 +0200 oleacc: Add get_accRole implementation for edit client accessible object. Signed-off-by: Connor McAdams <cmcadams(a)codeweavers.com> Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/oleacc/client.c | 32 ++++++++++++++++++++++++++++---- dlls/oleacc/oleacc_private.h | 1 + 2 files changed, 29 insertions(+), 4 deletions(-) diff --git a/dlls/oleacc/client.c b/dlls/oleacc/client.c index bb3d5641a08..fe9026e5bc1 100644 --- a/dlls/oleacc/client.c +++ b/dlls/oleacc/client.c @@ -26,6 +26,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(oleacc); +typedef struct win_class_vtbl win_class_vtbl; typedef struct { IAccessible IAccessible_iface; IOleWindow IOleWindow_iface; @@ -35,8 +36,15 @@ typedef struct { HWND hwnd; HWND enum_pos; + INT role; + + const win_class_vtbl *vtbl; } Client; +struct win_class_vtbl { + void (*init)(Client*); +}; + static inline Client* impl_from_Client(IAccessible *iface) { return CONTAINING_RECORD(iface, Client, IAccessible_iface); @@ -221,7 +229,7 @@ static HRESULT WINAPI Client_get_accRole(IAccessible *iface, VARIANT varID, VARI } V_VT(pvarRole) = VT_I4; - V_I4(pvarRole) = ROLE_SYSTEM_CLIENT; + V_I4(pvarRole) = This->role; return S_OK; } @@ -651,12 +659,21 @@ static const IEnumVARIANTVtbl ClientEnumVARIANTVtbl = { Client_EnumVARIANT_Clone }; +static void edit_init(Client *client) +{ + client->role = ROLE_SYSTEM_TEXT; +} + +static const win_class_vtbl edit_vtbl = { + edit_init, +}; + static const struct win_class_data classes[] = { {WC_LISTBOXW, 0x10000, TRUE}, {L"#32768", 0x10001, TRUE}, /* menu */ {WC_BUTTONW, 0x10002, TRUE}, {WC_STATICW, 0x10003, TRUE}, - {WC_EDITW, 0x10004, TRUE}, + {WC_EDITW, 0x10004, FALSE, &edit_vtbl}, {WC_COMBOBOXW, 0x10005, TRUE}, {L"#32770", 0x10006, TRUE}, /* dialog */ {L"#32771", 0x10007, TRUE}, /* winswitcher */ @@ -685,8 +702,9 @@ static const struct win_class_data classes[] = { HRESULT create_client_object(HWND hwnd, const IID *iid, void **obj) { + const struct win_class_data *data; Client *client; - HRESULT hres; + HRESULT hres = S_OK; if(!IsWindow(hwnd)) return E_FAIL; @@ -695,7 +713,7 @@ HRESULT create_client_object(HWND hwnd, const IID *iid, void **obj) if(!client) return E_OUTOFMEMORY; - find_class_data(hwnd, classes); + data = find_class_data(hwnd, classes); client->IAccessible_iface.lpVtbl = &ClientVtbl; client->IOleWindow_iface.lpVtbl = &ClientOleWindowVtbl; @@ -703,6 +721,12 @@ HRESULT create_client_object(HWND hwnd, const IID *iid, void **obj) client->ref = 1; client->hwnd = hwnd; client->enum_pos = 0; + client->role = ROLE_SYSTEM_CLIENT; + + if(data) + client->vtbl = data->vtbl; + if(client->vtbl && client->vtbl->init) + client->vtbl->init(client); hres = IAccessible_QueryInterface(&client->IAccessible_iface, iid, obj); IAccessible_Release(&client->IAccessible_iface); diff --git a/dlls/oleacc/oleacc_private.h b/dlls/oleacc/oleacc_private.h index 64926b6abb7..a01fc3615fe 100644 --- a/dlls/oleacc/oleacc_private.h +++ b/dlls/oleacc/oleacc_private.h @@ -22,6 +22,7 @@ struct win_class_data { const WCHAR *name; DWORD idx; BOOL stub; + const void *vtbl; }; const struct win_class_data* find_class_data(HWND, const struct win_class_data*) DECLSPEC_HIDDEN;

1 0

Piotr Caban : oleacc: Reorganize class specific behaviour handling.
by Alexandre Julliard 27 Sep '21

27 Sep '21

Module: wine Branch: master Commit: 589be8c1be83150754cee321912e7eb56ee4e14a URL: https://source.winehq.org/git/wine.git/?a=commit;h=589be8c1be83150754cee321… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Mon Sep 27 16:10:51 2021 +0200 oleacc: Reorganize class specific behaviour handling. Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/oleacc/client.c | 35 ++++++++++++++++++ dlls/oleacc/main.c | 82 ++++++------------------------------------ dlls/oleacc/oleacc_private.h | 7 ++++ dlls/oleacc/tests/main.c | 85 ++++++++++++++++++++++++++++++++++++++++++++ dlls/oleacc/window.c | 9 +++++ 5 files changed, 146 insertions(+), 72 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=589be8c1be83150754ce…

1 0

Chip Davis : winegcc: Only add -static-libgcc if -nodefaultlibs weren't given. (Clang).
by Alexandre Julliard 27 Sep '21

27 Sep '21

Module: wine Branch: master Commit: 9e133a9a22aee61f7b402e446fdf34cc984a2dbd URL: https://source.winehq.org/git/wine.git/?a=commit;h=9e133a9a22aee61f7b402e44… Author: Chip Davis <cdavis(a)codeweavers.com> Date: Fri Sep 24 19:56:23 2021 -0500 winegcc: Only add -static-libgcc if -nodefaultlibs weren't given. (Clang). The flag does nothing if -nodefaultlibs or -nostdlib were given, because the driver does not automatically link to libgcc/compiler-rt in that case. Clang will warn about this, which is annoying, which clutters up the output, and which makes it impossible to use -Werror when building with Clang. Signed-off-by: Chip Davis <cdavis(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/winegcc/winegcc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/winegcc/winegcc.c b/tools/winegcc/winegcc.c index 8e72c6bcfee..2bf13570bba 100644 --- a/tools/winegcc/winegcc.c +++ b/tools/winegcc/winegcc.c @@ -538,7 +538,8 @@ static struct strarray get_link_args( struct options *opts, const char *output_n strarray_add( &flags, "-Wl,--large-address-aware" ); /* make sure we don't need a libgcc_s dll on Windows */ - strarray_add( &flags, "-static-libgcc" ); + if (!opts->nodefaultlibs && !opts->use_msvcrt) + strarray_add( &flags, "-static-libgcc" ); if (opts->debug_file && strendswith(opts->debug_file, ".pdb")) strarray_add(&link_args, strmake("-Wl,-pdb,%s", opts->debug_file));

1 0

Nikolay Sivov : ole32/composite: Fix argument handling in ComposeWith().
by Alexandre Julliard 27 Sep '21

27 Sep '21

Module: wine Branch: master Commit: 281021d820307fa639e266cdc709931806ced534 URL: https://source.winehq.org/git/wine.git/?a=commit;h=281021d820307fa639e266cd… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Mon Sep 27 15:31:40 2021 +0300 ole32/composite: Fix argument handling in ComposeWith(). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ole32/compositemoniker.c | 24 +++++------------------- dlls/ole32/tests/moniker.c | 18 ++++++++++++++++++ 2 files changed, 23 insertions(+), 19 deletions(-) diff --git a/dlls/ole32/compositemoniker.c b/dlls/ole32/compositemoniker.c index 60d11b3ba90..40889ae42b2 100644 --- a/dlls/ole32/compositemoniker.c +++ b/dlls/ole32/compositemoniker.c @@ -485,28 +485,14 @@ CompositeMonikerImpl_Reduce(IMoniker* iface, IBindCtx* pbc, DWORD dwReduceHowFar } } -/****************************************************************************** - * CompositeMoniker_ComposeWith - ******************************************************************************/ -static HRESULT WINAPI -CompositeMonikerImpl_ComposeWith(IMoniker* iface, IMoniker* pmkRight, - BOOL fOnlyIfNotGeneric, IMoniker** ppmkComposite) +static HRESULT WINAPI CompositeMonikerImpl_ComposeWith(IMoniker *iface, IMoniker *right, + BOOL only_if_not_generic, IMoniker **composite) { - TRACE("(%p,%p,%d,%p)\n",iface,pmkRight,fOnlyIfNotGeneric,ppmkComposite); - - if ((ppmkComposite==NULL)||(pmkRight==NULL)) - return E_POINTER; - - *ppmkComposite=0; - - /* If fOnlyIfNotGeneric is TRUE, this method sets *pmkComposite to NULL and returns MK_E_NEEDGENERIC; */ - /* otherwise, the method returns the result of combining the two monikers by calling the */ - /* CreateGenericComposite function */ + TRACE("%p, %p, %d, %p.\n", iface, right, only_if_not_generic, composite); - if (fOnlyIfNotGeneric) - return MK_E_NEEDGENERIC; + *composite = NULL; - return CreateGenericComposite(iface,pmkRight,ppmkComposite); + return only_if_not_generic ? MK_E_NEEDGENERIC : CreateGenericComposite(iface, right, composite); } static void composite_get_components(IMoniker *moniker, IMoniker **components, unsigned int *index) diff --git a/dlls/ole32/tests/moniker.c b/dlls/ole32/tests/moniker.c index 962ae6c269f..7d88c132d3d 100644 --- a/dlls/ole32/tests/moniker.c +++ b/dlls/ole32/tests/moniker.c @@ -3359,6 +3359,24 @@ todo_wine IMoniker_Release(moniker2); IMoniker_Release(moniker1); + /* ComposeWith() */ + hr = create_moniker_from_desc("CI1I2", &moniker1); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + hr = create_moniker_from_desc("I3", &moniker2); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + + hr = IMoniker_ComposeWith(moniker1, NULL, FALSE, &moniker); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + ok(moniker == moniker1, "Unexpected pointer.\n"); + IMoniker_Release(moniker); + + hr = IMoniker_ComposeWith(moniker1, NULL, TRUE, &moniker); + ok(hr == MK_E_NEEDGENERIC, "Unexpected hr %#x.\n", hr); + ok(!moniker, "Unexpected pointer.\n"); + + IMoniker_Release(moniker2); + IMoniker_Release(moniker1); + IBindCtx_Release(bindctx); }

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits September 2021