Updating condition(s) under which the method throws #3152
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@stevewhims, |
|
cc: @Caesar1995 The System.InvalidOperationException can be thrown from BeginConnect for 2 possible reasons:
and
|
davidsh
suggested changes
Sep 16, 2017
davidsh
approved these changes
Sep 18, 2017
caesar-chen
approved these changes
Sep 18, 2017
mairaw
added
merge-on-hold-for-live-update
and removed
merge-on-hold-for-live-update
labels
OPSTestPPE
added a commit
to OPS-E2E-PPE/docs
that referenced
this pull request
Sep 22, 2017
* fixed broken link (dotnet#3169) * fixed pipes in tables (dotnet#3163) * added info about relative paths for output (dotnet#3042) * added info about relative paths for output * edits * Update how-to-write-a-simple-parallel-for-loop.md (dotnet#3174) Updated Cut to Copy in Copy/Cut and Paste this code into ... * fixed formulas (dotnet#3182) * fixed formulas * fixed formula * fixed formula * simplifying version list (dotnet#3176) * Minor fixes : inconsistent wording and some ghost characters (dotnet#3171) * what's new Tuples : sample use InvariantCulture * vb tuple example fix new language feature sample * minor typos * compiler warning corrected to match previously mentioned number * removed a "ghost" quote * fixed inconsistent "read-only" and "read only" * code formatting (missing space) * Adding C# version history post (dotnet#2867) * Create csharp-version-history.md * Added the actual text * Updated TOC with my entry. * first pass edit 1. Add Metadata 1. Change person 1. Formalize from blog post to docs style language. * Update csharp-version-history.md * fix build warnings on links. * respond to feedback * fixed build warnings * fixed version capitalization * Learn C# Tutorial (#2306) * First draft of new tutorial Learn the structure of a C# program, and explore the string class. proofread and tighten language. respond to feedback 2nd draft Move more code earlier. Tighten vocabulary discussions. Update outline start on the next draft. add section on searching strings fix build errors finish the string tutorial. reorder: ready for proofread. proofread update to use new syntax Update Tutorial to use the new engine typo in the title * build test, try samples * 2nd draft of the numbers tutorial. * proofread the second tutorial and create the third Branching and looping is ready for initial review. * respond to feedback on first tutorial. * proof read 3rd tutorial And add links to other C# Guide content * Remove links to upcoming tutorials. * Respond to feedback. * fix yaml parsing error. * More feedback. * feedback from team review Configure ms.technology for all quick starts to be devlang-csharp-interactive. Update .NET home page and C# Guide home page to point to the interactive tutorials. * Describe booleans, and update metadata. * add a tip about compiler messages * update timings of tutorials, update tip text * final review. * Fix typo: AesCrytpServiceProvider -> AesCryptoServiceProvider (dotnet#3185) * Fix typo: AesCrytpServiceProvider -> AesCryptoServiceProvider * Fix typo: crytopgraphy -> cryptography * added net core behavior (dotnet#3140) * added net core behavior * feedback * feedback * feedback * quick link fix to the hub page (dotnet#3193) The link target was updated for the Quick Starts, but the displayed text was not. * Removed duplicate entry from 'See Also' section (dotnet#3195) * removed invisible characters (dotnet#3192) * update snippet reference (dotnet#3191) * fixed broken link (dotnet#3190) * clarified example (dotnet#3184) * Updating condition(s) under which the method throws (dotnet#3152) * Updating condition(s) under which the method throws * Correcting a few hard-to-localize bits of content * Update mutually recursive functions snippet (dotnet#3186) * Update mutually recursive functions snippet Correct implementation should be something like (this one)[https://en.wikipedia.org/wiki/Mutual_recursion] * Typo fix in function name Event -> Even * 494407 Corrections to DnsSafeHost and IdnHost (dotnet#3143) * 494407 Corrections to DnsSafeHost and IdnHost * Actioning all comments :) * I used the wrong inline code delimiter * Improve Suits and Ranks definitions (dotnet#3197) * update target framework to 2.0 * Improve Suits and Ranks enumerable definitions * Revert "update target framework to 2.0" This reverts commit 008d19d. * Fixed Kelvin output, make culture independent (dotnet#3157) * Fixed Kelvin output, make culture independent - Fixed wrong expected output for Kelvin in the sample usage - Defined current culture in the sample usage to work out-of-the-box for international locales * Remove obsolete full qualified name * CurrentThread.CurrentCulture -> CultureInfo.CurrentCulture * Fixed Kelvin output, make culture independent (dotnet#3159) * Fixed Kelvin output, make culture independent - Fixed wrong expected output for Kelvin in the sample usage - Defined current culture in the sample usage to work out-of-the-box for international locales * Remove obsolete full qualified name * CurrentThread.CurrentCulture -> CultureInfo.CurrentCulture * Remove Duplicate Config Definition (dotnet#3200) For whatever reason, the sample XML was duplicated -- this commit removes that duplication so that the example is more readable. * fixed api link format (dotnet#3194) * Typo in document - IOne instead of One (dotnet#3202) Typo in document - IOne instead of One * Fix framework in dotnet store command topic (dotnet#3188) Reported by @pauliusnorkus. Thanks to @dasMulli for reminding me about the framework problem here. Using `netstandard2.0` worked during pre-release but no more.
OPSTestPPE
added a commit
to OPS-E2E-PPE/docs
that referenced
this pull request
Sep 22, 2017
* fixed broken link (dotnet#3169) * fixed pipes in tables (dotnet#3163) * added info about relative paths for output (dotnet#3042) * added info about relative paths for output * edits * Update how-to-write-a-simple-parallel-for-loop.md (dotnet#3174) Updated Cut to Copy in Copy/Cut and Paste this code into ... * fixed formulas (dotnet#3182) * fixed formulas * fixed formula * fixed formula * simplifying version list (dotnet#3176) * Minor fixes : inconsistent wording and some ghost characters (dotnet#3171) * what's new Tuples : sample use InvariantCulture * vb tuple example fix new language feature sample * minor typos * compiler warning corrected to match previously mentioned number * removed a "ghost" quote * fixed inconsistent "read-only" and "read only" * code formatting (missing space) * Adding C# version history post (dotnet#2867) * Create csharp-version-history.md * Added the actual text * Updated TOC with my entry. * first pass edit 1. Add Metadata 1. Change person 1. Formalize from blog post to docs style language. * Update csharp-version-history.md * fix build warnings on links. * respond to feedback * fixed build warnings * fixed version capitalization * Learn C# Tutorial (#2306) * First draft of new tutorial Learn the structure of a C# program, and explore the string class. proofread and tighten language. respond to feedback 2nd draft Move more code earlier. Tighten vocabulary discussions. Update outline start on the next draft. add section on searching strings fix build errors finish the string tutorial. reorder: ready for proofread. proofread update to use new syntax Update Tutorial to use the new engine typo in the title * build test, try samples * 2nd draft of the numbers tutorial. * proofread the second tutorial and create the third Branching and looping is ready for initial review. * respond to feedback on first tutorial. * proof read 3rd tutorial And add links to other C# Guide content * Remove links to upcoming tutorials. * Respond to feedback. * fix yaml parsing error. * More feedback. * feedback from team review Configure ms.technology for all quick starts to be devlang-csharp-interactive. Update .NET home page and C# Guide home page to point to the interactive tutorials. * Describe booleans, and update metadata. * add a tip about compiler messages * update timings of tutorials, update tip text * final review. * Fix typo: AesCrytpServiceProvider -> AesCryptoServiceProvider (dotnet#3185) * Fix typo: AesCrytpServiceProvider -> AesCryptoServiceProvider * Fix typo: crytopgraphy -> cryptography * added net core behavior (dotnet#3140) * added net core behavior * feedback * feedback * feedback * quick link fix to the hub page (dotnet#3193) The link target was updated for the Quick Starts, but the displayed text was not. * Removed duplicate entry from 'See Also' section (dotnet#3195) * removed invisible characters (dotnet#3192) * update snippet reference (dotnet#3191) * fixed broken link (dotnet#3190) * clarified example (dotnet#3184) * Updating condition(s) under which the method throws (dotnet#3152) * Updating condition(s) under which the method throws * Correcting a few hard-to-localize bits of content * Update mutually recursive functions snippet (dotnet#3186) * Update mutually recursive functions snippet Correct implementation should be something like (this one)[https://en.wikipedia.org/wiki/Mutual_recursion] * Typo fix in function name Event -> Even * 494407 Corrections to DnsSafeHost and IdnHost (dotnet#3143) * 494407 Corrections to DnsSafeHost and IdnHost * Actioning all comments :) * I used the wrong inline code delimiter * Improve Suits and Ranks definitions (dotnet#3197) * update target framework to 2.0 * Improve Suits and Ranks enumerable definitions * Revert "update target framework to 2.0" This reverts commit 008d19d. * Fixed Kelvin output, make culture independent (dotnet#3157) * Fixed Kelvin output, make culture independent - Fixed wrong expected output for Kelvin in the sample usage - Defined current culture in the sample usage to work out-of-the-box for international locales * Remove obsolete full qualified name * CurrentThread.CurrentCulture -> CultureInfo.CurrentCulture * Fixed Kelvin output, make culture independent (dotnet#3159) * Fixed Kelvin output, make culture independent - Fixed wrong expected output for Kelvin in the sample usage - Defined current culture in the sample usage to work out-of-the-box for international locales * Remove obsolete full qualified name * CurrentThread.CurrentCulture -> CultureInfo.CurrentCulture * Remove Duplicate Config Definition (dotnet#3200) For whatever reason, the sample XML was duplicated -- this commit removes that duplication so that the example is more readable. * fixed api link format (dotnet#3194) * Typo in document - IOne instead of One (dotnet#3202) Typo in document - IOne instead of One * Fix framework in dotnet store command topic (dotnet#3188) Reported by @pauliusnorkus. Thanks to @dasMulli for reminding me about the framework problem here. Using `netstandard2.0` worked during pre-release but no more.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Updating the condition(s) under which Socket.BeginConnect throws
Summary
There are four overloads; each behaves the same way with respect to this particular exception.