Skip to content

Use new IndexOfAnyValues throughout dotnet/runtime #78204

Closed

Description

@stephentoub

Once #78093 is merged, we'll want to use it in a variety of places where it'll provide benefits.

Some example possible use sites:


Already updated:

Activity

  1. added this to the 8.0.0 milestone on Nov 11, 2022
  2. self-assigned this
    on Nov 12, 2022
  3. adamsitnik commented on Dec 15, 2022

    @adamsitnik
    Member

    @MihaZupan I can see that you have merged plenty of PRs related to this issue. Is it done now?

  4. MihaZupan commented on Dec 17, 2022

    @MihaZupan
    Member

    My queries for interesting patterns that could be replaced are starting to dry up.
    There's still a handful of places that can be updated, after that, I'll close this issue.

    Still on my TODO to look at:

    https://github.com/dotnet/runtime/blob/main/src/libraries/Microsoft.Extensions.FileProviders.Physical/src/Internal/PathUtils.cs
    https://github.com/dotnet/runtime/blob/main/src/libraries/System.Private.DataContractSerialization/src/System/Xml/XmlUTF8TextWriter.cs
    https://github.com/dotnet/runtime/blob/main/src/libraries/System.Data.Common/src/System/Data/Filter/ExpressionParser.cs
    
  5. ghost added
    in-prThere is an active PR which will close this issue when it is merged
    on Apr 22, 2023
  6. ghost removed
    in-prThere is an active PR which will close this issue when it is merged
    on Apr 22, 2023
  7. ghost locked as resolved and limited conversation to collaborators on May 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions