Update pull-request-policy.md

Commit migrated from https://github.com/dotnet/corefx/commit/49e2b4955ff83acc4b19b503709bd1e56f409871

Create pull-request-policy.md (dotnet/corefx#40701)

* Create pull-request-validation.md

* Update pull-request-validation.md

* Update pull-request-validation.md

* Update pull-request-validation.md

* Update pull-request-validation.md

* Rename pull-request-validation.md to pull-request-policy.md

* Update pull-request-policy.md

Commit migrated from https://github.com/dotnet/corefx/commit/e2a3e98ee0ae608783906c3a352be27bf7e437b6

Simplify BitArray enumerator (dotnet/corefx#40735)

Commit migrated from https://github.com/dotnet/corefx/commit/130d8582e4973bbdb55ae737b942d214ad3c6570

Add tests for round trip serializing TimeZoneInfos. (dotnet/corefx#40637)

Fix dotnet/corefx#40578

Commit migrated from https://github.com/dotnet/corefx/commit/75a591d4e5606c49f00e187ea2f5ba694d712523

Add harvest validation logic to the all configurations tests (dotnet/corefx#40717)

Commit migrated from https://github.com/dotnet/corefx/commit/2c974d484209ab46815ec9abd35936ca66f2444e

Skip tests on Arm (dotnet/corefx#40716)

Commit migrated from https://github.com/dotnet/corefx/commit/c1974096f6e02099a7e4d030ce0dd903574dd24e

Annotate System.Threading.Channels for nullable reference types (dotnet/corefx#40653)

Commit migrated from https://github.com/dotnet/corefx/commit/60e5e452977d0c51bd8c20e6c390f47333ad85b9

Tests for abbdrivated genitive month names parsing (dotnet/corefx#40595)

Commit migrated from https://github.com/dotnet/corefx/commit/e62f9b61785a340a4a73182c2e228d6ef409dec4

Disable TestZeroTimeout hanging test (dotnet/corefx#40713)

In response to https://github.com/dotnet/corefx/issues/40711

Commit migrated from https://github.com/dotnet/corefx/commit/5fef8a05c2ba51b614dbc2d5350fc3cc0e8541b4

Httpstress: Add more checksum validations & expose Kestrel Http2 limits configuration (dotnet/corefx#40708)

Httpstress: Add more checksum validations & expose Kestrel Http2 limits configuration

Commit migrated from https://github.com/dotnet/corefx/commit/a64e40535283d6470c2909c8f08b178454a8718b

Annotate System.Linq for nullable reference types (dotnet/corefx#40651)

Commit migrated from https://github.com/dotnet/corefx/commit/fe969a5893b845c0ba4550d828894c49745f456f

Fixed missing Dispose call on Union on empty enumerable (dotnet/corefx#40384)

Commit migrated from https://github.com/dotnet/corefx/commit/e78caf82ce13b6e1d1359cc239bf0535943cc386

Annotate Microsoft.Win32.Primitives for nullable reference types (dotnet/corefx#40652)

Commit migrated from https://github.com/dotnet/corefx/commit/70d2b62dcfd328228f8ea6754faae0ed3b69ff37

Test for ArrayPool.Create allocating cleared arrays (dotnet/corefx#40592)

Commit migrated from https://github.com/dotnet/corefx/commit/eeca0c2c0876919a9029cfdc371070bba84da02b

Tests for Read_CharArrayTest reading more than necessary (dotnet/corefx#40521)

* Tests for Read_CharArrayTest reading more than necessary

* Update src/System.IO/tests/BinaryReader/BinaryReaderTests.cs

Co-Authored-By: Stephen Toub <stoub@microsoft.com>
Commit migrated from https://github.com/dotnet/corefx/commit/a192823a84145546be747cc520625062fe6cb5a8

Limit precision checks in arm/64 (dotnet/corefx#40696)

Commit migrated from https://github.com/dotnet/corefx/commit/6e01a780a8a5bc61332c8d743e8516c15bbbb293

Net.Http Use span.SequenceEqual rather than custom EqualsOrdinal (dotnet/corefx#40694)

* Net.Http Use span.SequenceEqual rather than custom EqualsOrdinal

* Fix tests

Commit migrated from https://github.com/dotnet/corefx/commit/d3cd1d05357d3c5f756a89d9800af6ef91dd8efe

Ignore casing in servicename tests on nano server (dotnet/corefx#40699)

Commit migrated from https://github.com/dotnet/corefx/commit/0c650b2765f64186f2fe84e332d8a74108866db2

Update pullrequest-builds.md

Commit migrated from https://github.com/dotnet/corefx/commit/261b2cdc260bc94967a8af6666831739d91ceb96

Disable EventLog failing test on Windows (dotnet/corefx#40688)

Relates to dotnet/corefx#40224

Commit migrated from https://github.com/dotnet/corefx/commit/ae0d0cb852a6186ca5ef8130b0bf862314485017

Disable two flaky tests Mutex/Netw (dotnet/corefx#40689)

Commit migrated from https://github.com/dotnet/corefx/commit/3ef9df83a04113947c42d69cef4b7aadedd612c6

Deserialize extension data properly for child objects (dotnet/corefx#40662)

Commit migrated from https://github.com/dotnet/corefx/commit/26782185622fde9ba354a3decd42f9a0a1a1a266

Enable macOS10.14 in PR builds (dotnet/corefx#40684)

Commit migrated from https://github.com/dotnet/corefx/commit/7e5791b9fb20e29b5fbf8d89ff2688663ae9310a

SmtpClient: fix Timeout for low values (dotnet/corefx#37462)

* SmtpClient: fix Timeout for low values

The timeout is triggered from a Timer.
There were two race conditions:
- The timer event could occur before there was a connection to abort.
- The connection abort performs a TcpClient.Dispose, which may not
work when there is a TcpClient.Connect occurring simultaneously.

* TCPClient: make Dispose thread-safe

* Fix NullReferenceException from _networkstream.Close

* SmtpTransport: fix racy aborted detection

* TestZeroTimeout: skip on .NET Framework

* PR feedback

* Don't null out socket on Dispose to let ODE propagate from Socket

* Fix merge

* Run TestZeroTimeout test on Linux too

* Add back ODE to TcpClient.Connect(IPEndPoint)

* Add comment about why we're disposing the socket

* PR feedback

Commit migrated from https://github.com/dotnet/corefx/commit/807a18da803b09cdca42bfb7d05652b843f5079b

Add custom enum converter test for having flags as array (dotnet/corefx#40678)

Commit migrated from https://github.com/dotnet/corefx/commit/e3644c538ab5dad6c19c6a7c975df0b160823875

changing netcoreapp targetframework from 3.0 to 5.0 and adding support for targetgroup = netcoreapp5.0 (dotnet/corefx#40049)

* adding 5.0 target moniker

* Restoring packages for netcoreapp3.0

* package config for crypto done

* Drawing.Common and System.Windows.Extensions done

* some left over crypto and some smaller other libraries

* security.permissions, WindowsRuntime, WindowsRuntime.Xaml done

* experimental projects and openssl cryptography

* harvesting the runtime package  version for windows Runtime

* update VS configurations

* reverting the shared framework package

* some remaining  projects

* Using sdk to restore package and targetFrameowrk for netcoreapp3.0

* removing harvesting and using ReferenceFromRuntime for windowsRuntime project

* For oob packages with no pkg folder

* addresses feedback

* addresses the feedback

* reverting a commit with extra packageconfig additions

* System.text.Encoding.Web configurations edited

Commit migrated from https://github.com/dotnet/corefx/commit/777dbdec193fcbe49b3f18614348ac96f77d57c3

Improve JsonSerializer support for derived types (dotnet/corefx#40654)

Commit migrated from https://github.com/dotnet/corefx/commit/c59f5bf9badcda76581cd56cbfc5507f29fb022c

Fix additional issue of dictionary properties being skipped (dotnet/corefx#40629)

Commit migrated from https://github.com/dotnet/corefx/commit/546fdc4198bdfd89504c8895c1bd608b4e92d80d

Fix HashSet.SetCapacity() that increases capacity failing after a successful Remove(). (dotnet/corefx#40638)

Fix HashSet.SetCapacity() not accounting for freelist when increasing capacity.

Resolves dotnet/corefx#40621

Commit migrated from https://github.com/dotnet/corefx/commit/2d6fe6b5cc3adba461bb8737c7e5c947992d4c7a

Calculate Harvest packages during build  instead of using checked-in source (dotnet/corefx#40548)

* Calculate Harvest packages during build  instead of using checked-in source

* Add ability to override the calculated package references and versions

* Switch to use PackageDownload instead of PackageReference

Commit migrated from https://github.com/dotnet/corefx/commit/fde678cf14cf1a8e76fe8b6d384d2110f49d253d

Reconfigure terminal for Console usage even if only one of stdin/stdout was used by the child (dotnet/corefx#40563)

* Process: Unix: ensure we reconfigure the terminal for Console usage if only one of stdin/stdout was used by the child.
Console: Unix: Fix cache check for VTIME (read timeout).

Fixes https://github.com/dotnet/corefx/issues/40557

* Also consider not redirecting stderr as terminal usage

Commit migrated from https://github.com/dotnet/corefx/commit/4b35a6fe24aed9ba44ef2534bd32b7a1e0bb7841

Port remaining members of ErrObject, Interaction, SpecialDirectories (dotnet/corefx#40365)

Commit migrated from https://github.com/dotnet/corefx/commit/063568705557b469606eef1069946484d78034ae

Remove Uri.IdnScope and perform cleanup (dotnet/corefx#40546)

Commit migrated from https://github.com/dotnet/corefx/commit/9630ecefc14671d26b2426e9bc76b8178dd3dd75

Remove useless GdiPlus cleanup (dotnet/corefx#40617)

Part of the cleanup was not compiled in and the rest didn't have any
effect.

Commit migrated from https://github.com/dotnet/corefx/commit/619f73197f28cc998265340c817cbbaa12d165c7

Remove stray "#nullable enable" from Pkcs (dotnet/corefx#40622)

This assembly hasn't been reviewed yet for nullability annotations.  We don't want to be mislead by this one file saying it has been.

Commit migrated from https://github.com/dotnet/corefx/commit/c46bd6c33f4c6ad0303d64522bb1f02c6dc5b64c

Add additional skip tests and prevent reset of property index for perf (dotnet/corefx#40598)

Commit migrated from https://github.com/dotnet/corefx/commit/7cc4a04ed3bb9319ec2d80c186a22da3803fda23

Change exception message when reading numbers with leading zeros (dotnet/corefx#40520)

* Change exception message when reading numbers with leading zeroes such as octals.

* Modified exception message and added condition to only throw it when an actual number is being read.

* Addressing suggestions.

* Narrowing scope to ConsumeZero and ConsumeZeroMultiSegment methods only.

* Addressing suggestion to use JsonHelpers.IsInRangeInclusive

Commit migrated from https://github.com/dotnet/corefx/commit/7055b496a30dfe0f66a2f555cad31502473d144b

Fix PLINQ outerloop tests (dotnet/corefx#40589)

* Revert dotnet/corefx#40415's changes to PLINQ tests

They broke outerloop significantly.

* Fix MemberData(BinaryOperations) usage in PLINQ

Lots of outerloop tests are failing because xunit is preferring the wrong overload of the BinaryOperations method via reflection.  This just renames some of the methods to avoid the overloads.

Commit migrated from https://github.com/dotnet/corefx/commit/8429c2a5cc01ac07122f4439f360f8ab2ca4ae86

Re-enable XML theories with English-only variants (dotnet/corefx#39607)

* Re-enable XML theories with English-only variants

* Disable more XML tests on non-English systems

It turns out a bunch of these tests are comparing error strings against hardcoded English text as to what the expected error message will be.

Commit migrated from https://github.com/dotnet/corefx/commit/a6059cdc54865304d1cb76b514096cadddda486f

JsonArray and transformations API implementation (dotnet/corefx#40469)

* JsonArray added
* Transformations API added
* Review Comments included

Commit migrated from https://github.com/dotnet/corefx/commit/ad4fdd9cd3d99d0240fa53b9b7bdc23bb7a3f4a8

Fix issue causing subsequent properties to be skipped during deserialization once a dictionary has been skipped (dotnet/corefx#40490)

Commit migrated from https://github.com/dotnet/corefx/commit/fb30a2d1a2bffbba2c039fd294c471962576a2d7

Allow PathAssemblyResolver to use an assembly with different PublicKeyToken if reference is Retargetable (dotnet/corefx#40581)

Commit migrated from https://github.com/dotnet/corefx/commit/fca60323d9606a90b327273d08089200ae6f527b

System.Numeric.Tests tests now have unique inputs (dotnet/corefx#40564)

Matrix add, subtract and multiplication tests now have unique inputs

Commit migrated from https://github.com/dotnet/corefx/commit/cadc61e7696ceaab416db7f2132db55acfe7c48c

Move ReferenceConverter to the new test base and add ComponentConverter tests (dotnet/corefx#38981)

Commit migrated from https://github.com/dotnet/corefx/commit/8038032294f7a43b5b23bf84d1b0b7cf2d692768

detect libinotify on FreeBSD (dotnet/corefx#40202)

Commit migrated from https://github.com/dotnet/corefx/commit/29e4b76150ace52cb7ccec2ab880fbf23f2dfab3

Remove a few allocations (dotnet/corefx#40580)

Commit migrated from https://github.com/dotnet/corefx/commit/0f7b2968f098f1231993b4e0751026884659b193

Add more span-based Vector<T> ctors and CopyTo tests (dotnet/corefx#36136)

Commit migrated from https://github.com/dotnet/corefx/commit/53edc76f126940875d6ea11ede1f6a00e4a4ce5b

Clean up JsonElementWriteTests (dotnet/corefx#40468)

* Make JsonElementWriteTests and JsonDocumentWriteTests share drivers
* Fix the casing of all escaped values in these tests (reducing the need for normalization)
* Hard-code the correct answer for a test that used NewtonSoft.Json to dynamically compute the same answer every run
* Make two async tests declare async, and await the async part.

As a followup the JsonElementWriteTests / JsonDocumentWriteTests / JsonReadonlyDomWriteTests classes can be split to multiple files, but in this PR it keeps the edits reduced to only the semantic changes (inheritance).

Commit migrated from https://github.com/dotnet/corefx/commit/95d012c8bb53bcbc0ba2b024342c114b9105b26b

Add tests for Encoding virtual methods (dotnet/corefx#40528)

* Add tests for Encoding virtual methods

* Update CustomEncoding

Commit migrated from https://github.com/dotnet/corefx/commit/04b70f289b2840926fe84f7512f96650de8c780b

set _requestQueueBoundHandle to null after disposing (dotnet/corefx#40466)

* set _requestQueueBoundHandle to null after disposing

* remove extra spaces

* feedback from review

Commit migrated from https://github.com/dotnet/corefx/commit/40949c48e1a24a735452248294c983609c77a26c

Contribute to IDE0055 (fix formatting)

Commit migrated from https://github.com/dotnet/corefx/commit/b49a8a9be1d53cd9e50cb68fd8540be25c65d433

Contribute to StyleCop SA1002 (semicolon spacing)

However, I'm not enabling the rule permanently, as it disagrees with VS IDE0055 about the spacing around semicolons at the end of empty while and for loops.

Because this also impacts `for (;;)` loops, I also searched and replaced those with `while (true)`, to standardize on that across the codebase.

Commit migrated from https://github.com/dotnet/corefx/commit/93285e4511a9656a30b92b8b6a9088608aac40e0

Fix StyleCop warning SA1001 (spaces around commas)

Commit migrated from https://github.com/dotnet/corefx/commit/37f351d310f07560274c63862ceaed8afd54a46f

Fix typo in netstandard.depproj (dotnet/corefx#40547)

Commit migrated from https://github.com/dotnet/corefx/commit/7b441c984058fe69acdace0b6386ebd7db18b1c6

Correcting baseline numbers here as 5.0 should have higher numbers then 3.1 (dotnet/corefx#40530)

* correcting baseline numbers here.

* removing runtime, private and meta packages

* updating the visual basic core package number

Commit migrated from https://github.com/dotnet/corefx/commit/66bbf33e750fd9284edb56c52577536f6b91c139

Remove explicit parameter and use arcade default (dotnet/corefx#40542)

for source link validation in the yml.

Commit migrated from https://github.com/dotnet/corefx/commit/76b0203a61cdaea49360604b616eb1139c83efdb

fix syntax for the publishUsingPipelines variable (dotnet/corefx#40545)

Commit migrated from https://github.com/dotnet/corefx/commit/20cce62d9853d9e3bf5d110a778cd1fd32772e62

Add AssemblyAttributesGoHere* and SuppressMergeCheckAttribute to S.R.CS.VC with typeforwards from mscorlib (dotnet/corefx#40470)

* Add AssemblyAttributesGoHere* and SuppressMergeCheckAttribute to System.Runtime.CompilerServices.VisualC as internal types with typeforwards from mscorlib.

* PR Feedback.

Commit migrated from https://github.com/dotnet/corefx/commit/7d94c79892c135d836ecd71e1d969031dba6c2a0

Add additional extension data tests (dotnet/corefx#40488)

Commit migrated from https://github.com/dotnet/corefx/commit/5f0a73134598165820c037c0c7ed2c3fba92a038

Fix UAF when removing trusted certs on OSX

CheckTrustSettings called CFRelease on cert, which downref'd it to zero
before IsCertInKeychain ran.

Now CheckTrustSettings leaves cert alive, and it's the caller's responsibility to CFRelease.

Commit migrated from https://github.com/dotnet/corefx/commit/9be99cb634ffe1b8dc60e4b3d8c01026347a0283

Httpstress: Add checksum validation & support plaintext http (dotnet/corefx#40360)

Httpstress: Add checksum validation & support plaintext http

Commit migrated from https://github.com/dotnet/corefx/commit/a25beeee0af0c849eef58c45579733efa0d55f8d

HTTP2: fix header corruption issue (dotnet/corefx#40487)

* HTTP2: fix header corruption issue

Fixes a bug in HPackEncoder which results in header
data corruption in cases where the write buffer has
not been zeroed out. Adds HPack roundtrip tests.
Fixes dotnet/corefx#40459.

Commit migrated from https://github.com/dotnet/corefx/commit/6f9796080563c9c40359e641e3287fabe42b580c

Move IsReadOnly check inside if-block to avoid cascading if-else (dotnet/corefx#40446)

Commit migrated from https://github.com/dotnet/corefx/commit/988783b4568682bf306875f28ae76e90e7da92a5

Fix unnecessary reliance on ValueTask being backed by a Task (dotnet/corefx#40511)

* Fix unnecessary reliance on ValueTask being backed by a Task

Arbitrary ValueTasks musn't be consumed twice.  Some code in WebSockets we doing so explicitly with the knowledge that the ValueTask it was consuming was actually safe in this pattern, but there's no need for that; with a minor restructuring, that can easily be avoided.

* Address PR feedback

Co-Authored-By: Gert Driesen <gert.driesen@telenet.be>
Commit migrated from https://github.com/dotnet/corefx/commit/cc4ceae30f14b69b625c5275315bbf873be84c01

Fix mail message encoding for transport when delivery method is pickup (dotnet/corefx#40492)

* Fix SmtpClient encoding message for transport when delivery method is pickup

* Reduce tested range of values to more closely bracket where error occurs

* Modify unit test to ensure line folds and validate result

Commit migrated from https://github.com/dotnet/corefx/commit/0de845b6baac154b27ff845774c467018b94c047

Add regression test for https://github.com/dotnet/corefx/issues/39770 (dotnet/corefx#40510)

Commit migrated from https://github.com/dotnet/corefx/commit/0b0c64c4302dfb1bc2a34ca2c1b5fbf9c5f609f6

Re-enable source link validation (dotnet/corefx#40507)

dotnet/corefx@fa168dd690798a683ef0d1e65d60ce5d6918d987 in arcade should have a fix to prevent OOM exceptions.

Commit migrated from https://github.com/dotnet/corefx/commit/4dae0831fccad7b81fc8c4ef187674edb052a1b3

Fix AsyncValueTaskMethodBuilder tests (dotnet/corefx#40505)

A few tests are intended to validate internal implementation details, and those are commented.  But the rest should be asserting implementation specifics.  This fixes those, and augments some validation a bit as well.

Commit migrated from https://github.com/dotnet/corefx/commit/ba9f27788bd99a29e5d832531f0ee86d8eae4959

Console tests: remove tests that expect to observe ANSI sequences (dotnet/corefx#40495)

These tests expect ANSI escape sequences to be written to the Console.SetOut stream,
which doesn't match with the implementation (since https://github.com/dotnet/corefx/commit/dotnet/corefx@98552707c585bc0f0c138fdeed991c28f00a0ac5).

Commit migrated from https://github.com/dotnet/corefx/commit/49a351478132e1c537f0cf3a02b77bbd232238f4

Re-enable xunit warning 2017 (dotnet/corefx#40467)

* Re-enable xunit warning 2017

Use specialized assertions for checking for elements in collections.

* Revert Re-enable xunit warning 2017, revert some test changes

* Address PR feedback

Commit migrated from https://github.com/dotnet/corefx/commit/2fc48e13e87a4e68576d7df7af420ac2874e929e

Ignore extension property if null. (dotnet/corefx#40431)

Commit migrated from https://github.com/dotnet/corefx/commit/41cd99d051102be4ed83f4f9105ae9e73aa48b7c

Fix official builds because of SDL authentication (dotnet/corefx#40483)

* Fix official builds because of SDL authentication

Commit migrated from https://github.com/dotnet/corefx/commit/a1578465f68f6793e7498152dc2d0d2f4eed3a4e

Fix namespace typo in test (dotnet/corefx#40460)

Commit migrated from https://github.com/dotnet/corefx/commit/6257672b2172bf993568cf57ff5d2397bf1dcc7d

Fixed filename case issue (dotnet/corefx#40457)

Commit migrated from https://github.com/dotnet/corefx/commit/c71d095e3e57e0ad2a75d8d97b93cdceb4f7f720

FromBase64Transform.InputBlockSize set to 4 instead of 1

Commit migrated from https://github.com/dotnet/corefx/commit/b456415bca6317b1ad6f98c6475ca59a20c57101

Update Debug.Assert to more clearly show the condition (brace for objects, bracket for array).

Commit migrated from https://github.com/dotnet/corefx/commit/30e06a4f4359f9a368e8fbab013259f46c866df4

Avoid WeakReference race with GC in LdapSessionOptions (dotnet/corefx#40340)

The GC could run between the check of IsAlive and the access to Target.  Better to just access Target and use the result if it's non-null.

Commit migrated from https://github.com/dotnet/corefx/commit/2fac89f63c644ab81d936d62c91e6003302e718d

Remove dead code in CacheMemoryMonitor.SetLimit (dotnet/corefx#40336)

The code was initializing _memoryLimit to 0.  That then makes the second branch of the cascading if/else if/else if dead and the third branch into a pure else.  As a result the whole structure can be brought down to a simple ternary.

Commit migrated from https://github.com/dotnet/corefx/commit/103fc3a785dc8d927357cb3614468fd31ef98864

Use UTC date in Json{Object,String}Tests which use MinValue

Commit migrated from https://github.com/dotnet/corefx/commit/4f7dc2e9d56035d514caa92b9a61d5c03f28c3d8

Fix reading past comments after an end object or end array token within Utf8JsonReader (dotnet/corefx#40436)

* Fix reading past comments after an end object or end array token.

* Add more test cases and update unreachable code paths with
Debug.Asserts.

Commit migrated from https://github.com/dotnet/corefx/commit/247793c64816a887c4ff47a92acdd9b527b7873b

Fix BitArray.CopyTo to byte[] regression - update offset for extra bits. (dotnet/corefx#40441)

Commit migrated from https://github.com/dotnet/corefx/commit/2547b52f31131f10b3cefc530242df35ce9d0e70

Fix failing System.Text.Json test (dotnet/corefx#40439)

Commit migrated from https://github.com/dotnet/corefx/commit/cf5c1b0a7c3cf0087ae72fd0e1995a274cfefe27

System.Text.Json - Don't add to readonly collections - Take 2 (dotnet/corefx#40366)

* Don't add to collection when IsReadOnly is true

* Throw NotSupportedException instead of JsonException

* Add test for readonly collections

* Revert back to InvalidCastException

Commit migrated from https://github.com/dotnet/corefx/commit/f66beeec1ab37a8ea8647caae8bb2ae901f01012

Fix Outerloop runs

Commit migrated from https://github.com/dotnet/corefx/commit/97abb7eb388206f0f976440bbde8f652e9cdd5e1

Use named argument syntax for null values in json serialization methods (dotnet/corefx#40435)

Commit migrated from https://github.com/dotnet/corefx/commit/2b2dd5120029081af767e0eea5e813fd68bded11

Make sure byte position for invalid JSON is consistent and the exception message is accurate (dotnet/corefx#40349)

* Add tests for reading numbers when data is within a multi-segment
ReadOnlySequence.

* Test rename and add debug.assert.

* Test clean up - remove unnecessary/expensive Debug.Assert.

* Test and code cleanup.

* Address feedback - add stronger test condition for token length.

* Revert state on incomplete read and on error. Make sure exception
message is accurate.

* Fix byte position in line for invalid JSON and update tests.

* Address feedback - pass by in and move private struct to end.

* Fix string consumption and add various tests.

* Fix off-by-one error when calculating position.

* Test clean up - make sure multi-segment tests are in the right file.

* Update string parsing - do not increment line number for escaped newline
chars within quoted strings.

* Remove unnecessary custom state struct for string and re-use the same
one for both string and numbers.

* Update tests based on PR feedback.

* Create helper to get start sequence position from roll back state.

Commit migrated from https://github.com/dotnet/corefx/commit/a690f1658cdc4d92936bd709e644074b2e2e288e

Re-enable xunit warning 1019 (dotnet/corefx#40415)

Make MemberData reference IEnumerable<object[]>

Commit migrated from https://github.com/dotnet/corefx/commit/3c09762fa14249c65e5bc676a5d812c2654c8941

Honor converters for implementing types (dotnet/corefx#40411)

* Honor converters for implementing types

* Cache implementing-type converter if one is detected

* Address feedback: name parameters and add IEnumerable wrapper test

Commit migrated from https://github.com/dotnet/corefx/commit/ee880076f3bcacde1a32e88ea4ebd701a609e52d

Use new instances of JsonSerializerOptions in property visibility tests to prevent re-using cached metadata (dotnet/corefx#40414)

Commit migrated from https://github.com/dotnet/corefx/commit/21ce6673922a93d34117b6c757f394600b7f7dbb

Add DecoratedNameAttribute (dotnet/corefx#40125)

Commit migrated from https://github.com/dotnet/corefx/commit/86db0ef4acc409ab29cd07908aec21c2dc237ea0

Honor JsonIgnore attribute when applied to unsupported collections (dotnet/corefx#40401)

* Honor JsonIgnore attribute when applied to unsupported collections

* Address feedback: add more unsupported collections

Commit migrated from https://github.com/dotnet/corefx/commit/6739eab2f4ee9a895575289893edf63b6c234d43

Avoid using double/floats within tests that don't roundtrip on netfx (dotnet/corefx#40408)

(for G17).

Commit migrated from https://github.com/dotnet/corefx/commit/b2ea249aa67d719b35c6a1c95befbf2b819bab17

Corefx dotnet/corefx#40133 Use TryFormatDouble TryFormatFloat (dotnet/corefx#40197)

* Corefx dotnet/corefx#40133 Use TryFormatDouble TryFormatFloat

* add test cases

* Changes as per review comments

* remove extra empty line

Commit migrated from https://github.com/dotnet/corefx/commit/3d72b56b8e7d1e4e44262be6c45c2aac580bbbc4

Change 'type' to 'inputType' on serialize methods (dotnet/corefx#40372)

Commit migrated from https://github.com/dotnet/corefx/commit/9217d338671fe78d2bd31138a610bb9ad5a12608

Use Count in Enumerable.Any if available (dotnet/corefx#40377)

* Use Count in Enumerable.Any if available

We've been hesitant to make this change in the past, as it adds several interface checks.  However, wide-spread "wisdom" is that `Any()` is as fast or faster than `Count() > 0`, and there are even FxCop rules/analyzers that warn about using the latter instead of the former, but in its current form that can frequently be incorrect: if the source does implement `ICollection<T>`, generally its `Count` is O(1) and allocation-free, whereas `Any()` will almost always end up allocating an enumerator.  On balance, it seems better to just have `Any()` map closely to `Count()` so that their performance can be reasoned about in parallel.

* Add test coverage for Enumerable.Any

Commit migrated from https://github.com/dotnet/corefx/commit/9021bc1cbc8c1189d2eb8de906e8891878430e6f

Disable AssociatePersistedKey_CAPIviaCNG_RSA on arm64 (dotnet/corefx#40375)

Commit migrated from https://github.com/dotnet/corefx/commit/9ea74a6d3ec09fef63975338105abdca6fac8e1c

Remove unnecessary field in PrinterSettings (dotnet/corefx#40339)

I'm not sure why it has its own ArrayEnumerator, but as long as it does, it doesn't need _startIndex.

Commit migrated from https://github.com/dotnet/corefx/commit/bfe2c58a4536db9a257940277c5d94bf9e26929a

Disabling test failing in Mac OSX 10.15 (dotnet/corefx#40332)

* Disabling test failing in Mac OSX 10.15

* Using active issue annotation

* Checking OS version instead of disabling test for any Mac

* Formatting update

Commit migrated from https://github.com/dotnet/corefx/commit/df3837cbced3fdcaab22464c18f0ce20b4aadfc7

Fix outerloop cron job (dotnet/corefx#40317)

Commit migrated from https://github.com/dotnet/corefx/commit/001f42b944e3a05a42f9f8cd7d14d338a06e6328

Remove unnecessary null check from ConstraintCollection (dotnet/corefx#40343)

`_table` will never be null.  The `ConstraintCollection` is only constructed in one place, where it's passed `this` by `DataTable`.  It's used in many other places without a guard check, as well (and later in this method, unguarded).

Commit migrated from https://github.com/dotnet/corefx/commit/f4344b0d3b45fc7c8f41e5c909ddcb2752bea0fa