Skip to content

Simplification and cleanup of our symbol index now that it is OOP.#43804

Merged
17 commits merged intodotnet:masterfrom
CyrusNajmabadi:indicesCaching
May 2, 2020
Merged

Simplification and cleanup of our symbol index now that it is OOP.#43804
17 commits merged intodotnet:masterfrom
CyrusNajmabadi:indicesCaching

Conversation

@CyrusNajmabadi
Copy link
Contributor

@CyrusNajmabadi CyrusNajmabadi commented Apr 29, 2020

Followup to #43801. That should go in first.

this should be reviewed commit-by-commit.

@CyrusNajmabadi CyrusNajmabadi requested a review from a team as a code owner April 29, 2020 22:34
// The .NET Foundation licenses this file to you under the MIT license.
// See the LICENSE file in the project root for more information.

using System.Collections.Concurrent;
Copy link
Member

Choose a reason for hiding this comment

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

using [](start = 0, length = 5)

#nullable enable

// The .NET Foundation licenses this file to you under the MIT license.
// See the LICENSE file in the project root for more information.

using System.Collections.Concurrent;
Copy link
Member

Choose a reason for hiding this comment

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

using [](start = 0, length = 5)

#nullable enable

// The .NET Foundation licenses this file to you under the MIT license.
// See the LICENSE file in the project root for more information.

using System.Collections.Generic;
Copy link
Member

Choose a reason for hiding this comment

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

using [](start = 0, length = 5)

#nullable enable

Copy link
Member

@tmat tmat left a comment

Choose a reason for hiding this comment

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

:shipit:

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Auto-approval

@ghost ghost merged commit b3bc9c5 into dotnet:master May 2, 2020
@CyrusNajmabadi CyrusNajmabadi deleted the indicesCaching branch May 2, 2020 23:38
@jinujoseph jinujoseph added the Concept-OOP Related to out-of-proc label May 3, 2020
@jinujoseph jinujoseph added this to the Next milestone May 3, 2020
@JoeRobich JoeRobich modified the milestones: Next, 16.7.P2 May 18, 2020
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants