[INFO] crate html2md 0.2.5 is already in cache
[INFO] extracting crate html2md 0.2.5 into work/ex/clippy-test-run/sources/stable/reg/html2md/0.2.5
[INFO] extracting crate html2md 0.2.5 into work/ex/clippy-test-run/sources/stable+rustflags=-Dclippy%3A%3Ainto_iter_on_array/reg/html2md/0.2.5
[INFO] validating manifest of html2md-0.2.5 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 html2md-0.2.5 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 html2md-0.2.5
[INFO] finished frobbing html2md-0.2.5
[INFO] frobbed toml for html2md-0.2.5 written to work/ex/clippy-test-run/sources/stable/reg/html2md/0.2.5/Cargo.toml
[INFO] started frobbing html2md-0.2.5
[INFO] finished frobbing html2md-0.2.5
[INFO] frobbed toml for html2md-0.2.5 written to work/ex/clippy-test-run/sources/stable+rustflags=-Dclippy%3A%3Ainto_iter_on_array/reg/html2md/0.2.5/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 html2md-0.2.5 against stable for clippy-test-run
[INFO] running `"docker" "create" "-v" "/mnt/big/crater/work/local/target-dirs/clippy-test-run/worker-3/stable:/opt/crater/target:rw,Z" "-v" "/mnt/big/crater/work/ex/clippy-test-run/sources/stable/reg/html2md/0.2.5:/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] d524c936e7edbd4f13aeb3bdf0ac2b82a374431d3beaa79fd25729cca15aa55f
[INFO] running `"docker" "start" "-a" "d524c936e7edbd4f13aeb3bdf0ac2b82a374431d3beaa79fd25729cca15aa55f"`
[INFO] [stderr] Checking html2md v0.2.5 (/opt/crater/workdir)
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/images.rs:53:13
[INFO] [stderr] |
[INFO] [stderr] 53 | return requested_attr.map(|attr| attr.value.to_string());
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove `return` as shown: `requested_attr.map(|attr| attr.value.to_string())`
[INFO] [stderr] |
[INFO] [stderr] = note: #[warn(clippy::needless_return)] on by default
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr]
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/images.rs:55:14
[INFO] [stderr] |
[INFO] [stderr] 55 | _ => return None
[INFO] [stderr] | ^^^^^^^^^^^ help: remove `return` as shown: `None`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr]
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/tables.rs:103:67
[INFO] [stderr] |
[INFO] [stderr] 103 | rows.retain(|row| { let children = row.children.borrow(); return children.iter().any(|child| tag_name(&child) == "td"); });
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove `return` as shown: `children.iter().any(|child| tag_name(&child) == "td")`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr]
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/tables.rs:124:9
[INFO] [stderr] |
[INFO] [stderr] 124 | return true;
[INFO] [stderr] | ^^^^^^^^^^^^ help: remove `return` as shown: `true`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr]
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/tables.rs:164:5
[INFO] [stderr] |
[INFO] [stderr] 164 | return result;
[INFO] [stderr] | ^^^^^^^^^^^^^^ help: remove `return` as shown: `result`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr]
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/tables.rs:170:5
[INFO] [stderr] |
[INFO] [stderr] 170 | / return match tag.data {
[INFO] [stderr] 171 | | NodeData::Element { ref name, .. } => name.local.to_string(),
[INFO] [stderr] 172 | | _ => String::new()
[INFO] [stderr] 173 | | }
[INFO] [stderr] | |_____^
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr] help: remove `return` as shown
[INFO] [stderr] |
[INFO] [stderr] 170 | match tag.data {
[INFO] [stderr] 171 | NodeData::Element { ref name, .. } => name.local.to_string(),
[INFO] [stderr] 172 | _ => String::new()
[INFO] [stderr] 173 | }
[INFO] [stderr] |
[INFO] [stderr]
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/tables.rs:190:5
[INFO] [stderr] |
[INFO] [stderr] 190 | return result;
[INFO] [stderr] | ^^^^^^^^^^^^^^ help: remove `return` as shown: `result`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr]
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/tables.rs:205:5
[INFO] [stderr] |
[INFO] [stderr] 205 | return result;
[INFO] [stderr] | ^^^^^^^^^^^^^^ help: remove `return` as shown: `result`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr]
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/tables.rs:222:5
[INFO] [stderr] |
[INFO] [stderr] 222 | return result;
[INFO] [stderr] | ^^^^^^^^^^^^^^ help: remove `return` as shown: `result`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr]
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/lib.rs:208:5
[INFO] [stderr] |
[INFO] [stderr] 208 | return data;
[INFO] [stderr] | ^^^^^^^^^^^^ help: remove `return` as shown: `data`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr]
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/lib.rs:258:9
[INFO] [stderr] |
[INFO] [stderr] 258 | return false;
[INFO] [stderr] | ^^^^^^^^^^^^^ help: remove `return` as shown: `false`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr]
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/images.rs:53:13
[INFO] [stderr] |
[INFO] [stderr] 53 | return requested_attr.map(|attr| attr.value.to_string());
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove `return` as shown: `requested_attr.map(|attr| attr.value.to_string())`
[INFO] [stderr] |
[INFO] [stderr] = note: #[warn(clippy::needless_return)] on by default
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr]
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/images.rs:55:14
[INFO] [stderr] |
[INFO] [stderr] 55 | _ => return None
[INFO] [stderr] | ^^^^^^^^^^^ help: remove `return` as shown: `None`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr]
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/tables.rs:103:67
[INFO] [stderr] |
[INFO] [stderr] 103 | rows.retain(|row| { let children = row.children.borrow(); return children.iter().any(|child| tag_name(&child) == "td"); });
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove `return` as shown: `children.iter().any(|child| tag_name(&child) == "td")`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr]
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/tables.rs:124:9
[INFO] [stderr] |
[INFO] [stderr] 124 | return true;
[INFO] [stderr] | ^^^^^^^^^^^^ help: remove `return` as shown: `true`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr]
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/tables.rs:164:5
[INFO] [stderr] |
[INFO] [stderr] 164 | return result;
[INFO] [stderr] | ^^^^^^^^^^^^^^ help: remove `return` as shown: `result`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr]
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/tables.rs:170:5
[INFO] [stderr] |
[INFO] [stderr] 170 | / return match tag.data {
[INFO] [stderr] 171 | | NodeData::Element { ref name, .. } => name.local.to_string(),
[INFO] [stderr] 172 | | _ => String::new()
[INFO] [stderr] 173 | | }
[INFO] [stderr] | |_____^
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr] help: remove `return` as shown
[INFO] [stderr] |
[INFO] [stderr] 170 | match tag.data {
[INFO] [stderr] 171 | NodeData::Element { ref name, .. } => name.local.to_string(),
[INFO] [stderr] 172 | _ => String::new()
[INFO] [stderr] 173 | }
[INFO] [stderr] |
[INFO] [stderr]
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/tables.rs:190:5
[INFO] [stderr] |
[INFO] [stderr] 190 | return result;
[INFO] [stderr] | ^^^^^^^^^^^^^^ help: remove `return` as shown: `result`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr]
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/tables.rs:205:5
[INFO] [stderr] |
[INFO] [stderr] 205 | return result;
[INFO] [stderr] | ^^^^^^^^^^^^^^ help: remove `return` as shown: `result`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr]
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/tables.rs:222:5
[INFO] [stderr] |
[INFO] [stderr] 222 | return result;
[INFO] [stderr] | ^^^^^^^^^^^^^^ help: remove `return` as shown: `result`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr]
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/lib.rs:208:5
[INFO] [stderr] |
[INFO] [stderr] 208 | return data;
[INFO] [stderr] | ^^^^^^^^^^^^ help: remove `return` as shown: `data`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr]
[INFO] [stderr] warning: unneeded return statement
[INFO] [stderr] --> src/lib.rs:258:9
[INFO] [stderr] |
[INFO] [stderr] 258 | return false;
[INFO] [stderr] | ^^^^^^^^^^^^^ help: remove `return` as shown: `false`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stderr]
[INFO] [stderr] warning: this creates an owned instance just for comparison
[INFO] [stderr] --> src/anchors.rs:16:53
[INFO] [stderr] |
[INFO] [stderr] 16 | let href = attrs.iter().find(|attr| attr.name.local.to_string() == "href");
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `attr.name.local`
[INFO] [stderr] |
[INFO] [stderr] = note: #[warn(clippy::cmp_owned)] on by default
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cmp_owned
[INFO] [stderr]
[INFO] [stderr] warning: this creates an owned instance just for comparison
[INFO] [stderr] --> src/images.rs:52:59
[INFO] [stderr] |
[INFO] [stderr] 52 | let requested_attr = attrs.iter().find(|attr| attr.name.local.to_string() == attr_name);
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `attr.name.local`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cmp_owned
[INFO] [stderr]
[INFO] [stderr] warning: you should use the `ends_with` method
[INFO] [stderr] --> src/lists.rs:45:12
[INFO] [stderr] |
[INFO] [stderr] 45 | if printer.data.trim_matches(' ').chars().last() != Some('\n') {
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: like this: `!printer.data.trim_matches(' ').ends_with('\n')`
[INFO] [stderr] |
[INFO] [stderr] = note: #[warn(clippy::chars_last_cmp)] on by default
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#chars_last_cmp
[INFO] [stderr]
[INFO] [stderr] warning: the loop variable `index` is used to index `column_widths`
[INFO] [stderr] --> src/tables.rs:38:30
[INFO] [stderr] |
[INFO] [stderr] 38 | for index in 0..column_count {
[INFO] [stderr] | ^^^^^^^^^^^^^^^
[INFO] [stderr] |
[INFO] [stderr] = note: #[warn(clippy::needless_range_loop)] on by default
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop
[INFO] [stderr] help: consider using an iterator
[INFO] [stderr] |
[INFO] [stderr] 38 | for (index, - ) in column_widths.iter_mut().enumerate().take(column_count) {
[INFO] [stderr] | ^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr]
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr] --> src/tables.rs:51:52
[INFO] [stderr] |
[INFO] [stderr] 51 | let header_tr = rows.iter().find(|row| collect_children(&row, th_matcher).len() > 0);
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!collect_children(&row, th_matcher).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: the loop variable `index` is used to index `column_widths`
[INFO] [stderr] --> src/tables.rs:56:30
[INFO] [stderr] |
[INFO] [stderr] 56 | for index in 0..column_count {
[INFO] [stderr] | ^^^^^^^^^^^^^^^
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop
[INFO] [stderr] help: consider using an iterator
[INFO] [stderr] |
[INFO] [stderr] 56 | for (index,
- ) in column_widths.iter().enumerate().take(column_count) {
[INFO] [stderr] | ^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr]
[INFO] [stderr] warning: the loop variable `index` is used to index `column_widths`
[INFO] [stderr] --> src/tables.rs:66:26
[INFO] [stderr] |
[INFO] [stderr] 66 | for index in 0..column_count {
[INFO] [stderr] | ^^^^^^^^^^^^^^^
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop
[INFO] [stderr] help: consider using an iterator
[INFO] [stderr] |
[INFO] [stderr] 66 | for (index,
- ) in column_widths.iter().enumerate().take(column_count) {
[INFO] [stderr] | ^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr]
[INFO] [stderr] warning: this creates an owned instance just for comparison
[INFO] [stderr] --> src/tables.rs:82:71
[INFO] [stderr] |
[INFO] [stderr] 82 | let align_attr = attrs.iter().find(|attr| attr.name.local.to_string() == "align");
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `attr.name.local`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cmp_owned
[INFO] [stderr]
[INFO] [stderr] warning: the loop variable `index` is used to index `column_widths`
[INFO] [stderr] --> src/tables.rs:107:26
[INFO] [stderr] |
[INFO] [stderr] 107 | for index in 0..column_count { // we need to fill all cells in a column, even if some rows don't have enough
[INFO] [stderr] | ^^^^^^^^^^^^^^^
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop
[INFO] [stderr] help: consider using an iterator
[INFO] [stderr] |
[INFO] [stderr] 107 | for (index,
- ) in column_widths.iter().enumerate().take(column_count) { // we need to fill all cells in a column, even if some rows don't have enough
[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/tables.rs:133:23
[INFO] [stderr] |
[INFO] [stderr] 133 | fn pad_cell_text(tag: &Option<&Handle>, column_width: usize) -> String {
[INFO] [stderr] | ^^^^^^^^^^^^^^^^ help: consider passing by value instead: `Option<&Handle>`
[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: you seem to be trying to use match for destructuring a single pattern. Consider using `if let`
[INFO] [stderr] --> src/tables.rs:212:5
[INFO] [stderr] |
[INFO] [stderr] 212 | / match tag.data {
[INFO] [stderr] 213 | | NodeData::Text { ref contents } => result.push_str(&contents.borrow().trim()),
[INFO] [stderr] 214 | | _ => {}
[INFO] [stderr] 215 | | }
[INFO] [stderr] | |_____^ help: try this: `if let NodeData::Text { ref contents } = tag.data { result.push_str(&contents.borrow().trim()) }`
[INFO] [stderr] |
[INFO] [stderr] warning: this creates an owned instance just for comparison
[INFO] [stderr] --> src/anchors.rs:16:53
[INFO] [stderr] |
[INFO] [stderr] 16 | let href = attrs.iter().find(|attr| attr.name.local.to_string() == "href");
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `attr.name.local`
[INFO] [stderr] |
[INFO] [stderr] = note: #[warn(clippy::cmp_owned)] on by default
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cmp_owned
[INFO] [stderr]
[INFO] [stderr] = note: #[warn(clippy::single_match)] on by default
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match
[INFO] [stderr]
[INFO] [stderr] warning: this creates an owned instance just for comparison
[INFO] [stderr] --> src/images.rs:52:59
[INFO] [stderr] |
[INFO] [stderr] 52 | let requested_attr = attrs.iter().find(|attr| attr.name.local.to_string() == attr_name);
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `attr.name.local`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cmp_owned
[INFO] [stderr]
[INFO] [stderr] error: this public function dereferences a raw pointer but is not marked `unsafe`
[INFO] [stderr] --> src/lib.rs:60:43
[INFO] [stderr] |
[INFO] [stderr] 60 | let in_html = unsafe { CStr::from_ptr(html) };
[INFO] [stderr] | ^^^^
[INFO] [stderr] |
[INFO] [stderr] = note: #[deny(clippy::not_unsafe_ptr_arg_deref)] on by default
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#not_unsafe_ptr_arg_deref
[INFO] [stderr]
[INFO] [stderr] warning: parameter of type `HashMap` should be generalized over different hashers
[INFO] [stderr] --> src/lib.rs:73:47
[INFO] [stderr] |
[INFO] [stderr] 73 | pub fn parse_html_custom(html: &str, custom: &HashMap>) -> String {
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr] |
[INFO] [stderr] = note: #[warn(clippy::implicit_hasher)] on by default
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#implicit_hasher
[INFO] [stderr] help: consider adding a type parameter
[INFO] [stderr] |
[INFO] [stderr] 73 | pub fn parse_html_custom(html: &str, custom: &HashMap, S>) -> String {
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr]
[INFO] [stderr] warning: you should use the `ends_with` method
[INFO] [stderr] --> src/lists.rs:45:12
[INFO] [stderr] |
[INFO] [stderr] 45 | if printer.data.trim_matches(' ').chars().last() != Some('\n') {
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: like this: `!printer.data.trim_matches(' ').ends_with('\n')`
[INFO] [stderr] |
[INFO] [stderr] = note: #[warn(clippy::chars_last_cmp)] on by default
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#chars_last_cmp
[INFO] [stderr]
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr] --> src/lib.rs:113:25
[INFO] [stderr] |
[INFO] [stderr] 113 | } else if !(text.trim().len() == 0 && (result.data.chars().last() == Some('\n') || result.data.chars().last() == Some(' '))) {
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `text.trim().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: you should use the `ends_with` method
[INFO] [stderr] --> src/lib.rs:113:52
[INFO] [stderr] |
[INFO] [stderr] 113 | } else if !(text.trim().len() == 0 && (result.data.chars().last() == Some('\n') || result.data.chars().last() == Some(' '))) {
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: like this: `result.data.ends_with('\n')`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#chars_last_cmp
[INFO] [stderr]
[INFO] [stderr] warning: you should use the `ends_with` method
[INFO] [stderr] --> src/lib.rs:113:96
[INFO] [stderr] |
[INFO] [stderr] 113 | } else if !(text.trim().len() == 0 && (result.data.chars().last() == Some('\n') || result.data.chars().last() == Some(' '))) {
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: like this: `result.data.ends_with(' ')`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#chars_last_cmp
[INFO] [stderr]
[INFO] [stderr] warning: the loop variable `index` is used to index `column_widths`
[INFO] [stderr] --> src/tables.rs:38:30
[INFO] [stderr] |
[INFO] [stderr] 38 | for index in 0..column_count {
[INFO] [stderr] | ^^^^^^^^^^^^^^^
[INFO] [stderr] warning: you seem to be trying to use match for destructuring a single pattern. Consider using `if let`
[INFO] [stderr] --> src/lib.rs:184:9
[INFO] [stderr] |
[INFO] [stderr] 184 | / match child.data {
[INFO] [stderr] 185 | | NodeData::Element { ref name, .. } => result.siblings.get_mut(¤t_depth).unwrap().push(name.local.to_string()),
[INFO] [stderr] 186 | | _ => {}
[INFO] [stderr] 187 | | };
[INFO] [stderr] | |_________^ help: try this: `if let NodeData::Element { ref name, .. } = child.data { result.siblings.get_mut(¤t_depth).unwrap().push(name.local.to_string()) }`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match
[INFO] [stderr]
[INFO] [stderr] |
[INFO] [stderr] = note: #[warn(clippy::needless_range_loop)] on by default
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop
[INFO] [stderr] help: consider using an iterator
[INFO] [stderr] |
[INFO] [stderr] 38 | for (index,
- ) in column_widths.iter_mut().enumerate().take(column_count) {
[INFO] [stderr] | ^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr]
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr] --> src/tables.rs:51:52
[INFO] [stderr] |
[INFO] [stderr] 51 | let header_tr = rows.iter().find(|row| collect_children(&row, th_matcher).len() > 0);
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!collect_children(&row, th_matcher).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: the loop variable `index` is used to index `column_widths`
[INFO] [stderr] --> src/tables.rs:56:30
[INFO] [stderr] |
[INFO] [stderr] 56 | for index in 0..column_count {
[INFO] [stderr] | ^^^^^^^^^^^^^^^
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop
[INFO] [stderr] help: consider using an iterator
[INFO] [stderr] |
[INFO] [stderr] 56 | for (index,
- ) in column_widths.iter().enumerate().take(column_count) {
[INFO] [stderr] | ^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr]
[INFO] [stderr] warning: the loop variable `index` is used to index `column_widths`
[INFO] [stderr] --> src/tables.rs:66:26
[INFO] [stderr] |
[INFO] [stderr] 66 | for index in 0..column_count {
[INFO] [stderr] | ^^^^^^^^^^^^^^^
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop
[INFO] [stderr] help: consider using an iterator
[INFO] [stderr] |
[INFO] [stderr] 66 | for (index,
- ) in column_widths.iter().enumerate().take(column_count) {
[INFO] [stderr] | ^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr]
[INFO] [stderr] warning: this creates an owned instance just for comparison
[INFO] [stderr] --> src/tables.rs:82:71
[INFO] [stderr] |
[INFO] [stderr] 82 | let align_attr = attrs.iter().find(|attr| attr.name.local.to_string() == "align");
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `attr.name.local`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cmp_owned
[INFO] [stderr]
[INFO] [stderr] warning: the loop variable `index` is used to index `column_widths`
[INFO] [stderr] --> src/tables.rs:107:26
[INFO] [stderr] |
[INFO] [stderr] 107 | for index in 0..column_count { // we need to fill all cells in a column, even if some rows don't have enough
[INFO] [stderr] | ^^^^^^^^^^^^^^^
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop
[INFO] [stderr] help: consider using an iterator
[INFO] [stderr] |
[INFO] [stderr] 107 | for (index,
- ) in column_widths.iter().enumerate().take(column_count) { // we need to fill all cells in a column, even if some rows don't have enough
[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/tables.rs:133:23
[INFO] [stderr] |
[INFO] [stderr] 133 | fn pad_cell_text(tag: &Option<&Handle>, column_width: usize) -> String {
[INFO] [stderr] | ^^^^^^^^^^^^^^^^ help: consider passing by value instead: `Option<&Handle>`
[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] error: aborting due to previous error
[INFO] [stderr]
[INFO] [stderr] warning: you seem to be trying to use match for destructuring a single pattern. Consider using `if let`
[INFO] [stderr] --> src/tables.rs:212:5
[INFO] [stderr] |
[INFO] [stderr] 212 | / match tag.data {
[INFO] [stderr] 213 | | NodeData::Text { ref contents } => result.push_str(&contents.borrow().trim()),
[INFO] [stderr] 214 | | _ => {}
[INFO] [stderr] 215 | | }
[INFO] [stderr] | |_____^ help: try this: `if let NodeData::Text { ref contents } = tag.data { result.push_str(&contents.borrow().trim()) }`
[INFO] [stderr] |
[INFO] [stderr] = note: #[warn(clippy::single_match)] on by default
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match
[INFO] [stderr]
[INFO] [stderr] error: this public function dereferences a raw pointer but is not marked `unsafe`
[INFO] [stderr] --> src/lib.rs:60:43
[INFO] [stderr] |
[INFO] [stderr] 60 | let in_html = unsafe { CStr::from_ptr(html) };
[INFO] [stderr] | ^^^^
[INFO] [stderr] |
[INFO] [stderr] = note: #[deny(clippy::not_unsafe_ptr_arg_deref)] on by default
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#not_unsafe_ptr_arg_deref
[INFO] [stderr]
[INFO] [stderr] warning: parameter of type `HashMap` should be generalized over different hashers
[INFO] [stderr] --> src/lib.rs:73:47
[INFO] [stderr] |
[INFO] [stderr] 73 | pub fn parse_html_custom(html: &str, custom: &HashMap>) -> String {
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr] |
[INFO] [stderr] = note: #[warn(clippy::implicit_hasher)] on by default
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#implicit_hasher
[INFO] [stderr] help: consider adding a type parameter
[INFO] [stderr] |
[INFO] [stderr] 73 | pub fn parse_html_custom(html: &str, custom: &HashMap, S>) -> String {
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr]
[INFO] [stderr] error: Could not compile `html2md`.
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stderr] warning: length comparison to zero
[INFO] [stderr] --> src/lib.rs:113:25
[INFO] [stderr] |
[INFO] [stderr] 113 | } else if !(text.trim().len() == 0 && (result.data.chars().last() == Some('\n') || result.data.chars().last() == Some(' '))) {
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `text.trim().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: you should use the `ends_with` method
[INFO] [stderr] --> src/lib.rs:113:52
[INFO] [stderr] |
[INFO] [stderr] 113 | } else if !(text.trim().len() == 0 && (result.data.chars().last() == Some('\n') || result.data.chars().last() == Some(' '))) {
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: like this: `result.data.ends_with('\n')`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#chars_last_cmp
[INFO] [stderr]
[INFO] [stderr] warning: you should use the `ends_with` method
[INFO] [stderr] --> src/lib.rs:113:96
[INFO] [stderr] |
[INFO] [stderr] 113 | } else if !(text.trim().len() == 0 && (result.data.chars().last() == Some('\n') || result.data.chars().last() == Some(' '))) {
[INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: like this: `result.data.ends_with(' ')`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#chars_last_cmp
[INFO] [stderr]
[INFO] [stderr] warning: you seem to be trying to use match for destructuring a single pattern. Consider using `if let`
[INFO] [stderr] --> src/lib.rs:184:9
[INFO] [stderr] |
[INFO] [stderr] 184 | / match child.data {
[INFO] [stderr] 185 | | NodeData::Element { ref name, .. } => result.siblings.get_mut(¤t_depth).unwrap().push(name.local.to_string()),
[INFO] [stderr] 186 | | _ => {}
[INFO] [stderr] 187 | | };
[INFO] [stderr] | |_________^ help: try this: `if let NodeData::Element { ref name, .. } = child.data { result.siblings.get_mut(¤t_depth).unwrap().push(name.local.to_string()) }`
[INFO] [stderr] |
[INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match
[INFO] [stderr]
[INFO] [stderr] error: aborting due to previous error
[INFO] [stderr]
[INFO] [stderr] error: Could not compile `html2md`.
[INFO] [stderr]
[INFO] [stderr] To learn more, run the command again with --verbose.
[INFO] running `"docker" "inspect" "d524c936e7edbd4f13aeb3bdf0ac2b82a374431d3beaa79fd25729cca15aa55f"`
[INFO] running `"docker" "rm" "-f" "d524c936e7edbd4f13aeb3bdf0ac2b82a374431d3beaa79fd25729cca15aa55f"`
[INFO] [stdout] d524c936e7edbd4f13aeb3bdf0ac2b82a374431d3beaa79fd25729cca15aa55f