fp.

Cloudflare does not notice (until a customer complains) that they are sending broken responses at scale? I would have thought they would notice this from sampling and linting a few replies.. just in case they did something like Cloudbleed again.

logicchains•37m ago

How does terrible code like this survive so long in such a key piece of infrastructure:

        let _ = self.poll_read(cx)?;
        let _ = self.poll_write(cx)?;
        let _ = self.poll_flush(cx)?;

Surely at the very least a linter should have flagged that the return values aren't handled.

3form•27m ago

Well, simple. You write a project without a linter, so as to be fast. Or on minimal settings. Then one day you turn it on and end up with 10k warnings at max checks. You ignore them and tune it down to a bearable level. Problem solved.

That, or the linter sucks (or the idiom is too popular to be linted against, so then it's more of a language issue).

lifthrasiir•11m ago

It is an explicit way to discard return values; `self.poll_read(cx)?` etc. alone would warn. Or in this case, `Poll<Result<(), Error>>` is unwrapped once and `Result<(), Error>` is being discarded. The decision to discard `Result<(), Error>` should have been intentional, albeit turned out to be not always the case.

QuantumNomad_•9m ago

Assigning to _ by Rust convention specifically means that you intentionally want to discard the value, and the clippy linter knows that.