SA1008, SA1012: false positives for property pattern inside parentheses

[adiessl]

Using StyleCop.Analyzers version 1.2.0-beta.556

public static class Helpers
{
    public static bool HasValidLength(string s)
    {
        // ⚠ SA1008 Opening parenthesis should not be preceded by a space.
        // ⚠ SA1012 Opening brace should be preceded by a space
        //              ↓↓
        return s is not ({ Length: < 5 } or { Length: > 10 });
    }
}

This is similar to the problem fixed in #3141.

[bjornhellander]

I can confirm SA1012, but I actually don't see a SA1008 in this example. Weird.

[adiessl]

Yes, technically correct it would be like this:

public static class Helpers
{
    public static bool HasValidLength(string s)
    {
        // ⚠ SA1012 Opening brace should be preceded by a space
        //               ↓
        return s is not ({ Length: < 5 } or { Length: > 10 });
    }
}

"Fixing" this leads to the other error:

public static class Helpers
{
    public static bool HasValidLength(string s)
    {
        // ⚠ SA1008 Opening parenthesis should not be preceded by a space.
        //                ↓
        return s is not ( { Length: < 5 } or { Length: > 10 });
    }
}

[bjornhellander]

Ok 👍

I am also seeing the same for ({ Length: < 5 } and { Length: > 2 }) and ({ Length: < 5 })