Re: [include/winternl.h #6] SYSTEM_PROCESS_INFORMATION

[Robert Shearman]

Paul Vriens wrote:
> Hi,

Hi Paul,
> the first bigger change to SYSTEM_PROCESS_INFORMATION. The structure now
> contains VM_COUNTERS (which had to move up in the file) instead of all
> the single (wrong) items, and IO_COUNTERS. I also added 2 extra FILETIME
> items (well actually I changed dwUnknown2[5]).
>
> This structure now has the correct size for W2K/WinXP and W2K3. NT
> doesn't have the IO_COUNTERS part, but for now I'm not to worried about
> dealing with that (unless somebody tells me to).
>
> I also had to change ntdll/nt.c because of this. I removed all the
> single sets to 0 as this is done by the memset anyway.
>
> Next patches will change the struct (and related files) to use
> UNICODE_STRING for the process name, but I'm having a few issues with
> that.
>
> Next patches will also contain tests to prove the above. Can anybody
> (who's not tired of reading until now) tell me if we really need the
> __WINESRC__ in that structure, the Windows structure shown is wrong
> anyway

We shouldn't need the __WINESRC__ ifdef. No code that is designed to be 
portable to Microsoft's headers should be using winternl.h.
> and how can I check 'my' struct in a test (for now I've copied it
> to the test and renamed it to something non-existent)?

I think that is the only way to do it.

-- 
Rob Shearman