Weird indentation for match arms in chain

[jdm]

fn default_user_agent_string(agent: UserAgent) -> String {
    match agent {
        UserAgent::Desktop => {
            DESKTOP_UA_STRING
        }
        UserAgent::Android => {
            "Mozilla/5.0 (Android; Mobile; rv:37.0) Servo/1.0 Firefox/37.0"
        }
    }.to_owned()
}

fn main() {}

yields

fn default_user_agent_string(agent: UserAgent) -> String {
    match agent {
            UserAgent::Desktop => DESKTOP_UA_STRING,
            UserAgent::Android => "Mozilla/5.0 (Android; Mobile; rv:37.0) Servo/1.0 Firefox/37.0",
        }
        .to_owned()
}

fn main() {} main() {}⏎

[jdm]

It works fine if the .to_owned() isn't present in the original.

[marcusklaas]

This is sort-of by design. Didn't you want to hold off the default settings discussion until after 1.0 @nrc?

[nrc]

How is it by design? I do want to hold off on getting the defaults exactly right, but this seems like a bug.

[marcusklaas]

This is the exact discussion we had here, if I'm not mistaken.

[nrc]

Oh yes, it is! I thought this would be fixed by #1016, but apparently not?

[marcusklaas]

This isn't question mark related. I'm glad you seem to agree now that it needs fixing though ;-)

[nrc]

Oh right, it is this thing. Yeah, looking at it with fresh eyes, it is pretty bad. Still don't like the alternative though.

[topecongiro]

Closing since this is fixed on the current master.