Uploaded image for project: 'Apache Avro'
  1. Apache Avro
  2. AVRO-3608

Rust: Fix clippy errors in Rust 1.63.0

    XMLWordPrintableJSON

Details

    Description

      Rust clippy (lint) checks fail after the release of Rust 1.63.0.

       

       error: you are deriving `PartialEq` and can implement `Eq`
        --> avro/src/codec.rs:37:30
         |
      37 | #[derive(Clone, Copy, Debug, PartialEq, EnumIter, EnumString, IntoStaticStr)]
         |                              ^^^^^^^^^ help: consider deriving `Eq` as well: `PartialEq, Eq`
         |
         = note: `-D clippy::derive-partial-eq-without-eq` implied by `-D clippy::all`
         = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#derive_partial_eq_without_eqerror: you are deriving `PartialEq` and can implement `Eq`
        --> avro/src/duration.rs:30:30
         |
      30 | #[derive(Debug, Copy, Clone, PartialEq)]
         |                              ^^^^^^^^^ help: consider deriving `Eq` as well: `PartialEq, Eq`
         |
         = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#derive_partial_eq_without_eqerror: you are deriving `PartialEq` and can implement `Eq`
        --> avro/src/duration.rs:57:30
         |
      57 | #[derive(Debug, Copy, Clone, PartialEq)]
         |                              ^^^^^^^^^ help: consider deriving `Eq` as well: `PartialEq, Eq`
         |
         = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#derive_partial_eq_without_eqerror: you are deriving `PartialEq` and can implement `Eq`
        --> avro/src/duration.rs:84:30
         |
      84 | #[derive(Debug, Copy, Clone, PartialEq)]
         |                              ^^^^^^^^^ help: consider deriving `Eq` as well: `PartialEq, Eq`
         |
         = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#derive_partial_eq_without_eqerror: deref on an immutable reference
         --> avro/src/encode.rs:153:33
          |
      153 |                 encode_internal(&*item, inner_schema, names, enclosing_namespace, buffer)?;
          |                                 ^^^^^^
          |
          = note: `-D clippy::borrow-deref-ref` implied by `-D clippy::all`
          = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#borrow_deref_ref
      help: if you would like to reborrow, try removing `&*`
          |
      153 |                 encode_internal(item, inner_schema, names, enclosing_namespace, buffer)?;
          |                                 ~~~~
      help: if you would like to deref, try using `&**`
          |
      153 |                 encode_internal(&**item, inner_schema, names, enclosing_namespace, buffer)?;
          |                                 ~~~~~~~error: you are deriving `PartialEq` and can implement `Eq`
         --> avro/src/schema.rs:581:24
          |
      581 | #[derive(Clone, Debug, PartialEq, EnumString)]
          |                        ^^^^^^^^^ help: consider deriving `Eq` as well: `PartialEq, Eq`
          |
          = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#derive_partial_eq_without_eqerror: unneeded `return` statement
         --> avro/src/types.rs:372:21
          |
      372 | /                     return Some(format!(
      373 | |                         "Unresolved schema reference: '{}'. Parsed names: {:?}",
      374 | |                         name,
      375 | |                         names.keys()
      376 | |                     ));
          | |_______________________^
          |
          = note: `-D clippy::needless-return` implied by `-D clippy::all`
          = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
      help: remove `return`
          |
      372 ~                     Some(format!(
      373 +                         "Unresolved schema reference: '{}'. Parsed names: {:?}",
      374 +                         name,
      375 +                         names.keys()
      376 +                     ))
          |error: could not compile `apache-avro` due to 7 previous errors
      warning: build failed, waiting for other jobs to finish...
      error: could not compile `apache-avro` due to 7 previous errors
      

      Attachments

        Issue Links

          Activity

            People

              mgrigorov Martin Tzvetanov Grigorov
              mgrigorov Martin Tzvetanov Grigorov
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 20m
                  20m