Apply clippy fixes

[fstxz]

No description provided.

[fstxz]

I silenced this warning, because if we collapse all 3 branches into one, the code becomes way less readable:

if is_function
    || (is_inner_class
        && matches!(
            previous_token_kind,
            Some(TokenKind::Method) | Some(TokenKind::InnerClass),
        ))
{
    output.push_str("\n\n");
} else {
    output.push('\n');
}

About this lint: https://rust-lang.github.io/rust-clippy/master/index.html#if_same_then_else

[NathanLovato]

Thank you! What do you think of the unwraps and other warnings suggested in #145 ? If I got it, turning unwraps into expects is just a way to give explicit error messages?

[fstxz]

What do you think of the unwraps and other warnings suggested in #145 ?

I think only unwrap_used lint is worth adding, but I personally never seen a project that has it enabled. Other lints are either related to style, or very minor performance improvements. But if someone else wants to fix these, they can do so. We can always disable them later if they are too annoying.

If I got it, turning unwraps into expects is just a way to give explicit error messages?

Yes, but we can also remove some unwraps completely and handle errors more gracefully.

[NathanLovato]

Thanks, makes sense!