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

Fix IDE0066 across corefx (dotnet/corefx#40288)

* Fix IDE0066 across corefx

"Fix all", but then manual tweaking to accommodate places the rule doesn't work quite right.

* Address PR feedback

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

Improve code coverage for VB Conversion module (dotnet/corefx#39937)

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

Fix BytesConsumed when reading JSON numbers within a multi-segment ReadOnlySequence via Utf8JsonReader (dotnet/corefx#40303)

* 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.

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

Minor cleanup tweaks in Regex (dotnet/corefx#40338)

* Remove unused local in RegexRunner.TextposDescription

* Remove redundant checks in RegexCharClass.AddRange

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

Remove duplicate _wrapped.WriteBase64 call in QueryOutputWriterV1 (dotnet/corefx#40345)

The same expression is used for both the if and the else branches.  But the if branch does make a mutating method call, which we keep.

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

adding ci to release 3.0 branch (dotnet/corefx#40359)

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

Avoid creating/populating unneeded certificate collection in SecureChannel.AcquireClientCredentials (dotnet/corefx#40354)

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

Tests for JsonSerializerOptions.GetConverter() and converter methods (dotnet/corefx#39750)

* Tests for JsonSerializerOptions.GetConverter

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

Support custom converters that treat non-null input as null (dotnet/corefx#40287)

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

Prevent threading issues with warm-up of property caching (dotnet/corefx#40261)

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

Renaming Microsoft.Framework.WebEncoders namespace (dotnet/corefx#40238)

* Renaming the namespace Microsoft.Framework.WebEncoders to System.Text.Encodings.Web.Tests

- Namespace Microsoft.Framework.WebEncoders renamed to System.Text.Encodings.Web.Tests

* Changed aliases naming from camelCase to PascalCase

* Deleted unnecessary using directives and reverted EncoderExtensionsTests

Deleted unnecessary using directives. Reverted changes made in EncoderCommonTests.cs to commit dotnet/corefx@db1dd29edac07716ac27b8916d5e12f9a85b4978 and changed using directive for test namespace to right one.

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

fixing ZipPackagePart.GetStreamCore crashes with NotSupportedException (dotnet/corefx#40319)

* fixing ZipPackagePart.GetStreamCore crashes with NotSupportedException

ZipArchiveEntry only ever supports opening once when the backing archive is in Create mode,  and the backing stream is non-seekable, so we shouldn't call SetLength in that case. You could still open an archive in Update mode then call part.GetStream(FileMode.Create), in which case we'll want this call to SetLength, so we only avoid this call when the backing Archive is in Create mode.

* updating test to explicitly test the Update path for ZipPackage

* skip UAP since we don't have access to the file system to create the .zip

* undo accidental change to existing test

* removing unnecessary variable

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

Use sealed securityContext in SecureChannel (dotnet/corefx#40265)

* Use sealed securityContext in SecureChannel

* Less asm

* Feedback

* Remove ternary

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

Remove duplicate code in WMIGenerator.ConvertToNumericValueAndAddToArray (dotnet/corefx#40346)

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

Apply feedback to pal_x509chain.c fix from release/3.0

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

Fix a few missing $s for string interpolation in tracing (dotnet/corefx#40322)

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

Fix non-ASCII characters across corefx (dotnet/corefx#40318)

* Fix several non-visible characters in test files

* Replace Unicode surrogate pairs with escaped hex values

* Fix non-ASCII characters in most C# files

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

Delete dead code from XMLDiffLoader.ReadOldRowData (dotnet/corefx#40344)

We already exited out of the method earlier if table == null.

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

Fix PKCS1MaskGenerationMethod.GenerateMask handling of invalid hash name (dotnet/corefx#40323)

* Fix PKCS1MaskGenerationMethod.GenerateMask handling of invalid hash name

* Address PR feedback

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

Remove unnecessary null check from QueryDynamicObject (dotnet/corefx#40320)

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

Remove dead else branch from SchemaGraph.Depends (dotnet/corefx#40325)

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

Remove duplicate casts from DbMetaDataFactory (dotnet/corefx#40328)

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

Remove redundant code in DirectorySearcher (dotnet/corefx#40321)

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

Remove duplicate branch in ColorTranslator.ToHtml (dotnet/corefx#40337)

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

Avoid tracing-induced null ref in HttpListener when {Begin}Read/Write called with invalid buffer (dotnet/corefx#40335)

HttpRequest/ResponseStream's {Begin}Read/Write methods trace the length of the caller-supplied buffer.  If the caller erroneously passes null, with tracing enabled it'll end up throwing a NullReferenceException instead of an ArgumentNullException.  This should never happen in real-world code, but might as well clean it up.

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

Fix null ref in ImmutableArray's structural comparer (dotnet/corefx#40341)

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

Delete redundant Debug.Assert in XPathNodePointer.ComparePosition

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

Remove unnecessary null check from JsonClassInfo.CreatePolymorphicProperty (dotnet/corefx#40334)

`property` will never be null in calls to this method, and even if it were, prior dereferences of it would result in exceptions.

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

Remove stale ctor and field from ObjectManager (dotnet/corefx#40326)

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

Remove dead branches from ObjectManager (dotnet/corefx#40327)

int.MaxValue is odd.  x * 2 will never be odd.

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

Fix potential null ref in RuntimeInformation.FrameworkDescription (dotnet/corefx#40329)

If there's no FX_PRODUCT_VERSION and then if there's no AssemblyInformationalVersionAttribute on the assembly or its InformationVersion is null, we'll try to dereference a null string.

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

Add comment to explain base.Pop call in CorrelationManager

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

JsonObject implementation and tests (dotnet/corefx#40206)

* JsonObject implementation added
* JsonObject tests added
* JsonObject documentation added
* Review comments included

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

Add IPEndPoint tests and fix some bugs (dotnet/corefx#39726)

* Cleanup IPEndPoint

* Add missing validation to IPEndPoint

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

Fix JSON Writer docs for data writter as Base64-encoded text (dotnet/corefx#40200)

* Fix JSON Writer docs for data writter as Base64-encoded text

* PR Feedback

* More Feedback

* One more try

Co-Authored-By: Ahson Khan <ahkha@microsoft.com>
* Use Base64 instead

* As a to as for text

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

Add NotNullIfNotNull attribute to Interlocked.CompareExchange (dotnet/corefx#40258)

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

Make many fields readonly or const across corefx (dotnet/corefx#40298)

* Make many fields readonly or const

* Address PR feedback

- Updated asn.xslt
- Regenerated .xml.cs files

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

Consolidate the Image class, fix bug (dotnet/corefx#40181)

* Add new values for Encoder, EncoderParameterValueType

* Support encoder parameters of type ValueTypePointer

* Fix an overflow error when reading EncoderParameters

* Add unit tests

* Consolidate the Image class

Share the following methods across Windows & Unix:
- FromFile
- GetEncoderParameterList
- GetPixelFormatSize
- IsAlphaPixelFormat

* Fix P/Invoke declaration

* Fix compliation on NetFX

* Skip test on NetFX

* Fix typo

* Always skip GetEncoderParameterList_ReturnsExpected on netFX

* Remove new API

* Revert changes which require an API change

* Suppress failing test

* Image: Consolidate CreateImageObject
Metafile: Consolidate initialization from native handle

* Throw SkipTestException instead of silently skipping the test

* Enable some on Unix which rely on consolidated code

* Add missing using statement

* Get SkipTestException right

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

Fixed Dispose in JsonDocument and JsonDocument.MetadataDb in order to guard against multiple calls to ArrayPool.Return (dotnet/corefx#40290)

* Fixed Dispose in JsonDocument and JsonDocument.MetadataDb in order to guard against multiple calls to ArrayPool.Return.

* Added condition to validate _utf8Json.Length is > 0 in order to prevent that extraRentedBytes.AsSpan(0, length).Clear() takes a zero length range in a race condition.

* Added test related to this scenario.

* Addressing suggestions.

* Removing unnecesary null assingment.

* Adding comment explaining the reason for minimumLength = 4 on ArrayPool.Rent.

* Adding using System.Threading.

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

[master] Update dependencies from dotnet/coreclr (dotnet/corefx#40309)

* Update dependencies from https://github.com/dotnet/coreclr build 20190813.7

- Microsoft.NET.Sdk.IL - 5.0.0-alpha1.19413.7
- Microsoft.NETCore.ILAsm - 5.0.0-alpha1.19413.7
- Microsoft.NETCore.Runtime.CoreCLR - 5.0.0-alpha1.19413.7

* Fix attribute target in ref

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

Remove Microsoft.VisualBasic.{ApplicationServices,Devices,MyServices} (dotnet/corefx#39972) (dotnet/corefx#40292)

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