Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

NamedPipeClientStream Connect with SafePipeHandle #110075

Closed
wants to merge 3 commits into from
Closed

NamedPipeClientStream Connect with SafePipeHandle #110075

wants to merge 3 commits into from

Commits on Nov 22, 2024

  1. Added an argument check for isConnected. 

    * Check that isConnected in the NamedPipeClientStream constructor is always true.
* Added an argument test.

Fixes dotnet#32760
    @koenigst
    koenigst committed Nov 22, 2024
    Configuration menu
    Copy the full SHA
    bfecb79 View commit details
    Browse the repository at this point in the history

  2. Removed the isConnected argument from the constructor. 

    * Created a new constructor without the argument.
* Marked the old constructor as obsolete.

Fixes dotnet#32760
    @koenigst
    koenigst committed Nov 22, 2024
    Configuration menu
    Copy the full SHA
    9c02e5f View commit details
    Browse the repository at this point in the history

  3. Corrected the nullability in the interop declarations.

    @koenigst
    koenigst committed Nov 22, 2024
    Configuration menu
    Copy the full SHA
    7ef2f1c View commit details
    Browse the repository at this point in the history