Skip to content

Merge pull request #2253 from quickwit-oss/issue/2251-bug-merge-json-… #729

Merge pull request #2253 from quickwit-oss/issue/2251-bug-merge-json-…

Merge pull request #2253 from quickwit-oss/issue/2251-bug-merge-json-… #729

GitHub Actions / clippy succeeded Nov 14, 2023 in 0s

clippy

5 warnings

Details

Results

Message level Amount
Internal compiler error 0
Error 0
Warning 5
Note 0
Help 0

Versions

  • rustc 1.73.0 (cc66ad468 2023-10-03)
  • cargo 1.73.0 (9c4383fb5 2023-08-26)
  • clippy 0.1.73 (cc66ad4 2023-10-03)

Annotations

Check warning on line 750 in src/snippet/mod.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

an array of `Range` that is only one element

warning: an array of `Range` that is only one element
   --> src/snippet/mod.rs:750:66
    |
750 |         assert_eq!(&collapse_overlapped_ranges(&[0..3, 1..2,]), &[0..3]);
    |                                                                  ^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_range_in_vec_init
help: if you wanted a `Vec` that contains the entire range, try
    |
750 |         assert_eq!(&collapse_overlapped_ranges(&[0..3, 1..2,]), &(0..3).collect::<std::vec::Vec<usize>>());
    |                                                                  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
help: if you wanted an array of len 3, try
    |
750 |         assert_eq!(&collapse_overlapped_ranges(&[0..3, 1..2,]), &[0; 3]);
    |                                                                   ~~~~

Check warning on line 749 in src/snippet/mod.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

an array of `Range` that is only one element

warning: an array of `Range` that is only one element
   --> src/snippet/mod.rs:749:66
    |
749 |         assert_eq!(&collapse_overlapped_ranges(&[0..2, 1..3,]), &[0..3]);
    |                                                                  ^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_range_in_vec_init
help: if you wanted a `Vec` that contains the entire range, try
    |
749 |         assert_eq!(&collapse_overlapped_ranges(&[0..2, 1..3,]), &(0..3).collect::<std::vec::Vec<usize>>());
    |                                                                  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
help: if you wanted an array of len 3, try
    |
749 |         assert_eq!(&collapse_overlapped_ranges(&[0..2, 1..3,]), &[0; 3]);
    |                                                                   ~~~~

Check warning on line 748 in src/snippet/mod.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

an array of `Range` that is only one element

warning: an array of `Range` that is only one element
   --> src/snippet/mod.rs:748:66
    |
748 |         assert_eq!(&collapse_overlapped_ranges(&[0..2, 1..2,]), &[0..2]);
    |                                                                  ^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_range_in_vec_init
    = note: `#[warn(clippy::single_range_in_vec_init)]` on by default
help: if you wanted a `Vec` that contains the entire range, try
    |
748 |         assert_eq!(&collapse_overlapped_ranges(&[0..2, 1..2,]), &(0..2).collect::<std::vec::Vec<usize>>());
    |                                                                  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
help: if you wanted an array of len 2, try
    |
748 |         assert_eq!(&collapse_overlapped_ranges(&[0..2, 1..2,]), &[0; 2]);
    |                                                                   ~~~~

Check warning on line 465 in src/core/tests.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

this expression creates a reference which is immediately dereferenced by the compiler

warning: this expression creates a reference which is immediately dereferenced by the compiler
   --> src/core/tests.rs:465:13
    |
465 |             &json_term_writer.term(),
    |             ^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `json_term_writer.term()`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow

Check warning on line 427 in src/core/tests.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

this expression creates a reference which is immediately dereferenced by the compiler

warning: this expression creates a reference which is immediately dereferenced by the compiler
   --> src/core/tests.rs:427:13
    |
427 |             &json_term_writer.term(),
    |             ^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `json_term_writer.term()`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
    = note: `#[warn(clippy::needless_borrow)]` on by default