April 2022 - Wine-Commits - list.winehq.org

Francois Gouget : testbot/web: Simplify linking patches to the job details pages.
by Alexandre Julliard April 18, 2022

April 18, 2022

Module: tools Branch: master Commit: 74981370d37c7951ed15268d7d069a62c1520adf URL: https://source.winehq.org/git/tools.git/?a=commit;h=74981370d37c7951ed15268… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Mon Apr 18 10:55:05 2022 +0200 testbot/web: Simplify linking patches to the job details pages. There is no need to call uri_escape() on integers. Also the URL is simple (no ampersand) and does not need to go through escapeHTML(). Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- testbot/web/PatchesList.pl | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/testbot/web/PatchesList.pl b/testbot/web/PatchesList.pl index 943b9c5..48c8382 100644 --- a/testbot/web/PatchesList.pl +++ b/testbot/web/PatchesList.pl @@ -25,7 +25,6 @@ package PatchesBlock; use ObjectModel::CGI::CollectionBlock; our @ISA = qw(ObjectModel::CGI::CollectionBlock); -use URI::Escape; use WineTestBot::Patches; @@ -54,9 +53,7 @@ sub GenerateDataView($$$) $Row->{Item}->Disposition =~ /job ([0-9]+)$/) { my $JobId = $1; - my $URI = "/JobDetails.pl?Key=" . uri_escape($JobId); - print "<a href='" . $self->escapeHTML($URI) . "'>" . - "Job " . $self->escapeHTML($JobId) . "</a>"; + print "<a href='/JobDetails.pl?Key=$JobId'>Job $JobId</a>"; } else {

1 0

Francois Gouget : testbot/cgi: Introduce ValueFormatter to handle value formatting.
by Alexandre Julliard April 18, 2022

April 18, 2022

Module: tools Branch: master Commit: cf5985ccd262fdce0eef2127b5a0cddc62590731 URL: https://source.winehq.org/git/tools.git/?a=commit;h=cf5985ccd262fdce0eef212… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Mon Apr 18 10:32:36 2022 +0200 testbot/cgi: Introduce ValueFormatter to handle value formatting. This allows sharing code between CollectionBlock and FormPage. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- testbot/lib/ObjectModel/CGI/CollectionBlock.pm | 38 ++++---------- testbot/lib/ObjectModel/CGI/FormPage.pm | 11 ++-- testbot/lib/ObjectModel/CGI/ValueFormatter.pm | 69 ++++++++++++++++++++++++++ 3 files changed, 82 insertions(+), 36 deletions(-) diff --git a/testbot/lib/ObjectModel/CGI/CollectionBlock.pm b/testbot/lib/ObjectModel/CGI/CollectionBlock.pm index 435c44e..36f1b01 100644 --- a/testbot/lib/ObjectModel/CGI/CollectionBlock.pm +++ b/testbot/lib/ObjectModel/CGI/CollectionBlock.pm @@ -47,6 +47,8 @@ our @EXPORT = qw(new); use POSIX qw(strftime); use URI::Escape; +use ObjectModel::CGI::ValueFormatter; + =pod =over 12 @@ -280,35 +282,15 @@ sub GenerateDataView($$$) my $PropertyName = $PropertyDescriptor->GetName(); my $Value = $Row->{Item}->$PropertyName; - if ($PropertyDescriptor->GetClass() eq "Itemref" and defined $Value) + if ($PropertyDescriptor->GetClass() eq "Basic" and + $PropertyDescriptor->GetType() eq "DT" and defined $Value) { - # Note: This only supports single-key Itemrefs - foreach my $ValuePD (@{$Value->GetPropertyDescriptors()}) - { - if ($ValuePD->GetIsKey()) - { - $PropertyName = $ValuePD->GetName(); - print $self->escapeHTML($Value->$PropertyName); - return; - } - } - } - elsif ($PropertyDescriptor->GetClass() eq "Basic") - { - if ($PropertyDescriptor->GetType() eq "B") - { - print $Value ? "Yes" : "No"; - return; - } - if ($PropertyDescriptor->GetType() eq "DT" and defined $Value) - { - print "<script type='text/javascript'></script><noscript><div>", - strftime("%Y-%m-%d %H:%M:%S", localtime($Value)), - "</div></noscript>\n"; - return; - } + print "<script type='text/javascript'></script><noscript><div>", + strftime("%Y-%m-%d %H:%M:%S", localtime($Value)), + "</div></noscript>\n"; + return; } print defined $Value ? $self->escapeHTML($Value) : " "; } diff --git a/testbot/lib/ObjectModel/CGI/FormPage.pm b/testbot/lib/ObjectModel/CGI/FormPage.pm index 4f29efe..7fd2f56 100644 --- a/testbot/lib/ObjectModel/CGI/FormPage.pm +++ b/testbot/lib/ObjectModel/CGI/FormPage.pm @@ -57,6 +57,8 @@ $MyFormPage->GeneratePage(); use ObjectModel::CGI::Page; our @ISA = qw(ObjectModel::CGI::Page); +use ObjectModel::CGI::ValueFormatter; + =pod =over 12 @@ -280,14 +282,7 @@ sub GenerateValueView($$$) { my ($self, $PropertyDescriptor, $Value) = @_; - if ($PropertyDescriptor->GetClass() eq "Basic") - { - if ($PropertyDescriptor->GetType() eq "B") - { - print $Value ? "Yes" : "No"; - } - } - print defined $Value ? $self->escapeHTML($Value) : " "; + GenerateValueHTML($self, $PropertyDescriptor, $Value); } sub GetInputType($$) diff --git a/testbot/lib/ObjectModel/CGI/ValueFormatter.pm b/testbot/lib/ObjectModel/CGI/ValueFormatter.pm new file mode 100644 index 0000000..cad6eee --- /dev/null +++ b/testbot/lib/ObjectModel/CGI/ValueFormatter.pm @@ -0,0 +1,69 @@ +# -*- Mode: Perl; perl-indent-level: 2; indent-tabs-mode: nil -*- +# Generates HTML providing a user-readable representation of a given value +# +# Copyright 2022 Francois Gouget +# +# This library is free software; you can redistribute it and/or +# modify it under the terms of the GNU Lesser General Public +# License as published by the Free Software Foundation; either +# version 2.1 of the License, or (at your option) any later version. +# +# This library is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU +# Lesser General Public License for more details. +# +# You should have received a copy of the GNU Lesser General Public +# License along with this library; if not, write to the Free Software +# Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + +use strict; + +package ObjectModel::CGI::ValueFormatter; + +use Exporter 'import'; +our @EXPORT = qw(GenerateValueHTML); + + +=pod +=over 12 + +=item C<GenerateValueHTML()> + +Generates an HTML snippet representing the value in a user-readable form. + +The Parent parameter must point to a class that implements the escapeHTML() +method. + +=back +=cut + +sub GenerateValueHTML($$$); + +sub GenerateValueHTML($$$) +{ + my ($Parent, $PropertyDescriptor, $Value) = @_; + + if ($PropertyDescriptor->GetClass() eq "Basic" and + $PropertyDescriptor->GetType() eq "B") + { + print $Value ? "Yes" : "No"; + return; + } + if ($PropertyDescriptor->GetClass() eq "Itemref" and defined $Value) + { + # Note: This only supports single-key Itemrefs + foreach my $ValuePD (@{$Value->GetPropertyDescriptors()}) + { + if ($ValuePD->GetIsKey()) + { + my $PropertyName = $ValuePD->GetName(); + GenerateValueHTML($Parent, $ValuePD, $Value->$PropertyName); + return; + } + } + } + print defined $Value ? $Parent->escapeHTML($Value) : " "; +} + +1;

1 0

Francois Gouget : testbot: Add User::AddRole().
by Alexandre Julliard April 18, 2022

April 18, 2022

Module: tools Branch: master Commit: eb3a8541027bbec95f2e1dd3b86616fcb9c3e021 URL: https://source.winehq.org/git/tools.git/?a=commit;h=eb3a8541027bbec95f2e1dd… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Mon Apr 18 10:32:52 2022 +0200 testbot: Add User::AddRole(). This simplifies adding a role to a user, particularly with regards to making sure the user's Roles collection is up to date. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- testbot/lib/WineTestBot/Users.pm | 21 ++++++++++++--------- 1 file changed, 12 insertions(+), 9 deletions(-) diff --git a/testbot/lib/WineTestBot/Users.pm b/testbot/lib/WineTestBot/Users.pm index bd9cf35..f2f2a19 100644 --- a/testbot/lib/WineTestBot/Users.pm +++ b/testbot/lib/WineTestBot/Users.pm @@ -106,22 +106,25 @@ sub GenerateResetCode($) } } +sub AddRole($$) +{ + my ($self, $Role) = @_; + + my $UserRole = $self->Roles->Add(); + my $OldKey = $UserRole->GetKey(); + $UserRole->Role($Role); + $self->Roles->KeyChanged($OldKey, $Role->GetKey()); +} + sub AddDefaultRoles($) { my ($self) = @_; - my $UserRoles = $self->Roles; my $DefaultRoles = CreateRoles(); $DefaultRoles->AddFilter("IsDefaultRole", [1]); - foreach my $RoleKey (@{$DefaultRoles->GetKeys()}) + foreach my $Role (@{$DefaultRoles->GetItems()}) { - if (! defined($UserRoles->GetItem($RoleKey))) - { - my $NewRole = $UserRoles->Add(); - my $OldKey = $NewRole->GetKey(); - $NewRole->Role($DefaultRoles->GetItem($RoleKey)); - $UserRoles->KeyChanged($OldKey, $RoleKey); - } + $self->AddRole($Role) if (!$self->Roles->GetItem($Role->Name)); } }

1 0

Jacek Caban : winex11: Directly use win32u in more places.
by Alexandre Julliard April 15, 2022

April 15, 2022

Module: wine Branch: master Commit: 262831bc63dca2d63171aa98d19e8f2566907dbc URL: https://source.winehq.org/git/wine.git/?a=commit;h=262831bc63dca2d63171aa98… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Apr 15 14:52:20 2022 +0200 winex11: Directly use win32u in more places. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winex11.drv/desktop.c | 24 ++++++++++++------------ dlls/winex11.drv/display.c | 5 +++-- dlls/winex11.drv/keyboard.c | 18 +++++++++--------- dlls/winex11.drv/vulkan.c | 4 ++-- dlls/winex11.drv/window.c | 4 ++-- dlls/winex11.drv/x11drv.h | 10 ++++++++++ dlls/winex11.drv/xrandr.c | 2 +- 7 files changed, 39 insertions(+), 28 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=262831bc63dca2d63171…

1 0

Jacek Caban : win32u: Move NtUserCheckMenuItem implementation from user32.
by Alexandre Julliard April 15, 2022

April 15, 2022

Module: wine Branch: master Commit: a4d6198ac597b0cfd1f0a89ba71e12ea772352ce URL: https://source.winehq.org/git/wine.git/?a=commit;h=a4d6198ac597b0cfd1f0a89b… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Apr 15 14:53:09 2022 +0200 win32u: Move NtUserCheckMenuItem implementation from user32. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/mdi.c | 2 +- dlls/user32/menu.c | 43 ------------------ dlls/user32/user32.spec | 2 +- dlls/win32u/menu.c | 106 +++++++++++++++++++++++++++++++++++++++++++ dlls/win32u/ntuser_private.h | 24 +++++++++- dlls/win32u/syscall.c | 1 + dlls/win32u/win32u.spec | 2 +- dlls/wow64win/syscall.h | 1 + dlls/wow64win/user.c | 9 ++++ include/ntuser.h | 1 + 10 files changed, 143 insertions(+), 48 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=a4d6198ac597b0cfd1f0…

1 0

Floris Renaud : po: Update Dutch translation.
by Alexandre Julliard April 15, 2022

April 15, 2022

Module: wine Branch: master Commit: fa7c22e4583be847c0a830c87de2a6c7636adbce URL: https://source.winehq.org/git/wine.git/?a=commit;h=fa7c22e4583be847c0a830c8… Author: Floris Renaud <jkfloris(a)dds.nl> Date: Fri Apr 15 18:13:40 2022 +0200 po: Update Dutch translation. Signed-off-by: Floris Renaud <jkfloris(a)dds.nl> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- po/nl.po | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/po/nl.po b/po/nl.po index 659afc3bf40..8318d5fc54c 100644 --- a/po/nl.po +++ b/po/nl.po @@ -5,7 +5,7 @@ msgstr "" "Project-Id-Version: Wine\n" "Report-Msgid-Bugs-To: https://bugs.winehq.org\n" "POT-Creation-Date: N/A\n" -"PO-Revision-Date: 2022-03-23 17:00+0100\n" +"PO-Revision-Date: 2022-04-15 18:10+0200\n" "Last-Translator: Floris Renaud <jkfloris(a)dds.nl>\n" "Language-Team: Dutch\n" "Language: nl\n" @@ -4006,10 +4006,8 @@ msgid "Cannot modify non-writable property '|'" msgstr "Kan onschrijfbare eigenschap '|' niet veranderen" #: dlls/jscript/jscript.rc:79 -#, fuzzy -#| msgid "'this' is not a Map object" msgid "'this' is not a | object" -msgstr "'this' is geen Map object" +msgstr "'this' is geen | object" #: dlls/jscript/jscript.rc:80 msgid "Property cannot have both accessors and a value"

1 0

Jacek Caban : win32u: Move DrawMenuBar implementation from user32.
by Alexandre Julliard April 15, 2022

April 15, 2022

Module: wine Branch: master Commit: a2dcfd1d6d0e5715a4bbd30935b81454dd18def0 URL: https://source.winehq.org/git/wine.git/?a=commit;h=a2dcfd1d6d0e5715a4bbd309… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Apr 15 14:53:43 2022 +0200 win32u: Move DrawMenuBar implementation from user32. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/menu.c | 23 ++--------------------- dlls/win32u/menu.c | 28 ++++++++++++++++++++++++++++ dlls/win32u/win32u_private.h | 1 + dlls/win32u/window.c | 3 +++ include/ntuser.h | 6 ++++++ 5 files changed, 40 insertions(+), 21 deletions(-) diff --git a/dlls/user32/menu.c b/dlls/user32/menu.c index d1efb9d49d3..b1fdacefc9c 100644 --- a/dlls/user32/menu.c +++ b/dlls/user32/menu.c @@ -4367,28 +4367,9 @@ HMENU WINAPI GetSubMenu( HMENU hMenu, INT nPos ) /********************************************************************** * DrawMenuBar (USER32.@) */ -BOOL WINAPI DrawMenuBar( HWND hWnd ) +BOOL WINAPI DrawMenuBar( HWND hwnd ) { - HMENU hMenu; - - if (!IsWindow( hWnd )) - return FALSE; - if (is_win_menu_disallowed(hWnd)) - return TRUE; - - if ((hMenu = GetMenu( hWnd ))) - { - POPUPMENU *menu = grab_menu_ptr(hMenu); - if (menu) - { - menu->Height = 0; /* Make sure we call MENU_MenuBarCalcSize */ - menu->hwndOwner = hWnd; - release_menu_ptr(menu); - } - } - - return NtUserSetWindowPos( hWnd, 0, 0, 0, 0, 0, SWP_NOSIZE | SWP_NOMOVE | - SWP_NOACTIVATE | SWP_NOZORDER | SWP_FRAMECHANGED ); + return NtUserDrawMenuBar( hwnd ); } /*********************************************************************** diff --git a/dlls/win32u/menu.c b/dlls/win32u/menu.c index 0f02bb15c7a..b107bd7e8e0 100644 --- a/dlls/win32u/menu.c +++ b/dlls/win32u/menu.c @@ -196,6 +196,11 @@ static POPUPMENU *find_menu_item( HMENU handle, UINT id, UINT flags, UINT *pos ) return menu; } +static BOOL is_win_menu_disallowed( HWND hwnd ) +{ + return (get_window_long(hwnd, GWL_STYLE) & (WS_CHILD | WS_POPUP)) == WS_CHILD; +} + /* see GetMenu */ HMENU get_menu( HWND hwnd ) { @@ -319,3 +324,26 @@ BOOL WINAPI NtUserEnableMenuItem( HMENU handle, UINT id, UINT flags ) return oldflags; } + +/* see DrawMenuBar */ +BOOL draw_menu_bar( HWND hwnd ) +{ + HMENU handle; + + if (!is_window( hwnd )) return FALSE; + if (is_win_menu_disallowed( hwnd )) return TRUE; + + if ((handle = get_menu( hwnd ))) + { + POPUPMENU *menu = grab_menu_ptr( handle ); + if (menu) + { + menu->Height = 0; /* Make sure we call MENU_MenuBarCalcSize */ + menu->hwndOwner = hwnd; + release_menu_ptr( menu ); + } + } + + return NtUserSetWindowPos( hwnd, 0, 0, 0, 0, 0, SWP_NOSIZE | SWP_NOMOVE | + SWP_NOACTIVATE | SWP_NOZORDER | SWP_FRAMECHANGED ); +} diff --git a/dlls/win32u/win32u_private.h b/dlls/win32u/win32u_private.h index 7db37e4a32c..ceff548526b 100644 --- a/dlls/win32u/win32u_private.h +++ b/dlls/win32u/win32u_private.h @@ -355,6 +355,7 @@ extern BOOL set_foreground_window( HWND hwnd, BOOL mouse ) DECLSPEC_HIDDEN; /* menu.c */ extern HMENU create_menu( BOOL is_popup ) DECLSPEC_HIDDEN; +extern BOOL draw_menu_bar( HWND hwnd ) DECLSPEC_HIDDEN; extern HMENU get_menu( HWND hwnd ) DECLSPEC_HIDDEN; /* message.c */ diff --git a/dlls/win32u/window.c b/dlls/win32u/window.c index 3df544b1bb1..9e3b085ac32 100644 --- a/dlls/win32u/window.c +++ b/dlls/win32u/window.c @@ -5069,6 +5069,9 @@ ULONG_PTR WINAPI NtUserCallHwnd( HWND hwnd, DWORD code ) case NtUserCallHwnd_ArrangeIconicWindows: return arrange_iconic_windows( hwnd ); + case NtUserCallHwnd_DrawMenuBar: + return draw_menu_bar( hwnd ); + case NtUserCallHwnd_GetDpiForWindow: return get_dpi_for_window( hwnd ); diff --git a/include/ntuser.h b/include/ntuser.h index ded3096b76c..79ea57590bf 100644 --- a/include/ntuser.h +++ b/include/ntuser.h @@ -777,6 +777,7 @@ static inline BOOL NtUserUnhookWindowsHook( INT id, HOOKPROC proc ) enum { NtUserCallHwnd_ArrangeIconicWindows, + NtUserCallHwnd_DrawMenuBar, NtUserCallHwnd_GetDpiForWindow, NtUserCallHwnd_GetParent, NtUserCallHwnd_GetWindowContextHelpId, @@ -793,6 +794,11 @@ static inline UINT NtUserArrangeIconicWindows( HWND parent ) return NtUserCallHwnd( parent, NtUserCallHwnd_ArrangeIconicWindows ); } +static inline BOOL NtUserDrawMenuBar( HWND hwnd ) +{ + return NtUserCallHwnd( hwnd, NtUserCallHwnd_DrawMenuBar ); +} + static inline DWORD NtUserGetWindowContextHelpId( HWND hwnd ) { return NtUserCallHwnd( hwnd, NtUserCallHwnd_GetWindowContextHelpId );

1 0

Jacek Caban : win32u: Move CreatePopupMenu implementation from user32.
by Alexandre Julliard April 15, 2022

April 15, 2022

Module: wine Branch: master Commit: 72d53fda590126acb8cee72cb1fd208946d55d8c URL: https://source.winehq.org/git/wine.git/?a=commit;h=72d53fda590126acb8cee72c… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Apr 15 14:52:51 2022 +0200 win32u: Move CreatePopupMenu implementation from user32. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/menu.c | 10 ++-------- dlls/win32u/menu.c | 5 +++-- dlls/win32u/sysparams.c | 6 +++--- dlls/win32u/win32u_private.h | 2 +- include/ntuser.h | 12 ++++++------ 5 files changed, 15 insertions(+), 20 deletions(-) diff --git a/dlls/user32/menu.c b/dlls/user32/menu.c index 185d596fbb8..6d506e21c3c 100644 --- a/dlls/user32/menu.c +++ b/dlls/user32/menu.c @@ -4158,13 +4158,7 @@ BOOL WINAPI ModifyMenuA( HMENU hMenu, UINT pos, UINT flags, */ HMENU WINAPI CreatePopupMenu(void) { - HMENU hmenu; - POPUPMENU *menu; - - if (!(hmenu = CreateMenu())) return 0; - menu = MENU_GetMenu( hmenu ); - menu->wFlags |= MF_POPUP; - return hmenu; + return NtUserCreateMenu( TRUE ); } @@ -4213,7 +4207,7 @@ BOOL WINAPI SetMenuItemBitmaps( HMENU hMenu, UINT nPos, UINT wFlags, */ HMENU WINAPI CreateMenu(void) { - return NtUserCreateMenu(); + return NtUserCreateMenu( FALSE ); } diff --git a/dlls/win32u/menu.c b/dlls/win32u/menu.c index 6ae55e5cee1..2b1a5c97ac1 100644 --- a/dlls/win32u/menu.c +++ b/dlls/win32u/menu.c @@ -116,8 +116,8 @@ HMENU get_menu( HWND hwnd ) return UlongToHandle( get_window_long( hwnd, GWLP_ID )); } -/* see CreateMenu */ -HMENU create_menu(void) +/* see CreateMenu and CreatePopupMenu */ +HMENU create_menu( BOOL is_popup ) { POPUPMENU *menu; HMENU handle; @@ -125,6 +125,7 @@ HMENU create_menu(void) if (!(menu = calloc( 1, sizeof(*menu) ))) return 0; menu->FocusedItem = NO_SELECTED_ITEM; menu->refcount = 1; + if (is_popup) menu->wFlags |= MF_POPUP; if (!(handle = alloc_user_handle( &menu->obj, NTUSER_OBJ_MENU ))) free( menu ); diff --git a/dlls/win32u/sysparams.c b/dlls/win32u/sysparams.c index 4d2455fe64e..f6cbda9fa20 100644 --- a/dlls/win32u/sysparams.c +++ b/dlls/win32u/sysparams.c @@ -4632,9 +4632,6 @@ ULONG_PTR WINAPI NtUserCallNoParam( ULONG code ) { switch(code) { - case NtUserCallNoParam_CreateMenu: - return HandleToUlong( create_menu() ); - case NtUserCallNoParam_GetDesktopWindow: return HandleToUlong( get_desktop_window() ); @@ -4679,6 +4676,9 @@ ULONG_PTR WINAPI NtUserCallOneParam( ULONG_PTR arg, ULONG code ) case NtUserCallOneParam_CreateCursorIcon: return HandleToUlong( alloc_cursoricon_handle( arg )); + case NtUserCallOneParam_CreateMenu: + return HandleToUlong( create_menu( arg ) ); + case NtUserCallOneParam_DispatchMessageA: return dispatch_message( (const MSG *)arg, TRUE ); diff --git a/dlls/win32u/win32u_private.h b/dlls/win32u/win32u_private.h index 20d0e4ac5b1..89b717c695d 100644 --- a/dlls/win32u/win32u_private.h +++ b/dlls/win32u/win32u_private.h @@ -353,7 +353,7 @@ extern BOOL set_capture_window( HWND hwnd, UINT gui_flags, HWND *prev_ret ) DECL extern BOOL set_foreground_window( HWND hwnd, BOOL mouse ) DECLSPEC_HIDDEN; /* menu.c */ -extern HMENU create_menu(void) DECLSPEC_HIDDEN; +extern HMENU create_menu( BOOL is_popup ) DECLSPEC_HIDDEN; extern HMENU get_menu( HWND hwnd ) DECLSPEC_HIDDEN; /* message.c */ diff --git a/include/ntuser.h b/include/ntuser.h index 34b4462f7fe..47b174bc065 100644 --- a/include/ntuser.h +++ b/include/ntuser.h @@ -577,7 +577,6 @@ HWND WINAPI NtUserWindowFromPoint( LONG x, LONG y ); /* NtUserCallNoParam codes, not compatible with Windows */ enum { - NtUserCallNoParam_CreateMenu, NtUserCallNoParam_GetDesktopWindow, NtUserCallNoParam_GetInputState, NtUserCallNoParam_GetMessagePos, @@ -588,11 +587,6 @@ enum NtUserUpdateClipboard, }; -static inline HMENU NtUserCreateMenu(void) -{ - return UlongToHandle( NtUserCallNoParam( NtUserCallNoParam_CreateMenu )); -} - static inline HWND NtUserGetDesktopWindow(void) { return UlongToHandle( NtUserCallNoParam( NtUserCallNoParam_GetDesktopWindow )); @@ -618,6 +612,7 @@ enum { NtUserCallOneParam_BeginDeferWindowPos, NtUserCallOneParam_CreateCursorIcon, + NtUserCallOneParam_CreateMenu, NtUserCallOneParam_DispatchMessageA, NtUserCallOneParam_EnableDC, NtUserCallOneParam_EnableThunkLock, @@ -651,6 +646,11 @@ static inline HICON NtUserCreateCursorIcon( BOOL is_icon ) return UlongToHandle( NtUserCallOneParam( is_icon, NtUserCallOneParam_CreateCursorIcon )); } +static inline HMENU NtUserCreateMenu( BOOL is_popup ) +{ + return UlongToHandle( NtUserCallOneParam( is_popup, NtUserCallOneParam_CreateMenu )); +} + static inline LRESULT NtUserDispatchMessageA( const MSG *msg ) { return NtUserCallOneParam( (UINT_PTR)msg, NtUserCallOneParam_DispatchMessageA );

1 0

Jacek Caban : winex11: Directly use win32u for user functions in event.c.
by Alexandre Julliard April 15, 2022

April 15, 2022

Module: wine Branch: master Commit: 21c37e8f957a8433e5bf78b0ff10b56959cbf22a URL: https://source.winehq.org/git/wine.git/?a=commit;h=21c37e8f957a8433e5bf78b0… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Apr 15 14:52:02 2022 +0200 winex11: Directly use win32u for user functions in event.c. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winex11.drv/event.c | 169 +++++++++++++++++++++++----------------------- dlls/winex11.drv/x11drv.h | 7 ++ 2 files changed, 92 insertions(+), 84 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=21c37e8f957a8433e5bf…

1 0

Gabriel Ivăncescu : jscript: Initialize hres in String constructor.
by Alexandre Julliard April 15, 2022

April 15, 2022

Module: wine Branch: master Commit: e254680ed15f43409114881d8ed613aa4b0de944 URL: https://source.winehq.org/git/wine.git/?a=commit;h=e254680ed15f43409114881d… Author: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Date: Fri Apr 15 16:00:21 2022 +0300 jscript: Initialize hres in String constructor. Signed-off-by: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/jscript/string.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/jscript/string.c b/dlls/jscript/string.c index e73b9e8b52f..dac89c9d8a9 100644 --- a/dlls/jscript/string.c +++ b/dlls/jscript/string.c @@ -1628,7 +1628,7 @@ static HRESULT StringConstr_fromCharCode(script_ctx_t *ctx, jsval_t vthis, WORD static HRESULT StringConstr_value(script_ctx_t *ctx, jsval_t vthis, WORD flags, unsigned argc, jsval_t *argv, jsval_t *r) { - HRESULT hres; + HRESULT hres = S_OK; TRACE("\n");

1 0