Movatterモバイル変換

Skip to content

dotnet/runtimePublic

NotificationsYou must be signed in to change notification settings
Fork5.2k
Star17.2k

Add Windows support to PosixSignal#55333

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

stephentoub merged 2 commits intodotnet:mainfromstephentoub:posixsignalwindows

Jul 9, 2021

Merged

Add Windows support to PosixSignal#55333

stephentoub merged 2 commits intodotnet:mainfromstephentoub:posixsignalwindows

Jul 9, 2021

Conversation

@stephentoub

Copy link

Member

stephentoub commentedJul 8, 2021

Fixes#50527
Fixes#55328

cc:@jkotas,@tmds,@davidfowl, @dotnet/area-system-console

@stephentoub

Add Windows support to PosixSignal

27c7433

@ghost

ghost added area-System.Runtime new-api-needs-documentation labels

@ghost

Copy link

ghost commentedJul 8, 2021

Note regarding thenew-api-needs-documentation label:

This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, to please make sure the API implementation in the src *.cs file is documented with triple slash comments, so the PR reviewers can sign off that change.

@ghost

Copy link

ghost commentedJul 8, 2021

Tagging subscribers to this area: @dotnet/area-system-runtime
See info inarea-owners.md if you want to be subscribed.

Issue Details

Fixes#50527
Fixes#55328

cc:@jkotas,@tmds,@davidfowl, @dotnet/area-system-console

Author:	stephentoub
Assignees:	-
Labels:	`area-System.Runtime`,`new-api-needs-documentation`
Milestone:	-

tmds

tmds reviewed

View reviewed changes

src/libraries/System.Console/src/System/Console.csShow resolvedHide resolved

tmds

tmds reviewed

View reviewed changes

...me.InteropServices/tests/System/Runtime/InteropServices/PosixSignalRegistrationTests.Unix.cs OutdatedShow resolvedHide resolved

tmds

tmds reviewed

View reviewed changes

...untime.InteropServices/src/System/Runtime/InteropServices/PosixSignalRegistration.Windows.csShow resolvedHide resolved

jkotas

jkotas reviewed

View reviewed changes

src/libraries/System.Console/src/System/ConsolePal.Windows.csShow resolvedHide resolved

src/libraries/System.Runtime.InteropServices/src/System.Runtime.InteropServices.csprojShow resolvedHide resolved

...untime.InteropServices/src/System/Runtime/InteropServices/PosixSignalRegistration.Windows.csShow resolvedHide resolved

...untime.InteropServices/src/System/Runtime/InteropServices/PosixSignalRegistration.Windows.cs OutdatedShow resolvedHide resolved

...me.InteropServices/tests/System/Runtime/InteropServices/PosixSignalRegistrationTests.Unix.csShow resolvedHide resolved

davidfowl

davidfowl reviewed

View reviewed changes

src/libraries/System.Runtime.InteropServices/ref/System.Runtime.InteropServices.csShow resolvedHide resolved

@stephentoub

Address PR feedback

92003ed

jkotas

jkotas approved these changes

View reviewed changes

tmds

tmds reviewed

View reviewed changes

...untime.InteropServices/src/System/Runtime/InteropServices/PosixSignalRegistration.Windows.csShow resolvedHide resolved

@stephentoub

stephentoub merged commitf1bf5ee intodotnet:main

@stephentoub

stephentoub deleted the posixsignalwindows branch

July 9, 2021 10:46

@stephentoub

stephentoub mentioned this pull request

Move PosixSignalRegistration to corelib#55433

Merged

@ManickaP

ManickaP mentioned this pull request

[QUIC] Remove AppContext switch from S.N.Quic#56027

Merged

@ghost

ghost locked asresolvedand limited conversation to collaborators

Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Labels

area-System.Runtime new-api-needs-documentation

4 participants

@stephentoub

@davidfowl

@tmds

@jkotas

[8]ページ先頭

©2009-2025 Movatter.jp