just checking if this is still good to be worked on? i am aware it has been a while since this was raised! if it is, i don't mind picking it up :)!

cc: @flip1995 @Manishearth @giraffate (sorry if i've overtagged someone)

Should be fine. We should figure out the name and the lint level but you can start work on it before we decide.

I'm open to the level being style or complexity as long as lintcheck isn't too noisy

yeah, that sounds good. as you said, both of those things shouldn't require a lot of re-work if i start on this now. will get a bit familiarised with the tool and we can continue the discussion either here or when i raise the pr (about naming and lint level). thanks again :).

@rustbot claim

I found myself rewriting:

let select = paren_select(p);
match select {
    Some(cm) => {
        if p.at_ts(COMPOUND_SELECT_FIRST) {
            opt_compound_select(p, cm)
        } else {
            Some(cm)
        }
    }
    None => None,
}

as

paren_select(p).and_then(|cm| {
    if p.at_ts(COMPOUND_SELECT_FIRST) {
        opt_compound_select(p, cm)
    } else {
        Some(cm)
    }
})

before switching to the question mark:

let cm = paren_select(p)?;
if p.at_ts(COMPOUND_SELECT_FIRST) {
    opt_compound_select(p, cm)
} else {
    Some(cm)
}

@rustbot claim