Re: include: add initial winsta.h
On Tue, May 26, 2015 at 10:21 PM, Austin English <austinenglish(a)gmail.com> wrote:
-- -Austin
Patch has been pending for several days, is there something I can do to improve it? Nikolay asked in #winehackers where this came from, it's on MSDN: https://msdn.microsoft.com/en-us/library/cc248871.aspx and also in Process Hacker's source: http://processhacker.sourceforge.net/doc/winsta_8h_source.html -- -Austin
Austin English <austinenglish(a)gmail.com> writes:
On Tue, May 26, 2015 at 10:21 PM, Austin English <austinenglish(a)gmail.com> wrote:
-- -Austin
Patch has been pending for several days, is there something I can do to improve it?
Nikolay asked in #winehackers where this came from, it's on MSDN: https://msdn.microsoft.com/en-us/library/cc248871.aspx
and also in Process Hacker's source: http://processhacker.sourceforge.net/doc/winsta_8h_source.html
It doesn't look like it's part of the SDK. What do you need this for? -- Alexandre Julliard julliard(a)winehq.org
On Tue, Jun 2, 2015 at 11:51 PM, Alexandre Julliard <julliard(a)winehq.org> wrote:
Austin English <austinenglish(a)gmail.com> writes:
On Tue, May 26, 2015 at 10:21 PM, Austin English <austinenglish(a)gmail.com> wrote:
-- -Austin
Patch has been pending for several days, is there something I can do to improve it?
Nikolay asked in #winehackers where this came from, it's on MSDN: https://msdn.microsoft.com/en-us/library/cc248871.aspx
and also in Process Hacker's source: http://processhacker.sourceforge.net/doc/winsta_8h_source.html
It doesn't look like it's part of the SDK. What do you need this for?
-- Alexandre Julliard julliard(a)winehq.org
https://bugs.winehq.org/show_bug.cgi?id=38102 Should I instead put winsta.h in dlls/winsta/ ? -- -Austin
Austin English <austinenglish(a)gmail.com> writes:
On Tue, Jun 2, 2015 at 11:51 PM, Alexandre Julliard <julliard(a)winehq.org> wrote:
Austin English <austinenglish(a)gmail.com> writes:
Patch has been pending for several days, is there something I can do to improve it?
Nikolay asked in #winehackers where this came from, it's on MSDN: https://msdn.microsoft.com/en-us/library/cc248871.aspx
and also in Process Hacker's source: http://processhacker.sourceforge.net/doc/winsta_8h_source.html
It doesn't look like it's part of the SDK. What do you need this for?
https://bugs.winehq.org/show_bug.cgi?id=38102
Should I instead put winsta.h in dlls/winsta/ ?
Probably. You also have to fix handling of anonymous unions, and your TS_ALL_PROCESSES_INFO definition doesn't look right. -- Alexandre Julliard julliard(a)winehq.org
On Wed, Jun 3, 2015 at 12:01 AM, Alexandre Julliard <julliard(a)winehq.org> wrote:
Austin English <austinenglish(a)gmail.com> writes:
On Tue, Jun 2, 2015 at 11:51 PM, Alexandre Julliard <julliard(a)winehq.org> wrote:
Austin English <austinenglish(a)gmail.com> writes:
Patch has been pending for several days, is there something I can do to improve it?
Nikolay asked in #winehackers where this came from, it's on MSDN: https://msdn.microsoft.com/en-us/library/cc248871.aspx
and also in Process Hacker's source: http://processhacker.sourceforge.net/doc/winsta_8h_source.html
It doesn't look like it's part of the SDK. What do you need this for?
https://bugs.winehq.org/show_bug.cgi?id=38102
Should I instead put winsta.h in dlls/winsta/ ?
Probably. You also have to fix handling of anonymous unions, and your TS_ALL_PROCESSES_INFO definition doesn't look right.
-- Alexandre Julliard julliard(a)winehq.org
Okay, thanks for reviewing. -- -Austin
participants (2)
-
Alexandre Julliard -
Austin English