Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Delete dead code after Hashtable.cs move#29432

Merged
stephentoub merged 1 commit intodotnet:masterfrom
jkotas:deadcode
May 1, 2018
Merged

Delete dead code after Hashtable.cs move#29432
stephentoub merged 1 commit intodotnet:masterfrom
jkotas:deadcode

Conversation

@jkotas
Copy link
Member

@jkotas jkotas commented May 1, 2018

No description provided.

</data>
<data name="ArgumentOutOfRange_FileLengthTooBig" xml:space="preserve">
<value>Specified file length was too large for the file system.</value>
</data>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one appears to still be used:

/Users/dotnet-bot/j/workspace/dotnet_corefx/master/osx-TGroup_netcoreapp+CGroup_Debug+AGroup_x64+TestOuter_false_prtest/src/Common/src/CoreLib/Interop/Unix/Interop.IOErrors.cs(154,68): error CS0117: 'SR' does not contain a definition for 'ArgumentOutOfRange_FileLengthTooBig' [/Users/dotnet-bot/j/workspace/dotnet_corefx/master/osx-TGroup_netcoreapp+CGroup_Debug+AGroup_x64+TestOuter_false_prtest/src/System.Runtime.Extensions/src/System.Runtime.Extensions.csproj]

Copy link
Member

@stephentoub stephentoub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM once CI is green.

@stephentoub stephentoub merged commit 4ec2d19 into dotnet:master May 1, 2018
@jkotas jkotas deleted the deadcode branch May 1, 2018 22:21
@karelz karelz added this to the 2.2.0 milestone May 5, 2018
morganbr pushed a commit to morganbr/corefx that referenced this pull request Jul 7, 2018
zacharycmontoya pushed a commit that referenced this pull request Aug 14, 2018
* Manually update the version of Microsoft.TargetingPack.Private.NETNative

* Set Microsoft.TargetPack.Private.NETNative version to track live TFS version

* Move Hashtable & friends to shared parition

Include serialization info table for Hashtable

Remove hashtable & hashprovider src link

* Compile hashtable & friends only on uapaot

* Fix build breaks

* Remove Opcodes due to Opcodes moved to S.P.Corelib

* Move Hashtable & friends to shared parition (dotnet/coreclr#17316) (#29307)

* Move Hashtable & friends to shared parition

* Move HashHelper serialization logic into its own file

* Remove unchecked keyword in Hashtable

Signed-off-by: dotnet-bot-corefx-mirror <dotnet-bot@microsoft.com>

* Moving ConcurrentQueue to shared folder (dotnet/coreclr#17956)

* Moving ConcurrentQueue to shared folder

Fixes: #17751

Signed-off-by: dotnet-bot-corefx-mirror <dotnet-bot@microsoft.com>

* Moving ConcurrentQueue to shared (dotnet/coreclr#18024)

Making IProducerConsumerCollectionDebugView apis public

Fixes: #17751

Signed-off-by: dotnet-bot-corefx-mirror <dotnet-bot@microsoft.com>

* Modify corefx to use ConcurrentQueue from shared

* Delete dead code after Hashtable.cs move (#29432)

* Replace LowLevelDictionary with Dictionary (#29411)

* Avoid substring allocations in WebUtility.HtmlDecode (#29402)

* Avoid substring allocations in WebUtility.HtmlDecode

* Update changes to HtmlDecode based on feedback

- Use regular Dictionary instead of LowLevelDictionary
- Use AsSpan overload instead of AsSpan() and Slice
- Avoid shift by variable amount
- Use helper method to generate keys to make the code easier to maintain

* Assert that entity length is <= 8 in ToUInt64Key

- Add assert to ToUInt64Key
- Replace default with 0

* Remove CoreSetup, External, and Sni dependencies

* Update the CoreFx dependency to release/2.1 so that Microsoft.NETCore.Platforms is defined correctly. Manually invoke the dependency update

* Removing Fedora 26 and adding 28 as appropriate.  Fix a couple README.md typos.

* Remove Ubuntu 17.10 from CoreFX official runs
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants