Please come to a decision. As for my own opinion, there is no need in #defines, because those functions are most probably inlined by the compiler anyway (they are forwarders).


On Jan 9, 2009, at 6:26 PM, Alex Ionescu wrote:

The Windows binaries don't have this behavior, why should ReactOS?

The NtUser/Gdi layer should be internal, not called from these sorts of functions -- it violates principles of private vs public methods... I know the code isn't OOP but Nt still has strong OOP design principles at its core.

The code in the libraries should, as much as possible, be Win32 code, not Native API -- ESPECIALLY in the case of NtUser/Gdi.

It also makes the binaries MUCH harder to test on Windows, as the NtUser/Gdi layer in ReactOS Isn't yet compatible with Windows...so functions would take different parameters, maybe even have different names.

Best regards,
Alex Ionescu


2009/1/9 Aleksey Bragin <aleksey@reactos.org>
It was not confirmed with me or anyone from Win32k team, however after talking with Dmitry it looks like there is nothing bad in it. All those substituted functions are direct forwarders to win32k both in ReactOS and in Windows.

If James or Timo object, feel free to revert.

WBR,
Aleksey.


On Jan 9, 2009, at 5:49 PM, Ged wrote:

Lol, I've been through this with Aleksey about 30 minutes ago  ;)

 

Ged.