Unverified Commit 454e2b7f authored by Nugine's avatar Nugine Committed by GitHub
Browse files

deny clippy::pedantic (#39) 

* prepare clippy::pedantic

* fix clippy::cast_lossless

* fix clippy::cast_possible_truncation

* fix clippy::cast_precision_loss

* fix clippy::default_trait_access

* fix clippy::doc_markdown

* fix clippy::explicit_iter_loop

* fix clippy::implicit_clone

* fix clippy::inline_always

* fix clippy::manual_string_new

* fix clippy::match_same_arms

* fix clippy::missing_errors_doc

* fix clippy::missing_panics_doc

* fix clippy::needless_pass_by_value

* fix clippy::range_plus_one

* fix clippy::redundant_closure_for_method_calls

* fix clippy::semicolon_if_nothing_returned

* fix clippy::similar_names

* fix clippy::too_many_lines

* fix clippy::uninlined_format_args

* fix clippy::unnecessary_wraps

* fix clippy::unreadable_literal

* fix clippy::unused_self

* fix clippy::manual_let_else

* fix clippy::let_underscore_untyped
parent 24dd9fce

codegen/src/aws_conv.rs

+1 −1
Original line number Diff line number Diff line
@@ -11,7 +11,7 @@ use heck::ToUpperCamelCase; 


pub fn codegen(ops: &Operations, rust_types: &RustTypes, g: &mut Codegen) {

    g.lines([

        "//! Auto generated by codegen/src/aws_conv.rs", //

        "//! Auto generated by `codegen/src/aws_conv.rs`", //

        "",

        "use super::*;",

        "",

codegen/src/aws_proxy.rs

+1 −1
Original line number Diff line number Diff line
@@ -8,7 +8,7 @@ use heck::ToSnakeCase; 


pub fn codegen(ops: &Operations, rust_types: &RustTypes, g: &mut Codegen) {

    g.lines([

        "//! Auto generated by codegen/src/aws_proxy.rs",

        "//! Auto generated by `codegen/src/aws_proxy.rs`",

        "",

        "use super::*;",

        "",

codegen/src/dto.rs

+2 −1
Original line number Diff line number Diff line
@@ -329,9 +329,10 @@ fn unify_operation_types(ops: &Operations, space: &mut RustTypes) { 


pub fn codegen(rust_types: &RustTypes, g: &mut Codegen) {

    g.lines([

        "//! Auto generated by codegen/src/dto.rs",

        "//! Auto generated by `codegen/src/dto.rs`",

        "",

        "#![allow(clippy::empty_structs_with_brackets)]",

        "#![allow(clippy::too_many_lines)]",

        "",

        "use super::*;",

        "",

codegen/src/error.rs

+2 −2
Original line number Diff line number Diff line
@@ -120,7 +120,7 @@ pub fn codegen(model: &smithy::Model, g: &mut Codegen) { 
    let errors = collect_errors(model);



    g.lines([

        "//! Auto generated by codegen/src/error.rs",

        "//! Auto generated by `codegen/src/error.rs`",

        "",

        "use bytestring::ByteString;", //

        "use hyper::StatusCode;",      //
@@ -176,7 +176,6 @@ pub fn codegen(model: &smithy::Model, g: &mut Codegen) { 
        g.ln("];");

        g.lf();



        g.ln("#[inline(always)]");

        g.ln("#[must_use]");

        g.ln("fn as_enum_tag(&self) -> usize {");

        g.ln("match self {");
@@ -212,6 +211,7 @@ pub fn codegen(model: &smithy::Model, g: &mut Codegen) { 
    }



    {

        g.ln("#[allow(clippy::match_same_arms)]");

        g.ln("#[must_use]");

        g.ln("pub fn status_code(&self) -> Option<StatusCode> {");

codegen/src/headers.rs

+1 −1
Original line number Diff line number Diff line
@@ -30,7 +30,7 @@ pub fn codegen(model: &smithy::Model, g: &mut Codegen) { 
    }



    g.lines([

        "//! Auto generated by codegen/src/headers.rs",

        "//! Auto generated by `codegen/src/headers.rs`",

        "",

        "#![allow(clippy::declare_interior_mutable_const)]",

        "",