Support rewriting string.Remove in IDE0057 "Use range operator"

[reflectronic]

The IDE0057 code style analyzer suggests replacing calls to string.Substring with the range operator. In some cases, calls to string.Remove can also be replaced with the slice syntax. The analyzer should suggest replacing such calls as well.

The following forms are equivalent:

• string.Remove(string.Length - x)
• string.Remove(string.Length - x, x)
• string.Substring(0, string.Length - x)
• string[..^x]

And:

• string.Remove(0, string.Length - x)
• string.Substring(string.Length - x)
• string[^x..]

And:

• string.Remove(x, string.Length - x)
• string.Substring(0, x)
• string.Remove(x)
• string[..x]

And:

• string.Remove(0, x)
• string.Substring(x)
• string[x..]

And:

• string.Remove(x, 0)
• string

I believe there are no more cases where string.Remove may be rewritten as a slice.

[CyrusNajmabadi]

This doesn't seem to be the case. For example:

        var v = "the quick brown fox jumped over the lazy dog";
        Console.WriteLine(v.Remove(10));
        Console.WriteLine(v[..^10]);

[reflectronic]

Sorry, I meant to write string.Remove(string.Length - x) there :-(

string.Remove(x) should be next to string[..x]