Unused result lint should not fail if the result is contained by a struct

[carllerche]

In my opinion, the following should not fail:

#![deny(unused_result)]

pub struct Zomg {
    res: Result<(), ()>
}

fn zomg() -> Zomg {
    Zomg { res: Ok(()) }
}

pub fn main() {
    zomg();
}

[alexcrichton]

There are actually two lints in play here, the unused_result and unused_must_use lint. The latter is turned on by default, whereas the former is not turned on by default.

The unused_result lint is much broader than the unused_must_use lint where it applies to any type, not just those marked with #[must_use]. In this example you're opting-in to the more strict semantics instead of relying on just unused_must_use.

Just the must-use lint will not warn about this program, so I'm going to close this as working as intended.