Re: [PATCH v3 0/3] MR3855: Fixes for ntlea

[Alex Henrie (＠alexhenrie)]

Alex Henrie (@alexhenrie) commented about dlls/user32/win.c:
> -
> - and will try to skip over this instruction when calling `GetWindowLongA`,
> -
> - (i.e. it tries to jump to `GetWindowLongA + 7`, 5 bytes for the prologue, 2
> -
> - bytes for the `pushl`.). We have to anticipate that and make sure the result
> -
> - of doing this won't be a messed up stack, or a desynced PC.
> - */
>
> +__ASM_STDCALL_FUNC(GetWindowLongA, 8,
> -
>    "movl.s %edi, %edi\n"
> -
>    "pushl %ebp\n"
> -
>    "movl.s %esp, %ebp\n"
> -
>    "pushl $-2\n"
> -
>    "addl $4, %esp\n"
> -
>    "popl %ebp\n"
> -
>    "jmp " __ASM_STDCALL("get_window_longA", 8) "\n")
>
> +LONG WINAPI get_window_longA( HWND hwnd, INT offset )
> +#else
> +LONG WINAPI DECLSPEC_HOTPATCH GetWindowLongA(HWND hwnd, INT offset)

Could you please use a consistent style here, either with extra spaces in the parentheses in both argument lists or without extra spaces in both?