[INFO] crate mapbox_vector_tile 0.1.0 is already in cache
[INFO] extracting crate mapbox_vector_tile 0.1.0 into work/ex/clippy-test-run/sources/stable/reg/mapbox_vector_tile/0.1.0
[INFO] extracting crate mapbox_vector_tile 0.1.0 into work/ex/clippy-test-run/sources/stable+rustflags=-Dclippy%3A%3Ainto_iter_on_array/reg/mapbox_vector_tile/0.1.0
[INFO] validating manifest of mapbox_vector_tile-0.1.0 on toolchain stable
[INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "read-manifest" "--manifest-path" "Cargo.toml"`
[INFO] validating manifest of mapbox_vector_tile-0.1.0 on toolchain stable+rustflags=-Dclippy::into_iter_on_array
[INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "read-manifest" "--manifest-path" "Cargo.toml"`
[INFO] started frobbing mapbox_vector_tile-0.1.0
[INFO] finished frobbing mapbox_vector_tile-0.1.0
[INFO] frobbed toml for mapbox_vector_tile-0.1.0 written to work/ex/clippy-test-run/sources/stable/reg/mapbox_vector_tile/0.1.0/Cargo.toml
[INFO] started frobbing mapbox_vector_tile-0.1.0
[INFO] finished frobbing mapbox_vector_tile-0.1.0
[INFO] frobbed toml for mapbox_vector_tile-0.1.0 written to work/ex/clippy-test-run/sources/stable+rustflags=-Dclippy%3A%3Ainto_iter_on_array/reg/mapbox_vector_tile/0.1.0/Cargo.toml
[INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "generate-lockfile" "--manifest-path" "Cargo.toml" "-Zno-index-update"`
[INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "generate-lockfile" "--manifest-path" "Cargo.toml" "-Zno-index-update"`
[INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "fetch" "--locked" "--manifest-path" "Cargo.toml"`
[INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "fetch" "--locked" "--manifest-path" "Cargo.toml"`
[INFO] linting mapbox_vector_tile-0.1.0 against stable for clippy-test-run
[INFO] running `"docker" "create" "-v" "/mnt/big/crater/work/local/target-dirs/clippy-test-run/worker-1/stable:/opt/crater/target:rw,Z" "-v" "/mnt/big/crater/work/ex/clippy-test-run/sources/stable/reg/mapbox_vector_tile/0.1.0:/opt/crater/workdir:ro,Z" "-v" "/mnt/big/crater/work/local/cargo-home:/opt/crater/cargo-home:ro,Z" "-v" "/mnt/big/crater/work/local/rustup-home:/opt/crater/rustup-home:ro,Z" "-e" "USER_ID=1000" "-e" "SOURCE_DIR=/opt/crater/workdir" "-e" "MAP_USER_ID=1000" "-e" "CARGO_TARGET_DIR=/opt/crater/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/crater/cargo-home" "-e" "RUSTUP_HOME=/opt/crater/rustup-home" "-w" "/opt/crater/workdir" "-m" "1536M" "--network" "none" "rustops/crates-build-env" "/opt/crater/cargo-home/bin/cargo" "+stable" "clippy" "--frozen" "--all" "--all-targets"`
[INFO] [stdout] 46cdbee365ed862ee344126eb1cd4a0fd0b09d36c3df4027bba5737497b4a580
[INFO] running `"docker" "start" "-a" "46cdbee365ed862ee344126eb1cd4a0fd0b09d36c3df4027bba5737497b4a580"`
[INFO] [stderr]     Checking slippy-map-tiles v0.14.0
[INFO] [stderr]     Checking geo v0.7.4
[INFO] [stderr]     Checking mapbox_vector_tile v0.1.0 (/opt/crater/workdir)
[INFO] [stderr] warning: lint name `clippy` is deprecated and may not have an effect in the future. Also `cfg_attr(cargo-clippy)` won't be necessary anymore
[INFO] [stderr]  --> src/vector_tile.rs:6:10
[INFO] [stderr]   |
[INFO] [stderr] 6 | #![allow(clippy)]
[INFO] [stderr]   |          ^^^^^^ help: change it to: `clippy::all`
[INFO] [stderr]   |
[INFO] [stderr]   = note: #[warn(renamed_and_removed_lints)] on by default
[INFO] [stderr] 
[INFO] [stderr] warning: redundant field names in struct initialization
[INFO] [stderr]   --> src/lib.rs:86:18
[INFO] [stderr]    |
[INFO] [stderr] 86 |         Feature{ geometry: geometry, properties: properties }
[INFO] [stderr]    |                  ^^^^^^^^^^^^^^^^^^ help: replace it with: `geometry`
[INFO] [stderr]    |
[INFO] [stderr]    = note: #[warn(clippy::redundant_field_names)] on by default
[INFO] [stderr]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
[INFO] [stderr] 
[INFO] [stderr] warning: redundant field names in struct initialization
[INFO] [stderr]   --> src/lib.rs:86:38
[INFO] [stderr]    |
[INFO] [stderr] 86 |         Feature{ geometry: geometry, properties: properties }
[INFO] [stderr]    |                                      ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `properties`
[INFO] [stderr]    |
[INFO] [stderr]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
[INFO] [stderr] 
[INFO] [stderr] warning: redundant field names in struct initialization
[INFO] [stderr]    --> src/lib.rs:136:16
[INFO] [stderr]     |
[INFO] [stderr] 136 |         Layer{ name: name, features: Vec::new(), extent: 4096 }
[INFO] [stderr]     |                ^^^^^^^^^^ help: replace it with: `name`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
[INFO] [stderr] 
[INFO] [stderr] warning: redundant field names in struct initialization
[INFO] [stderr]    --> src/lib.rs:140:16
[INFO] [stderr]     |
[INFO] [stderr] 140 |         Layer{ name: name, features: Vec::new(), extent: extent }
[INFO] [stderr]     |                ^^^^^^^^^^ help: replace it with: `name`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
[INFO] [stderr] 
[INFO] [stderr] warning: redundant field names in struct initialization
[INFO] [stderr]    --> src/lib.rs:140:50
[INFO] [stderr]     |
[INFO] [stderr] 140 |         Layer{ name: name, features: Vec::new(), extent: extent }
[INFO] [stderr]     |                                                  ^^^^^^^^^^^^^^ help: replace it with: `extent`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
[INFO] [stderr] 
[INFO] [stderr] warning: redundant field names in struct initialization
[INFO] [stderr]    --> src/lib.rs:507:12
[INFO] [stderr]     |
[INFO] [stderr] 507 |     Layer{ name: name, extent: extent, features: features }
[INFO] [stderr]     |            ^^^^^^^^^^ help: replace it with: `name`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
[INFO] [stderr] 
[INFO] [stderr] warning: redundant field names in struct initialization
[INFO] [stderr]    --> src/lib.rs:507:24
[INFO] [stderr]     |
[INFO] [stderr] 507 |     Layer{ name: name, extent: extent, features: features }
[INFO] [stderr]     |                        ^^^^^^^^^^^^^^ help: replace it with: `extent`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
[INFO] [stderr] 
[INFO] [stderr] warning: redundant field names in struct initialization
[INFO] [stderr]    --> src/lib.rs:507:40
[INFO] [stderr]     |
[INFO] [stderr] 507 |     Layer{ name: name, extent: extent, features: features }
[INFO] [stderr]     |                                        ^^^^^^^^^^^^^^^^^^ help: replace it with: `features`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
[INFO] [stderr] 
[INFO] [stderr] warning: lint name `clippy` is deprecated and may not have an effect in the future. Also `cfg_attr(cargo-clippy)` won't be necessary anymore
[INFO] [stderr]  --> src/vector_tile.rs:6:10
[INFO] [stderr]   |
[INFO] [stderr] 6 | #![allow(clippy)]
[INFO] [stderr]   |          ^^^^^^ help: change it to: `clippy::all`
[INFO] [stderr]   |
[INFO] [stderr]   = note: #[warn(renamed_and_removed_lints)] on by default
[INFO] [stderr] 
[INFO] [stderr] warning: redundant field names in struct initialization
[INFO] [stderr]   --> src/lib.rs:86:18
[INFO] [stderr]    |
[INFO] [stderr] 86 |         Feature{ geometry: geometry, properties: properties }
[INFO] [stderr]    |                  ^^^^^^^^^^^^^^^^^^ help: replace it with: `geometry`
[INFO] [stderr]    |
[INFO] [stderr]    = note: #[warn(clippy::redundant_field_names)] on by default
[INFO] [stderr]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
[INFO] [stderr] 
[INFO] [stderr] warning: redundant field names in struct initialization
[INFO] [stderr]   --> src/lib.rs:86:38
[INFO] [stderr]    |
[INFO] [stderr] 86 |         Feature{ geometry: geometry, properties: properties }
[INFO] [stderr]    |                                      ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `properties`
[INFO] [stderr]    |
[INFO] [stderr]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
[INFO] [stderr] 
[INFO] [stderr] warning: redundant field names in struct initialization
[INFO] [stderr]    --> src/lib.rs:136:16
[INFO] [stderr]     |
[INFO] [stderr] 136 |         Layer{ name: name, features: Vec::new(), extent: 4096 }
[INFO] [stderr]     |                ^^^^^^^^^^ help: replace it with: `name`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
[INFO] [stderr] 
[INFO] [stderr] warning: redundant field names in struct initialization
[INFO] [stderr]    --> src/lib.rs:140:16
[INFO] [stderr]     |
[INFO] [stderr] 140 |         Layer{ name: name, features: Vec::new(), extent: extent }
[INFO] [stderr]     |                ^^^^^^^^^^ help: replace it with: `name`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
[INFO] [stderr] 
[INFO] [stderr] warning: redundant field names in struct initialization
[INFO] [stderr]    --> src/lib.rs:140:50
[INFO] [stderr]     |
[INFO] [stderr] 140 |         Layer{ name: name, features: Vec::new(), extent: extent }
[INFO] [stderr]     |                                                  ^^^^^^^^^^^^^^ help: replace it with: `extent`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
[INFO] [stderr] 
[INFO] [stderr] warning: redundant field names in struct initialization
[INFO] [stderr]    --> src/lib.rs:507:12
[INFO] [stderr]     |
[INFO] [stderr] 507 |     Layer{ name: name, extent: extent, features: features }
[INFO] [stderr]     |            ^^^^^^^^^^ help: replace it with: `name`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
[INFO] [stderr] 
[INFO] [stderr] warning: redundant field names in struct initialization
[INFO] [stderr]    --> src/lib.rs:507:24
[INFO] [stderr]     |
[INFO] [stderr] 507 |     Layer{ name: name, extent: extent, features: features }
[INFO] [stderr]     |                        ^^^^^^^^^^^^^^ help: replace it with: `extent`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
[INFO] [stderr] 
[INFO] [stderr] warning: redundant field names in struct initialization
[INFO] [stderr]    --> src/lib.rs:507:40
[INFO] [stderr]     |
[INFO] [stderr] 507 |     Layer{ name: name, extent: extent, features: features }
[INFO] [stderr]     |                                        ^^^^^^^^^^^^^^^^^^ help: replace it with: `features`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
[INFO] [stderr] 
[INFO] [stderr] warning: lint name `clippy` is deprecated and may not have an effect in the future. Also `cfg_attr(cargo-clippy)` won't be necessary anymore
[INFO] [stderr]  --> src/vector_tile.rs:6:10
[INFO] [stderr]   |
[INFO] [stderr] 6 | #![allow(clippy)]
[INFO] [stderr]   |          ^^^^^^ help: change it to: `clippy::all`
[INFO] [stderr] 
[INFO] [stderr] warning: returning the result of a let binding from a block. Consider returning the expression directly.
[INFO] [stderr]    --> src/lib.rs:381:9
[INFO] [stderr]     |
[INFO] [stderr] 381 |         new_bytes
[INFO] [stderr]     |         ^^^^^^^^^
[INFO] [stderr]     |
[INFO] [stderr]     = note: #[warn(clippy::let_and_return)] on by default
[INFO] [stderr] note: this expression can be directly returned
[INFO] [stderr]    --> src/lib.rs:379:25
[INFO] [stderr]     |
[INFO] [stderr] 379 |         let new_bytes = compressor.finish().unwrap();
[INFO] [stderr]     |                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_and_return
[INFO] [stderr] 
[INFO] [stderr] warning: lint name `clippy` is deprecated and may not have an effect in the future. Also `cfg_attr(cargo-clippy)` won't be necessary anymore
[INFO] [stderr]  --> src/vector_tile.rs:6:10
[INFO] [stderr]   |
[INFO] [stderr] 6 | #![allow(clippy)]
[INFO] [stderr]   |          ^^^^^^ help: change it to: `clippy::all`
[INFO] [stderr] 
[INFO] [stderr] warning: returning the result of a let binding from a block. Consider returning the expression directly.
[INFO] [stderr]    --> src/lib.rs:381:9
[INFO] [stderr]     |
[INFO] [stderr] 381 |         new_bytes
[INFO] [stderr]     |         ^^^^^^^^^
[INFO] [stderr]     |
[INFO] [stderr]     = note: #[warn(clippy::let_and_return)] on by default
[INFO] [stderr] note: this expression can be directly returned
[INFO] [stderr]    --> src/lib.rs:379:25
[INFO] [stderr]     |
[INFO] [stderr] 379 |         let new_bytes = compressor.finish().unwrap();
[INFO] [stderr]     |                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_and_return
[INFO] [stderr] 
[INFO] [stderr] warning: you should consider deriving a `Default` implementation for `Properties`
[INFO] [stderr]   --> src/lib.rs:51:5
[INFO] [stderr]    |
[INFO] [stderr] 51 | /     pub fn new() -> Self {
[INFO] [stderr] 52 | |         Properties(HashMap::new())
[INFO] [stderr] 53 | |     }
[INFO] [stderr]    | |_____^
[INFO] [stderr]    |
[INFO] [stderr]    = note: #[warn(clippy::new_without_default_derive)] on by default
[INFO] [stderr]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#new_without_default_derive
[INFO] [stderr] help: try this
[INFO] [stderr]    |
[INFO] [stderr] 35 | #[derive(Default)]
[INFO] [stderr]    |
[INFO] [stderr] 
[INFO] [stderr] warning: explicit lifetimes given in parameter types where they could be elided (or replaced with `'_` if needed by type declaration)
[INFO] [stderr]   --> src/lib.rs:93:5
[INFO] [stderr]    |
[INFO] [stderr] 93 | /     pub fn get_point<'a>(&'a self) -> Option<Point<i32>> {
[INFO] [stderr] 94 | |         match self.geometry {
[INFO] [stderr] 95 | |             Geometry::Point(p) => Some(p),
[INFO] [stderr] 96 | |             _ => None,
[INFO] [stderr] 97 | |         }
[INFO] [stderr] 98 | |     }
[INFO] [stderr]    | |_____^
[INFO] [stderr]    |
[INFO] [stderr]    = note: #[warn(clippy::needless_lifetimes)] on by default
[INFO] [stderr]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_lifetimes
[INFO] [stderr] 
[INFO] [stderr] warning: methods called `to_*` usually take self by reference; consider choosing a less ambiguous name
[INFO] [stderr]    --> src/lib.rs:181:21
[INFO] [stderr]     |
[INFO] [stderr] 181 |     pub fn to_bytes(self) -> Vec<u8> {
[INFO] [stderr]     |                     ^^^^
[INFO] [stderr]     |
[INFO] [stderr]     = note: #[warn(clippy::wrong_self_convention)] on by default
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#wrong_self_convention
[INFO] [stderr] 
[INFO] [stderr] warning: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices.
[INFO] [stderr]    --> src/lib.rs:252:29
[INFO] [stderr]     |
[INFO] [stderr] 252 | fn create_indexes(features: &Vec<Feature>) -> (TagIndex<String>, TagIndex<Value>) {
[INFO] [stderr]     |                             ^^^^^^^^^^^^^ help: change this to: `&[Feature]`
[INFO] [stderr]     |
[INFO] [stderr]     = note: #[warn(clippy::ptr_arg)] on by default
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg
[INFO] [stderr] 
[INFO] [stderr] warning: you should consider deriving a `Default` implementation for `Tile`
[INFO] [stderr]    --> src/lib.rs:321:5
[INFO] [stderr]     |
[INFO] [stderr] 321 | /     pub fn new() -> Self {
[INFO] [stderr] 322 | |         Tile{ layers: Vec::new() }
[INFO] [stderr] 323 | |     }
[INFO] [stderr]     | |_____^
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#new_without_default_derive
[INFO] [stderr] help: try this
[INFO] [stderr]     |
[INFO] [stderr] 316 | #[derive(Default)]
[INFO] [stderr]     |
[INFO] [stderr] 
[INFO] [stderr] warning: methods called `to_*` usually take self by reference; consider choosing a less ambiguous name
[INFO] [stderr]    --> src/lib.rs:366:21
[INFO] [stderr]     |
[INFO] [stderr] 366 |     pub fn to_bytes(self) -> Vec<u8> {
[INFO] [stderr]     |                     ^^^^
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#wrong_self_convention
[INFO] [stderr] 
[INFO] [stderr] warning: methods called `to_*` usually take self by reference; consider choosing a less ambiguous name
[INFO] [stderr]    --> src/lib.rs:372:32
[INFO] [stderr]     |
[INFO] [stderr] 372 |     pub fn to_compressed_bytes(self) -> Vec<u8> {
[INFO] [stderr]     |                                ^^^^
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#wrong_self_convention
[INFO] [stderr] 
[INFO] [stderr] warning: redundant closure found
[INFO] [stderr]    --> src/lib.rs:489:54
[INFO] [stderr]     |
[INFO] [stderr] 489 |     Tile{ layers: tile.take_layers().into_iter().map(|l| pbflayer_to_layer(l)).collect() }
[INFO] [stderr]     |                                                      ^^^^^^^^^^^^^^^^^^^^^^^^ help: remove closure as shown: `pbflayer_to_layer`
[INFO] [stderr]     |
[INFO] [stderr]     = note: #[warn(clippy::redundant_closure)] on by default
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure
[INFO] [stderr] 
[INFO] [stderr] warning: you don't need to add `&` to all patterns
[INFO] [stderr]    --> src/lib.rs:521:9
[INFO] [stderr]     |
[INFO] [stderr] 521 | /         if let &DrawingCommand::MoveTo(_) = self {
[INFO] [stderr] 522 | |             true
[INFO] [stderr] 523 | |         } else {
[INFO] [stderr] 524 | |             false
[INFO] [stderr] 525 | |         }
[INFO] [stderr]     | |_________^
[INFO] [stderr]     |
[INFO] [stderr]     = note: #[warn(clippy::match_ref_pats)] on by default
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#match_ref_pats
[INFO] [stderr] help: instead of prefixing all patterns with `&`, you can dereference the expression
[INFO] [stderr]     |
[INFO] [stderr] 521 |         if let DrawingCommand::MoveTo(_) = *self {
[INFO] [stderr]     |                ^^^^^^^^^^^^^^^^^^^^^^^^^   ^^^^^
[INFO] [stderr] 
[INFO] [stderr] warning: you don't need to add `&` to all patterns
[INFO] [stderr]    --> src/lib.rs:529:9
[INFO] [stderr]     |
[INFO] [stderr] 529 | /         if let &DrawingCommand::LineTo(_) = self {
[INFO] [stderr] 530 | |             true
[INFO] [stderr] 531 | |         } else {
[INFO] [stderr] 532 | |             false
[INFO] [stderr] 533 | |         }
[INFO] [stderr]     | |_________^
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#match_ref_pats
[INFO] [stderr] help: instead of prefixing all patterns with `&`, you can dereference the expression
[INFO] [stderr]     |
[INFO] [stderr] 529 |         if let DrawingCommand::LineTo(_) = *self {
[INFO] [stderr]     |                ^^^^^^^^^^^^^^^^^^^^^^^^^   ^^^^^
[INFO] [stderr] 
[INFO] [stderr] warning: you don't need to add `&` to all patterns
[INFO] [stderr]    --> src/lib.rs:537:9
[INFO] [stderr]     |
[INFO] [stderr] 537 | /         if let &DrawingCommand::ClosePath = self {
[INFO] [stderr] 538 | |             true
[INFO] [stderr] 539 | |         } else {
[INFO] [stderr] 540 | |             false
[INFO] [stderr] 541 | |         }
[INFO] [stderr]     | |_________^
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#match_ref_pats
[INFO] [stderr] help: instead of prefixing all patterns with `&`, you can dereference the expression
[INFO] [stderr]     |
[INFO] [stderr] 537 |         if let DrawingCommand::ClosePath = *self {
[INFO] [stderr]     |                ^^^^^^^^^^^^^^^^^^^^^^^^^   ^^^^^
[INFO] [stderr] 
[INFO] [stderr] warning: this argument is passed by reference, but would be more efficient if passed by value
[INFO] [stderr]    --> src/lib.rs:599:41
[INFO] [stderr]     |
[INFO] [stderr] 599 | fn decode_geom(data: &[u32], geom_type: &vector_tile::Tile_GeomType) -> Geometry<i32> {
[INFO] [stderr]     |                                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider passing by value instead: `vector_tile::Tile_GeomType`
[INFO] [stderr]     |
[INFO] [stderr]     = note: #[warn(clippy::trivially_copy_pass_by_ref)] on by default
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#trivially_copy_pass_by_ref
[INFO] [stderr] 
[INFO] [stderr] warning: the function has a cyclomatic complexity of 39
[INFO] [stderr]    --> src/lib.rs:622:5
[INFO] [stderr]     |
[INFO] [stderr] 622 | /     fn from(commands: DrawingCommands) -> Geometry<i32> {
[INFO] [stderr] 623 | |         let geom_type = deduce_geom_type(&commands);
[INFO] [stderr] 624 | |
[INFO] [stderr] 625 | |         match geom_type {
[INFO] [stderr] ...   |
[INFO] [stderr] 763 | |         }
[INFO] [stderr] 764 | |     }
[INFO] [stderr]     | |_____^
[INFO] [stderr]     |
[INFO] [stderr]     = note: #[warn(clippy::cyclomatic_complexity)] on by default
[INFO] [stderr]     = help: you could split it up into multiple smaller functions
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cyclomatic_complexity
[INFO] [stderr] 
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr]    --> src/lib.rs:632:28
[INFO] [stderr]     |
[INFO] [stderr] 632 |                         if points.len() == 0 {
[INFO] [stderr]     |                            ^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `points.is_empty()`
[INFO] [stderr]     |
[INFO] [stderr]     = note: #[warn(clippy::len_zero)] on by default
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stderr] 
[INFO] [stderr] warning: this .into_iter() call is equivalent to .iter() and will not move the Vec
[INFO] [stderr]    --> src/lib.rs:640:45
[INFO] [stderr]     |
[INFO] [stderr] 640 |                             for p in points.into_iter() {
[INFO] [stderr]     |                                             ^^^^^^^^^ help: call directly: `iter`
[INFO] [stderr]     |
[INFO] [stderr]     = note: #[warn(clippy::into_iter_on_ref)] on by default
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#into_iter_on_ref
[INFO] [stderr] 
[INFO] [stderr] warning: manual implementation of an assign operation
[INFO] [stderr]    --> src/lib.rs:641:33
[INFO] [stderr]     |
[INFO] [stderr] 641 |                                 cx = cx + p.0;
[INFO] [stderr]     |                                 ^^^^^^^^^^^^^ help: replace it with: `cx += p.0`
[INFO] [stderr]     |
[INFO] [stderr]     = note: #[warn(clippy::assign_op_pattern)] on by default
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern
[INFO] [stderr] 
[INFO] [stderr] warning: manual implementation of an assign operation
[INFO] [stderr]    --> src/lib.rs:642:33
[INFO] [stderr]     |
[INFO] [stderr] 642 |                                 cy = cy + p.1;
[INFO] [stderr]     |                                 ^^^^^^^^^^^^^ help: replace it with: `cy += p.1`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern
[INFO] [stderr] 
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr]    --> src/lib.rs:673:33
[INFO] [stderr]     |
[INFO] [stderr] 673 |                         assert!(points.len() > 0);
[INFO] [stderr]     |                                 ^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!points.is_empty()`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stderr] 
[INFO] [stderr] warning: this .into_iter() call is equivalent to .iter() and will not move the Vec
[INFO] [stderr]    --> src/lib.rs:675:49
[INFO] [stderr]     |
[INFO] [stderr] 675 |                         for &(dx, dy) in points.into_iter() {
[INFO] [stderr]     |                                                 ^^^^^^^^^ help: call directly: `iter`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#into_iter_on_ref
[INFO] [stderr] 
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr]    --> src/lib.rs:687:25
[INFO] [stderr]     |
[INFO] [stderr] 687 |                 assert!(lines.len() > 0);
[INFO] [stderr]     |                         ^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!lines.is_empty()`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stderr] 
[INFO] [stderr] warning: this .into_iter() call is equivalent to .iter() and will not move the Vec
[INFO] [stderr]    --> src/lib.rs:716:49
[INFO] [stderr]     |
[INFO] [stderr] 716 |                         for &(dx, dy) in points.into_iter() {
[INFO] [stderr]     |                                                 ^^^^^^^^^ help: call directly: `iter`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#into_iter_on_ref
[INFO] [stderr] 
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr]    --> src/lib.rs:734:25
[INFO] [stderr]     |
[INFO] [stderr] 734 |                 assert!(rings.len() > 0);
[INFO] [stderr]     |                         ^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!rings.is_empty()`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stderr] 
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr]    --> src/lib.rs:737:24
[INFO] [stderr]     |
[INFO] [stderr] 737 |                     if rings.len() == 0 {
[INFO] [stderr]     |                        ^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `rings.is_empty()`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stderr] 
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr]    --> src/lib.rs:742:24
[INFO] [stderr]     |
[INFO] [stderr] 742 |                     if rings.len() > 0 {
[INFO] [stderr]     |                        ^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!rings.is_empty()`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stderr] 
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr]    --> src/lib.rs:744:32
[INFO] [stderr]     |
[INFO] [stderr] 744 |                             if rings.len() == 0 || rings[0].1 == Some(WindingOrder::Clockwise)  {
[INFO] [stderr]     |                                ^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `rings.is_empty()`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stderr] 
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr]    --> src/lib.rs:747:37
[INFO] [stderr]     |
[INFO] [stderr] 747 |                             assert!(rings.len() > 0);
[INFO] [stderr]     |                                     ^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!rings.is_empty()`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stderr] 
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr]    --> src/lib.rs:753:25
[INFO] [stderr]     |
[INFO] [stderr] 753 |                 assert!(polygons.len() > 0);
[INFO] [stderr]     |                         ^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!polygons.is_empty()`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stderr] 
[INFO] [stderr] warning: this argument is passed by reference, but would be more efficient if passed by value
[INFO] [stderr]    --> src/lib.rs:767:49
[INFO] [stderr]     |
[INFO] [stderr] 767 | fn move_cursor(current: &mut (i32, i32), point: &Point<i32>) -> (i32, i32) {
[INFO] [stderr]     |                                                 ^^^^^^^^^^^ help: consider passing by value instead: `Point<i32>`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#trivially_copy_pass_by_ref
[INFO] [stderr] 
[INFO] [stderr] warning: you don't need to add `&` to all patterns
[INFO] [stderr]    --> src/lib.rs:934:9
[INFO] [stderr]     |
[INFO] [stderr] 934 | /         match g {
[INFO] [stderr] 935 | |             &Geometry::Point(ref x) => x.into(),
[INFO] [stderr] 936 | |             &Geometry::LineString(ref x) => x.into(),
[INFO] [stderr] 937 | |             &Geometry::Polygon(ref x) => x.into(),
[INFO] [stderr] ...   |
[INFO] [stderr] 941 | |             _ => unimplemented!(),
[INFO] [stderr] 942 | |         }
[INFO] [stderr]     | |_________^
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#match_ref_pats
[INFO] [stderr] help: instead of prefixing all patterns with `&`, you can dereference the expression
[INFO] [stderr]     |
[INFO] [stderr] 934 |         match *g {
[INFO] [stderr] 935 |             Geometry::Point(ref x) => x.into(),
[INFO] [stderr] 936 |             Geometry::LineString(ref x) => x.into(),
[INFO] [stderr] 937 |             Geometry::Polygon(ref x) => x.into(),
[INFO] [stderr] 938 |             Geometry::MultiPoint(ref x) => x.into(),
[INFO] [stderr] 939 |             Geometry::MultiLineString(ref x) => x.into(),
[INFO] [stderr]   ...
[INFO] [stderr] 
[INFO] [stderr] warning: you should consider deriving a `Default` implementation for `Properties`
[INFO] [stderr]   --> src/lib.rs:51:5
[INFO] [stderr]    |
[INFO] [stderr] 51 | /     pub fn new() -> Self {
[INFO] [stderr] 52 | |         Properties(HashMap::new())
[INFO] [stderr] 53 | |     }
[INFO] [stderr]    | |_____^
[INFO] [stderr]    |
[INFO] [stderr]    = note: #[warn(clippy::new_without_default_derive)] on by default
[INFO] [stderr]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#new_without_default_derive
[INFO] [stderr] help: try this
[INFO] [stderr]    |
[INFO] [stderr] 35 | #[derive(Default)]
[INFO] [stderr]    |
[INFO] [stderr] 
[INFO] [stderr] warning: explicit lifetimes given in parameter types where they could be elided (or replaced with `'_` if needed by type declaration)
[INFO] [stderr]   --> src/lib.rs:93:5
[INFO] [stderr]    |
[INFO] [stderr] 93 | /     pub fn get_point<'a>(&'a self) -> Option<Point<i32>> {
[INFO] [stderr] 94 | |         match self.geometry {
[INFO] [stderr] 95 | |             Geometry::Point(p) => Some(p),
[INFO] [stderr] 96 | |             _ => None,
[INFO] [stderr] 97 | |         }
[INFO] [stderr] 98 | |     }
[INFO] [stderr]    | |_____^
[INFO] [stderr]    |
[INFO] [stderr]    = note: #[warn(clippy::needless_lifetimes)] on by default
[INFO] [stderr]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_lifetimes
[INFO] [stderr] 
[INFO] [stderr] warning: methods called `to_*` usually take self by reference; consider choosing a less ambiguous name
[INFO] [stderr]    --> src/lib.rs:181:21
[INFO] [stderr]     |
[INFO] [stderr] 181 |     pub fn to_bytes(self) -> Vec<u8> {
[INFO] [stderr]     |                     ^^^^
[INFO] [stderr]     |
[INFO] [stderr]     = note: #[warn(clippy::wrong_self_convention)] on by default
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#wrong_self_convention
[INFO] [stderr] 
[INFO] [stderr] warning: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices.
[INFO] [stderr]    --> src/lib.rs:252:29
[INFO] [stderr]     |
[INFO] [stderr] 252 | fn create_indexes(features: &Vec<Feature>) -> (TagIndex<String>, TagIndex<Value>) {
[INFO] [stderr]     |                             ^^^^^^^^^^^^^ help: change this to: `&[Feature]`
[INFO] [stderr]     |
[INFO] [stderr]     = note: #[warn(clippy::ptr_arg)] on by default
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg
[INFO] [stderr] 
[INFO] [stderr] warning: you should consider deriving a `Default` implementation for `Tile`
[INFO] [stderr]    --> src/lib.rs:321:5
[INFO] [stderr]     |
[INFO] [stderr] 321 | /     pub fn new() -> Self {
[INFO] [stderr] 322 | |         Tile{ layers: Vec::new() }
[INFO] [stderr] 323 | |     }
[INFO] [stderr]     | |_____^
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#new_without_default_derive
[INFO] [stderr] help: try this
[INFO] [stderr]     |
[INFO] [stderr] 316 | #[derive(Default)]
[INFO] [stderr]     |
[INFO] [stderr] 
[INFO] [stderr] warning: methods called `to_*` usually take self by reference; consider choosing a less ambiguous name
[INFO] [stderr]    --> src/lib.rs:366:21
[INFO] [stderr]     |
[INFO] [stderr] 366 |     pub fn to_bytes(self) -> Vec<u8> {
[INFO] [stderr]     |                     ^^^^
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#wrong_self_convention
[INFO] [stderr] 
[INFO] [stderr] warning: methods called `to_*` usually take self by reference; consider choosing a less ambiguous name
[INFO] [stderr]    --> src/lib.rs:372:32
[INFO] [stderr]     |
[INFO] [stderr] 372 |     pub fn to_compressed_bytes(self) -> Vec<u8> {
[INFO] [stderr]     |                                ^^^^
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#wrong_self_convention
[INFO] [stderr] 
[INFO] [stderr] warning: redundant closure found
[INFO] [stderr]    --> src/lib.rs:489:54
[INFO] [stderr]     |
[INFO] [stderr] 489 |     Tile{ layers: tile.take_layers().into_iter().map(|l| pbflayer_to_layer(l)).collect() }
[INFO] [stderr]     |                                                      ^^^^^^^^^^^^^^^^^^^^^^^^ help: remove closure as shown: `pbflayer_to_layer`
[INFO] [stderr]     |
[INFO] [stderr]     = note: #[warn(clippy::redundant_closure)] on by default
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure
[INFO] [stderr] 
[INFO] [stderr] warning: you don't need to add `&` to all patterns
[INFO] [stderr]    --> src/lib.rs:521:9
[INFO] [stderr]     |
[INFO] [stderr] 521 | /         if let &DrawingCommand::MoveTo(_) = self {
[INFO] [stderr] 522 | |             true
[INFO] [stderr] 523 | |         } else {
[INFO] [stderr] 524 | |             false
[INFO] [stderr] 525 | |         }
[INFO] [stderr]     | |_________^
[INFO] [stderr]     |
[INFO] [stderr]     = note: #[warn(clippy::match_ref_pats)] on by default
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#match_ref_pats
[INFO] [stderr] help: instead of prefixing all patterns with `&`, you can dereference the expression
[INFO] [stderr]     |
[INFO] [stderr] 521 |         if let DrawingCommand::MoveTo(_) = *self {
[INFO] [stderr]     |                ^^^^^^^^^^^^^^^^^^^^^^^^^   ^^^^^
[INFO] [stderr] 
[INFO] [stderr] warning: you don't need to add `&` to all patterns
[INFO] [stderr]    --> src/lib.rs:529:9
[INFO] [stderr]     |
[INFO] [stderr] 529 | /         if let &DrawingCommand::LineTo(_) = self {
[INFO] [stderr] 530 | |             true
[INFO] [stderr] 531 | |         } else {
[INFO] [stderr] 532 | |             false
[INFO] [stderr] 533 | |         }
[INFO] [stderr]     | |_________^
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#match_ref_pats
[INFO] [stderr] help: instead of prefixing all patterns with `&`, you can dereference the expression
[INFO] [stderr]     |
[INFO] [stderr] 529 |         if let DrawingCommand::LineTo(_) = *self {
[INFO] [stderr]     |                ^^^^^^^^^^^^^^^^^^^^^^^^^   ^^^^^
[INFO] [stderr] 
[INFO] [stderr] warning: you don't need to add `&` to all patterns
[INFO] [stderr]    --> src/lib.rs:537:9
[INFO] [stderr]     |
[INFO] [stderr] 537 | /         if let &DrawingCommand::ClosePath = self {
[INFO] [stderr] 538 | |             true
[INFO] [stderr] 539 | |         } else {
[INFO] [stderr] 540 | |             false
[INFO] [stderr] 541 | |         }
[INFO] [stderr]     | |_________^
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#match_ref_pats
[INFO] [stderr] help: instead of prefixing all patterns with `&`, you can dereference the expression
[INFO] [stderr]     |
[INFO] [stderr] 537 |         if let DrawingCommand::ClosePath = *self {
[INFO] [stderr]     |                ^^^^^^^^^^^^^^^^^^^^^^^^^   ^^^^^
[INFO] [stderr] 
[INFO] [stderr] warning: this argument is passed by reference, but would be more efficient if passed by value
[INFO] [stderr]    --> src/lib.rs:599:41
[INFO] [stderr]     |
[INFO] [stderr] 599 | fn decode_geom(data: &[u32], geom_type: &vector_tile::Tile_GeomType) -> Geometry<i32> {
[INFO] [stderr]     |                                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider passing by value instead: `vector_tile::Tile_GeomType`
[INFO] [stderr]     |
[INFO] [stderr]     = note: #[warn(clippy::trivially_copy_pass_by_ref)] on by default
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#trivially_copy_pass_by_ref
[INFO] [stderr] 
[INFO] [stderr] warning: the function has a cyclomatic complexity of 39
[INFO] [stderr]    --> src/lib.rs:622:5
[INFO] [stderr]     |
[INFO] [stderr] 622 | /     fn from(commands: DrawingCommands) -> Geometry<i32> {
[INFO] [stderr] 623 | |         let geom_type = deduce_geom_type(&commands);
[INFO] [stderr] 624 | |
[INFO] [stderr] 625 | |         match geom_type {
[INFO] [stderr] ...   |
[INFO] [stderr] 763 | |         }
[INFO] [stderr] 764 | |     }
[INFO] [stderr]     | |_____^
[INFO] [stderr]     |
[INFO] [stderr]     = note: #[warn(clippy::cyclomatic_complexity)] on by default
[INFO] [stderr]     = help: you could split it up into multiple smaller functions
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cyclomatic_complexity
[INFO] [stderr] 
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr]    --> src/lib.rs:632:28
[INFO] [stderr]     |
[INFO] [stderr] 632 |                         if points.len() == 0 {
[INFO] [stderr]     |                            ^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `points.is_empty()`
[INFO] [stderr]     |
[INFO] [stderr]     = note: #[warn(clippy::len_zero)] on by default
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stderr] 
[INFO] [stderr] warning: this .into_iter() call is equivalent to .iter() and will not move the Vec
[INFO] [stderr]    --> src/lib.rs:640:45
[INFO] [stderr]     |
[INFO] [stderr] 640 |                             for p in points.into_iter() {
[INFO] [stderr]     |                                             ^^^^^^^^^ help: call directly: `iter`
[INFO] [stderr]     |
[INFO] [stderr]     = note: #[warn(clippy::into_iter_on_ref)] on by default
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#into_iter_on_ref
[INFO] [stderr] 
[INFO] [stderr] warning: manual implementation of an assign operation
[INFO] [stderr]    --> src/lib.rs:641:33
[INFO] [stderr]     |
[INFO] [stderr] 641 |                                 cx = cx + p.0;
[INFO] [stderr]     |                                 ^^^^^^^^^^^^^ help: replace it with: `cx += p.0`
[INFO] [stderr]     |
[INFO] [stderr]     = note: #[warn(clippy::assign_op_pattern)] on by default
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern
[INFO] [stderr] 
[INFO] [stderr] warning: manual implementation of an assign operation
[INFO] [stderr]    --> src/lib.rs:642:33
[INFO] [stderr]     |
[INFO] [stderr] 642 |                                 cy = cy + p.1;
[INFO] [stderr]     |                                 ^^^^^^^^^^^^^ help: replace it with: `cy += p.1`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern
[INFO] [stderr] 
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr]    --> src/lib.rs:673:33
[INFO] [stderr]     |
[INFO] [stderr] 673 |                         assert!(points.len() > 0);
[INFO] [stderr]     |                                 ^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!points.is_empty()`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stderr] 
[INFO] [stderr] warning: this .into_iter() call is equivalent to .iter() and will not move the Vec
[INFO] [stderr]    --> src/lib.rs:675:49
[INFO] [stderr]     |
[INFO] [stderr] 675 |                         for &(dx, dy) in points.into_iter() {
[INFO] [stderr]     |                                                 ^^^^^^^^^ help: call directly: `iter`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#into_iter_on_ref
[INFO] [stderr] 
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr]    --> src/lib.rs:687:25
[INFO] [stderr]     |
[INFO] [stderr] 687 |                 assert!(lines.len() > 0);
[INFO] [stderr]     |                         ^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!lines.is_empty()`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stderr] 
[INFO] [stderr] warning: this .into_iter() call is equivalent to .iter() and will not move the Vec
[INFO] [stderr]    --> src/lib.rs:716:49
[INFO] [stderr]     |
[INFO] [stderr] 716 |                         for &(dx, dy) in points.into_iter() {
[INFO] [stderr]     |                                                 ^^^^^^^^^ help: call directly: `iter`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#into_iter_on_ref
[INFO] [stderr] 
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr]    --> src/lib.rs:734:25
[INFO] [stderr]     |
[INFO] [stderr] 734 |                 assert!(rings.len() > 0);
[INFO] [stderr]     |                         ^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!rings.is_empty()`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stderr] 
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr]    --> src/lib.rs:737:24
[INFO] [stderr]     |
[INFO] [stderr] 737 |                     if rings.len() == 0 {
[INFO] [stderr]     |                        ^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `rings.is_empty()`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stderr] 
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr]    --> src/lib.rs:742:24
[INFO] [stderr]     |
[INFO] [stderr] 742 |                     if rings.len() > 0 {
[INFO] [stderr]     |                        ^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!rings.is_empty()`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stderr] 
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr]    --> src/lib.rs:744:32
[INFO] [stderr]     |
[INFO] [stderr] 744 |                             if rings.len() == 0 || rings[0].1 == Some(WindingOrder::Clockwise)  {
[INFO] [stderr]     |                                ^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `rings.is_empty()`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stderr] 
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr]    --> src/lib.rs:747:37
[INFO] [stderr]     |
[INFO] [stderr] 747 |                             assert!(rings.len() > 0);
[INFO] [stderr]     |                                     ^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!rings.is_empty()`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stderr] 
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr]    --> src/lib.rs:753:25
[INFO] [stderr]     |
[INFO] [stderr] 753 |                 assert!(polygons.len() > 0);
[INFO] [stderr]     |                         ^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!polygons.is_empty()`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stderr] 
[INFO] [stderr] warning: this argument is passed by reference, but would be more efficient if passed by value
[INFO] [stderr]    --> src/lib.rs:767:49
[INFO] [stderr]     |
[INFO] [stderr] 767 | fn move_cursor(current: &mut (i32, i32), point: &Point<i32>) -> (i32, i32) {
[INFO] [stderr]     |                                                 ^^^^^^^^^^^ help: consider passing by value instead: `Point<i32>`
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#trivially_copy_pass_by_ref
[INFO] [stderr] 
[INFO] [stderr] warning: you don't need to add `&` to all patterns
[INFO] [stderr]    --> src/lib.rs:934:9
[INFO] [stderr]     |
[INFO] [stderr] 934 | /         match g {
[INFO] [stderr] 935 | |             &Geometry::Point(ref x) => x.into(),
[INFO] [stderr] 936 | |             &Geometry::LineString(ref x) => x.into(),
[INFO] [stderr] 937 | |             &Geometry::Polygon(ref x) => x.into(),
[INFO] [stderr] ...   |
[INFO] [stderr] 941 | |             _ => unimplemented!(),
[INFO] [stderr] 942 | |         }
[INFO] [stderr]     | |_________^
[INFO] [stderr]     |
[INFO] [stderr]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#match_ref_pats
[INFO] [stderr] help: instead of prefixing all patterns with `&`, you can dereference the expression
[INFO] [stderr]     |
[INFO] [stderr] 934 |         match *g {
[INFO] [stderr] 935 |             Geometry::Point(ref x) => x.into(),
[INFO] [stderr] 936 |             Geometry::LineString(ref x) => x.into(),
[INFO] [stderr] 937 |             Geometry::Polygon(ref x) => x.into(),
[INFO] [stderr] 938 |             Geometry::MultiPoint(ref x) => x.into(),
[INFO] [stderr] 939 |             Geometry::MultiLineString(ref x) => x.into(),
[INFO] [stderr]   ...
[INFO] [stderr] 
[INFO] [stderr]     Finished dev [unoptimized + debuginfo] target(s) in 10.07s
[INFO] running `"docker" "inspect" "46cdbee365ed862ee344126eb1cd4a0fd0b09d36c3df4027bba5737497b4a580"`
[INFO] running `"docker" "rm" "-f" "46cdbee365ed862ee344126eb1cd4a0fd0b09d36c3df4027bba5737497b4a580"`
[INFO] [stdout] 46cdbee365ed862ee344126eb1cd4a0fd0b09d36c3df4027bba5737497b4a580