Skip to content

Conversation

cxwtool
Copy link

@cxwtool cxwtool commented Mar 24, 2020

Please don't squash-merge this PR.

Gnbrkm41 and others added 11 commits March 23, 2020 10:57
…4028)

* Add remarks regarding the initial content of span in string.Create

* Apply suggestions

Co-Authored-By: stephentoub <stoub@microsoft.com>

* Apply suggestions

Co-Authored-By: Jeremy Barton <jbarton@microsoft.com>

Co-authored-by: stephentoub <stoub@microsoft.com>
Co-authored-by: Jeremy Barton <jbarton@microsoft.com>
The remarks section uses the following text for both the `UploadString()` and `UploadStringAsync()` methods:

> Sends a String to the resource, without blocking the calling thread.

I think this is a copy-and-paste error because only the `UploadStringAsync()` method is non-blocking. I have updated the description for `UploadString()` to follow the pattern used when describing other upload methods.
* RSACng.Encrypt used wrong "Pkcs1" link.

* Checked the source code. Corrrection.

* Resolve comment.

* Keep exception description in Decrypt consistent with Encrypt.
The current implementation of `Flush` **does** flush the internal buffer.
Calling the Write method will not block current thread.
* Document behavior of StartProfile with null argument

Contributes to dotnet/runtime#33426

* Update xml/System.Runtime/ProfileOptimization.xml

Co-Authored-By: Stephen Toub <stoub@microsoft.com>

Co-authored-by: Stephen Toub <stoub@microsoft.com>
@dotnet-bot dotnet-bot added this to the March 2020 milestone Mar 24, 2020
@BillWagner BillWagner merged commit e035659 into live Mar 24, 2020
@BillWagner BillWagner deleted the publish-15166 branch March 24, 2020 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.