Skip to content

Commit

Permalink
Merge pull request #125 from microsoft/master
Browse files Browse the repository at this point in the history
Cut another release to fix the bug in 2.0.153
  • Loading branch information
mjcheetham authored May 22, 2020
2 parents b54ac74 + 9d8d5c8 commit b11949c
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
1 change: 1 addition & 0 deletions .github/workflows/release-homebrew.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ jobs:
uses: actions/[email protected]
id: version
with:
github-token: ${{secrets.GITHUB_TOKEN}}
result-encoding: string
script: |
const { data } = await github.repos.getReleaseByTag({
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ public unsafe bool IsDesktopSession
// In .NET 5 the implementation on Windows has been 'fixed', but still POSIX versions always return true.
//
// This code is lifted from the .NET 5 targeting dotnet/runtime implementation for Windows:
// https://github.com/dotnet/runtime/blob/8c10a98c13263eea806d4fcecb76d29e38593539/src/libraries/System.Private.CoreLib/src/System/Environment.Windows.cs#L125-L145
// https://github.com/dotnet/runtime/blob/cf654f08fb0078a96a4e414a0d2eab5e6c069387/src/libraries/System.Private.CoreLib/src/System/Environment.Windows.cs#L125-L145

// Per documentation of GetProcessWindowStation, this handle should not be closed
IntPtr handle = User32.GetProcessWindowStation();
Expand All @@ -31,7 +31,7 @@ public unsafe bool IsDesktopSession
if (User32.GetUserObjectInformation(handle, User32.UOI_FLAGS, &flags,
(uint) sizeof(USEROBJECTFLAGS), ref dummy))
{
return (flags.dwFlags & User32.WSF_VISIBLE) == 0;
return (flags.dwFlags & User32.WSF_VISIBLE) != 0;
}
}

Expand Down

0 comments on commit b11949c

Please sign in to comment.