Skip to content

Inconsistent indentation between is and other operators #1601

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
PetSerAl opened this issue May 8, 2025 · 0 comments
Open

Inconsistent indentation between is and other operators #1601

PetSerAl opened this issue May 8, 2025 · 0 comments

Comments

@PetSerAl
Copy link
Contributor

PetSerAl commented May 8, 2025

To think more about #1562. I think it should have been resolved in different direction (removing indentation in non-parenthesis case). Because it stand out as inconsistent with outer operators and is itself in different contexts.
Input:

var b1 = System.Environment.SpecialFolder.AdminTools is System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools;
var b2 = System.Environment.SpecialFolder.AdminTools is System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools || true;
var b3 = (System.Environment.SpecialFolder.AdminTools is System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools);
var b4 = (System.Environment.SpecialFolder.AdminTools is System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools || true);
var b5 = !(System.Environment.SpecialFolder.AdminTools is System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools);
var b6 = !(System.Environment.SpecialFolder.AdminTools is System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools || true);
var b7 = ((System.Environment.SpecialFolder.AdminTools is System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools));
var b8 = ((System.Environment.SpecialFolder.AdminTools is System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools || true));

if (System.Environment.SpecialFolder.AdminTools is System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools);
if (System.Environment.SpecialFolder.AdminTools is System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools || true);
if ((System.Environment.SpecialFolder.AdminTools is System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools));
if ((System.Environment.SpecialFolder.AdminTools is System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools or System.Environment.SpecialFolder.AdminTools || true));

var i1 = 000_000_000_000_000_000_000_000_000_000 + 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000;
var i2 = 000_000_000_000_000_000_000_000_000_000 + 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000 | 0;
var i3 = (000_000_000_000_000_000_000_000_000_000 + 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000);
var i4 = (000_000_000_000_000_000_000_000_000_000 + 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000 | 0);
var i5 = ~(000_000_000_000_000_000_000_000_000_000 + 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000);
var i6 = ~(000_000_000_000_000_000_000_000_000_000 + 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000 | 0);
var i7 = ((000_000_000_000_000_000_000_000_000_000 + 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000));
var i8 = ((000_000_000_000_000_000_000_000_000_000 + 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000 | 0));

if (000_000_000_000_000_000_000_000_000_000 + 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000);
if (000_000_000_000_000_000_000_000_000_000 + 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000 | 0);
if ((000_000_000_000_000_000_000_000_000_000 + 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000));
if ((000_000_000_000_000_000_000_000_000_000 + 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000 * 000_000_000_000_000_000_000_000_000_000 | 0));

Output:

var b1 =
    System.Environment.SpecialFolder.AdminTools
        is System.Environment.SpecialFolder.AdminTools
            or System.Environment.SpecialFolder.AdminTools
            or System.Environment.SpecialFolder.AdminTools;
var b2 =
    System.Environment.SpecialFolder.AdminTools
        is System.Environment.SpecialFolder.AdminTools
            or System.Environment.SpecialFolder.AdminTools
            or System.Environment.SpecialFolder.AdminTools
    || true;
var b3 = (
    System.Environment.SpecialFolder.AdminTools
        is System.Environment.SpecialFolder.AdminTools
            or System.Environment.SpecialFolder.AdminTools
            or System.Environment.SpecialFolder.AdminTools
);
var b4 = (
    System.Environment.SpecialFolder.AdminTools
        is System.Environment.SpecialFolder.AdminTools
            or System.Environment.SpecialFolder.AdminTools
            or System.Environment.SpecialFolder.AdminTools
    || true
);
var b5 = !(
    System.Environment.SpecialFolder.AdminTools
    is System.Environment.SpecialFolder.AdminTools
        or System.Environment.SpecialFolder.AdminTools
        or System.Environment.SpecialFolder.AdminTools
);
var b6 = !(
    System.Environment.SpecialFolder.AdminTools
        is System.Environment.SpecialFolder.AdminTools
            or System.Environment.SpecialFolder.AdminTools
            or System.Environment.SpecialFolder.AdminTools
    || true
);
var b7 = (
    (
        System.Environment.SpecialFolder.AdminTools
        is System.Environment.SpecialFolder.AdminTools
            or System.Environment.SpecialFolder.AdminTools
            or System.Environment.SpecialFolder.AdminTools
    )
);
var b8 = (
    (
        System.Environment.SpecialFolder.AdminTools
            is System.Environment.SpecialFolder.AdminTools
                or System.Environment.SpecialFolder.AdminTools
                or System.Environment.SpecialFolder.AdminTools
        || true
    )
);

if (
    System.Environment.SpecialFolder.AdminTools
    is System.Environment.SpecialFolder.AdminTools
        or System.Environment.SpecialFolder.AdminTools
        or System.Environment.SpecialFolder.AdminTools
)
    ;
if (
    System.Environment.SpecialFolder.AdminTools
        is System.Environment.SpecialFolder.AdminTools
            or System.Environment.SpecialFolder.AdminTools
            or System.Environment.SpecialFolder.AdminTools
    || true
)
    ;
if (
    (
        System.Environment.SpecialFolder.AdminTools
        is System.Environment.SpecialFolder.AdminTools
            or System.Environment.SpecialFolder.AdminTools
            or System.Environment.SpecialFolder.AdminTools
    )
)
    ;
if (
    (
        System.Environment.SpecialFolder.AdminTools
            is System.Environment.SpecialFolder.AdminTools
                or System.Environment.SpecialFolder.AdminTools
                or System.Environment.SpecialFolder.AdminTools
        || true
    )
)
    ;

var i1 =
    000_000_000_000_000_000_000_000_000_000
    + 000_000_000_000_000_000_000_000_000_000
        * 000_000_000_000_000_000_000_000_000_000
        * 000_000_000_000_000_000_000_000_000_000;
var i2 =
    000_000_000_000_000_000_000_000_000_000
        + 000_000_000_000_000_000_000_000_000_000
            * 000_000_000_000_000_000_000_000_000_000
            * 000_000_000_000_000_000_000_000_000_000
    | 0;
var i3 = (
    000_000_000_000_000_000_000_000_000_000
    + 000_000_000_000_000_000_000_000_000_000
        * 000_000_000_000_000_000_000_000_000_000
        * 000_000_000_000_000_000_000_000_000_000
);
var i4 = (
    000_000_000_000_000_000_000_000_000_000
        + 000_000_000_000_000_000_000_000_000_000
            * 000_000_000_000_000_000_000_000_000_000
            * 000_000_000_000_000_000_000_000_000_000
    | 0
);
var i5 = ~(
    000_000_000_000_000_000_000_000_000_000
    + 000_000_000_000_000_000_000_000_000_000
        * 000_000_000_000_000_000_000_000_000_000
        * 000_000_000_000_000_000_000_000_000_000
);
var i6 = ~(
    000_000_000_000_000_000_000_000_000_000
        + 000_000_000_000_000_000_000_000_000_000
            * 000_000_000_000_000_000_000_000_000_000
            * 000_000_000_000_000_000_000_000_000_000
    | 0
);
var i7 = (
    (
        000_000_000_000_000_000_000_000_000_000
        + 000_000_000_000_000_000_000_000_000_000
            * 000_000_000_000_000_000_000_000_000_000
            * 000_000_000_000_000_000_000_000_000_000
    )
);
var i8 = (
    (
        000_000_000_000_000_000_000_000_000_000
            + 000_000_000_000_000_000_000_000_000_000
                * 000_000_000_000_000_000_000_000_000_000
                * 000_000_000_000_000_000_000_000_000_000
        | 0
    )
);

if (
    000_000_000_000_000_000_000_000_000_000
    + 000_000_000_000_000_000_000_000_000_000
        * 000_000_000_000_000_000_000_000_000_000
        * 000_000_000_000_000_000_000_000_000_000
)
    ;
if (
    000_000_000_000_000_000_000_000_000_000
        + 000_000_000_000_000_000_000_000_000_000
            * 000_000_000_000_000_000_000_000_000_000
            * 000_000_000_000_000_000_000_000_000_000
    | 0
)
    ;
if (
    (
        000_000_000_000_000_000_000_000_000_000
        + 000_000_000_000_000_000_000_000_000_000
            * 000_000_000_000_000_000_000_000_000_000
            * 000_000_000_000_000_000_000_000_000_000
    )
)
    ;
if (
    (
        000_000_000_000_000_000_000_000_000_000
            + 000_000_000_000_000_000_000_000_000_000
                * 000_000_000_000_000_000_000_000_000_000
                * 000_000_000_000_000_000_000_000_000_000
        | 0
    )
)
    ;

Expected behavior:

var b1 =
    System.Environment.SpecialFolder.AdminTools
    is System.Environment.SpecialFolder.AdminTools
        or System.Environment.SpecialFolder.AdminTools
        or System.Environment.SpecialFolder.AdminTools;
var b3 = (
    System.Environment.SpecialFolder.AdminTools
    is System.Environment.SpecialFolder.AdminTools
        or System.Environment.SpecialFolder.AdminTools
        or System.Environment.SpecialFolder.AdminTools
);
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@PetSerAl