Skip to content

Inconsistent indentation between is and other operators #1601

@PetSerAl

Description

@PetSerAl

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
);

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions