Avoid copying into OwnedBytes when opening a fast field column Dictionary.
#105
stuhoodAnnotations
144 warnings
|
this lint expectation is unfulfilled:
src/indexer/stamper.rs#L120
warning: this lint expectation is unfulfilled
--> src/indexer/stamper.rs:120:14
|
120 | #[expect(clippy::redundant_clone)]
| ^^^^^^^^^^^^^^^^^^^^^^^
|
|
this lint expectation is unfulfilled:
src/indexer/stamper.rs#L104
warning: this lint expectation is unfulfilled
--> src/indexer/stamper.rs:104:14
|
104 | #[expect(clippy::redundant_clone)]
| ^^^^^^^^^^^^^^^^^^^^^^^
|
= note: `#[warn(unfulfilled_lint_expectations)]` on by default
|
|
variables can be used directly in the `format!` string:
src/compat_tests.rs#L33
warning: variables can be used directly in the `format!` string
--> src/compat_tests.rs:33:5
|
33 | format!("./tests/compat_tests_data/index_v{}/", version)
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
33 - format!("./tests/compat_tests_data/index_v{}/", version)
33 + format!("./tests/compat_tests_data/index_v{version}/")
|
|
|
variables can be used directly in the `format!` string:
src/reader/mod.rs#L89
warning: variables can be used directly in the `format!` string
--> src/reader/mod.rs:89:25
|
89 | / error!(
90 | | "Error while loading searcher after commit was detected. {:?}",
91 | | err
92 | | );
| |_________________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
|
|
this can be `std::io::Error::other(_)`:
src/termdict/mod.rs#L96
warning: this can be `std::io::Error::other(_)`
--> src/termdict/mod.rs:96:24
|
96 | return Err(io::Error::new(
| ________________________^
97 | | io::ErrorKind::Other,
98 | | format!(
99 | | "Unsupported dictionary type, compiled tantivy with {CURRENT_TYPE:?}, but got \
100 | | {dict_type:?}",
101 | | ),
102 | | ));
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
96 ~ return Err(io::Error::other(
97 ~ format!(
|
|
|
this can be `std::io::Error::other(_)`:
src/termdict/mod.rs#L89
warning: this can be `std::io::Error::other(_)`
--> src/termdict/mod.rs:89:13
|
89 | / io::Error::new(
90 | | io::ErrorKind::Other,
91 | | format!("Unsupported dictionary type, found {dict_type}"),
92 | | )
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
89 ~ io::Error::other(
90 ~ format!("Unsupported dictionary type, found {dict_type}"),
|
|
|
this can be `std::io::Error::other(_)`:
src/termdict/fst_termdict/termdict.rs#L131
warning: this can be `std::io::Error::other(_)`
--> src/termdict/fst_termdict/termdict.rs:131:24
|
131 | return Err(io::Error::new(
| ________________________^
132 | | io::ErrorKind::Other,
133 | | format!("Unsupported fst version, expected {version}, found {FST_VERSION}",),
134 | | ));
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
131 ~ return Err(io::Error::other(
132 ~ format!("Unsupported fst version, expected {version}, found {FST_VERSION}",),
|
|
|
this can be `std::io::Error::other(_)`:
src/termdict/fst_termdict/termdict.rs#L16
warning: this can be `std::io::Error::other(_)`
--> src/termdict/fst_termdict/termdict.rs:16:5
|
16 | io::Error::new(io::ErrorKind::Other, e)
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
16 - io::Error::new(io::ErrorKind::Other, e)
16 + io::Error::other(e)
|
|
|
method `next` can be confused for the standard trait method `std::iter::Iterator::next`:
src/termdict/fst_termdict/streamer.rs#L288
warning: method `next` can be confused for the standard trait method `std::iter::Iterator::next`
--> src/termdict/fst_termdict/streamer.rs:288:5
|
288 | / pub fn next(&mut self) -> Option<(&[u8], &TermInfo, A::State)> {
289 | | if self.advance() {
290 | | let state = self.current_state.take().unwrap(); // always Some(_) after advance
291 | | Some((self.key(), self.value(), state))
... |
295 | | }
| |_____^
|
= help: consider implementing the trait `std::iter::Iterator` or choosing a less ambiguous method name
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#should_implement_trait
= note: `#[warn(clippy::should_implement_trait)]` on by default
|
|
this can be `std::io::Error::other(_)`:
src/store/store_compressor.rs#L238
warning: this can be `std::io::Error::other(_)`
--> src/store/store_compressor.rs:238:25
|
238 | .map_err(|_err| io::Error::new(io::ErrorKind::Other, "Compressing thread panicked."))?
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
238 - .map_err(|_err| io::Error::new(io::ErrorKind::Other, "Compressing thread panicked."))?
238 + .map_err(|_err| io::Error::other("Compressing thread panicked."))?
|
|
|
this can be `std::io::Error::other(_)`:
src/store/store_compressor.rs#L235
warning: this can be `std::io::Error::other(_)`
--> src/store/store_compressor.rs:235:24
|
235 | .ok_or_else(|| io::Error::new(io::ErrorKind::Other, "Thread already joined."))?;
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
235 - .ok_or_else(|| io::Error::new(io::ErrorKind::Other, "Thread already joined."))?;
235 + .ok_or_else(|| io::Error::other("Thread already joined."))?;
|
|
|
this can be `std::io::Error::other(_)`:
src/store/store_compressor.rs#L218
warning: this can be `std::io::Error::other(_)`
--> src/store/store_compressor.rs:218:24
|
218 | return Err(io::Error::new(io::ErrorKind::Other, "Unidentified error."));
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
218 - return Err(io::Error::new(io::ErrorKind::Other, "Unidentified error."));
218 + return Err(io::Error::other("Unidentified error."));
|
|
|
variables can be used directly in the `format!` string:
src/store/reader.rs#L55
warning: variables can be used directly in the `format!` string
--> src/store/reader.rs:55:21
|
55 | format!("Invalid doc store version {}", v),
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
55 - format!("Invalid doc store version {}", v),
55 + format!("Invalid doc store version {v}"),
|
|
|
this can be `std::io::Error::other(_)`:
src/schema/document/se.rs#L279
warning: this can be `std::io::Error::other(_)`
--> src/schema/document/se.rs:279:24
|
279 | return Err(io::Error::new(
| ________________________^
280 | | io::ErrorKind::Other,
281 | | format!(
282 | | "Unexpected number of entries written to serializer, expected {} entries, got \
... |
285 | | ),
286 | | ));
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
279 ~ return Err(io::Error::other(
280 ~ format!(
|
|
|
this can be `std::io::Error::other(_)`:
src/schema/document/se.rs#L218
warning: this can be `std::io::Error::other(_)`
--> src/schema/document/se.rs:218:24
|
218 | return Err(io::Error::new(
| ________________________^
219 | | io::ErrorKind::Other,
220 | | format!(
221 | | "Unexpected number of entries written to serializer, expected {} entries, got \
... |
224 | | ),
225 | | ));
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
218 ~ return Err(io::Error::other(
219 ~ format!(
|
|
|
this can be `std::io::Error::other(_)`:
src/schema/document/se.rs#L58
warning: this can be `std::io::Error::other(_)`
--> src/schema/document/se.rs:58:24
|
58 | return Err(io::Error::new(
| ________________________^
59 | | io::ErrorKind::Other,
60 | | format!(
61 | | "Unexpected number of entries written to serializer, expected \
62 | | {num_field_values} entries, got {actual_length} entries",
63 | | ),
64 | | ));
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
58 ~ return Err(io::Error::other(
59 ~ format!(
|
|
|
item uses `packed` representation without ABI-qualification:
src/schema/document/default_document.rs#L485
warning: item uses `packed` representation without ABI-qualification
--> src/schema/document/default_document.rs:485:1
|
483 | #[repr(packed)]
| ------ `packed` representation set here
484 | /// The value type and the address to its payload in the container.
485 | / struct ValueAddr {
486 | | type_id: ValueType,
487 | | /// This is the address to the value in the vec, except for bool and null, which are inlined
488 | | val_addr: Addr,
489 | | }
| |_^
|
= warning: unqualified `#[repr(packed)]` defaults to `#[repr(Rust, packed)]`, which has no stable ABI
= help: qualify the desired ABI explicitly via `#[repr(C, packed)]` or `#[repr(Rust, packed)]`
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#repr_packed_without_abi
|
|
item uses `packed` representation without ABI-qualification:
src/schema/document/default_document.rs#L21
warning: item uses `packed` representation without ABI-qualification
--> src/schema/document/default_document.rs:21:1
|
18 | #[repr(packed)]
| ------ `packed` representation set here
...
21 | / struct FieldValueAddr {
22 | | pub field: u16,
23 | | pub value_addr: ValueAddr,
24 | | }
| |_^
|
= warning: unqualified `#[repr(packed)]` defaults to `#[repr(Rust, packed)]`, which has no stable ABI
= help: qualify the desired ABI explicitly via `#[repr(C, packed)]` or `#[repr(Rust, packed)]`
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#repr_packed_without_abi
= note: `#[warn(clippy::repr_packed_without_abi)]` on by default
|
|
this expression creates a reference which is immediately dereferenced by the compiler:
src/query/mod.rs#L131
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/query/mod.rs:131:43
|
131 | query.query_terms(text_field, &segment_reader, &mut |term, pos| {
| ^^^^^^^^^^^^^^^ help: change this to: `segment_reader`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
|
this expression creates a reference which is immediately dereferenced by the compiler:
src/query/mod.rs#L123
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/query/mod.rs:123:43
|
123 | query.query_terms(text_field, &segment_reader, &mut |term, pos| {
| ^^^^^^^^^^^^^^^ help: change this to: `segment_reader`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
|
this expression creates a reference which is immediately dereferenced by the compiler:
src/query/mod.rs#L115
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/query/mod.rs:115:43
|
115 | query.query_terms(text_field, &segment_reader, &mut |term, pos| {
| ^^^^^^^^^^^^^^^ help: change this to: `segment_reader`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
|
this expression creates a reference which is immediately dereferenced by the compiler:
src/query/mod.rs#L104
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/query/mod.rs:104:43
|
104 | query.query_terms(text_field, &segment_reader, &mut |term, pos| {
| ^^^^^^^^^^^^^^^ help: change this to: `segment_reader`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
|
this expression creates a reference which is immediately dereferenced by the compiler:
src/query/mod.rs#L96
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/query/mod.rs:96:43
|
96 | query.query_terms(text_field, &segment_reader, &mut |term, pos| {
| ^^^^^^^^^^^^^^^ help: change this to: `segment_reader`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
|
this can be `std::io::Error::other(_)`:
src/query/set_query.rs#L55
warning: this can be `std::io::Error::other(_)`
--> src/query/set_query.rs:55:26
|
55 | .map_err(|e| std::io::Error::new(std::io::ErrorKind::Other, e))?;
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
55 - .map_err(|e| std::io::Error::new(std::io::ErrorKind::Other, e))?;
55 + .map_err(|e| std::io::Error::other(e))?;
|
|
|
variables can be used directly in the `format!` string:
src/query/range_query/range_query_fastfield.rs#L1245
warning: variables can be used directly in the `format!` string
--> src/query/range_query/range_query_fastfield.rs:1245:17
|
1245 | format!("json.{}", field)
| ^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
1245 - format!("json.{}", field)
1245 + format!("json.{field}")
|
|
|
variables can be used directly in the `format!` string:
src/query/range_query/range_query_fastfield.rs#L1061
warning: variables can be used directly in the `format!` string
--> src/query/range_query/range_query_fastfield.rs:1061:22
|
1061 | id_name: format!("id_name{:010}", id),
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
1061 - id_name: format!("id_name{:010}", id),
1061 + id_name: format!("id_name{id:010}"),
|
|
|
variables can be used directly in the `format!` string:
src/query/range_query/range_query_fastfield.rs#L258
warning: variables can be used directly in the `format!` string
--> src/query/range_query/range_query_fastfield.rs:258:9
|
258 | / panic!(
259 | | "internal error: couldn't cast to numerical_type: {:?}",
260 | | col_type
261 | | )
| |_________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
|
|
variables can be used directly in the `format!` string:
src/query/range_query/range_query_fastfield.rs#L195
warning: variables can be used directly in the `format!` string
--> src/query/range_query/range_query_fastfield.rs:195:62
|
195 | return Err(TantivyError::InvalidArgument(format!(
| ______________________________________________________________^
196 | | "Expected term with u64, i64, f64 or date, but got {:?}",
197 | | term
198 | | )));
| |_____________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
|
|
variables can be used directly in the `format!` string:
src/query/range_query/range_query_fastfield.rs#L178
warning: variables can be used directly in the `format!` string
--> src/query/range_query/range_query_fastfield.rs:178:13
|
178 | / assert!(
179 | | maps_to_u64_fastfield(field_type.value_type()),
180 | | "{:?}",
181 | | field_type
182 | | );
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
|
|
variables can be used directly in the `format!` string:
src/query/phrase_query/regex_phrase_weight.rs#L83
warning: variables can be used directly in the `format!` string
--> src/query/phrase_query/regex_phrase_weight.rs:83:65
|
83 | return Err(crate::TantivyError::InvalidArgument(format!(
| _________________________________________________________________^
84 | | "Phrase query exceeded max expansions {}",
85 | | num_terms
86 | | )));
| |_________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
|
|
variables can be used directly in the `format!` string:
src/query/phrase_query/regex_phrase_query.rs#L125
warning: variables can be used directly in the `format!` string
--> src/query/phrase_query/regex_phrase_query.rs:125:57
|
125 | return Err(crate::TantivyError::SchemaError(format!(
| _________________________________________________________^
126 | | "RegexPhraseQuery can only be used with a field of type text currently, but got \
127 | | {:?}",
128 | | field_type
129 | | )));
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
|
|
variables can be used directly in the `format!` string:
src/query/exist_query.rs#L382
warning: variables can be used directly in the `format!` string
--> src/query/exist_query.rs:382:9
|
382 | / assert_eq!(
383 | | searcher
384 | | .search(&ExistsQuery::new(field.to_string(), json_subpaths), &Count)
385 | | .unwrap_err()
386 | | .to_string(),
387 | | format!("The field does not exist: '{}'", field)
388 | | );
| |_________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
387 - format!("The field does not exist: '{}'", field)
387 + format!("The field does not exist: '{field}'")
|
|
|
this call to `as_ref` does nothing:
src/positions/borrowed_position_reader.rs#L67
warning: this call to `as_ref` does nothing
--> src/positions/borrowed_position_reader.rs:67:31
|
67 | let num_bits: usize = self.bit_widths.as_ref()[..num_blocks]
| ^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `self.bit_widths`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_asref
= note: `#[warn(clippy::useless_asref)]` on by default
|
|
this expression creates a reference which is immediately dereferenced by the compiler:
src/index/segment_reader.rs#L557
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/index/segment_reader.rs:557:37
|
557 | CompositeFile::open(&positions_file)
| ^^^^^^^^^^^^^^^ help: change this to: `positions_file`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
|
this expression creates a reference which is immediately dereferenced by the compiler:
src/index/segment_reader.rs#L248
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/index/segment_reader.rs:248:41
|
248 | CompositeFile::open(&positions_file)
| ^^^^^^^^^^^^^^^ help: change this to: `positions_file`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
|
variables can be used directly in the `format!` string:
src/directory/watch_event_router.rs#L93
warning: variables can be used directly in the `format!` string
--> src/directory/watch_event_router.rs:93:13
|
93 | / error!(
94 | | "Failed to spawn thread to call watch callbacks. Cause: {:?}",
95 | | err
96 | | );
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
|
|
this can be `std::io::Error::other(_)`:
src/directory/ram_directory.rs#L194
warning: this can be `std::io::Error::other(_)`
--> src/directory/ram_directory.rs:194:36
|
194 | io_error: Arc::new(io::Error::new(io::ErrorKind::Other, e.to_string())),
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
194 - io_error: Arc::new(io::Error::new(io::ErrorKind::Other, e.to_string())),
194 + io_error: Arc::new(io::Error::other(e.to_string())),
|
|
|
variables can be used directly in the `format!` string:
src/directory/managed_directory.rs#L191
warning: variables can be used directly in the `format!` string
--> src/directory/managed_directory.rs:191:33
|
191 | ... error!("Failed to delete {:?}", file_to_delete);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
191 - error!("Failed to delete {:?}", file_to_delete);
191 + error!("Failed to delete {file_to_delete:?}");
|
|
|
variables can be used directly in the `format!` string:
src/directory/managed_directory.rs#L178
warning: variables can be used directly in the `format!` string
--> src/directory/managed_directory.rs:178:21
|
178 | info!("Deleted {:?}", file_to_delete);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
178 - info!("Deleted {:?}", file_to_delete);
178 + info!("Deleted {file_to_delete:?}");
|
|
|
variables can be used directly in the `format!` string:
src/directory/file_watcher.rs#L80
warning: variables can be used directly in the `format!` string
--> src/directory/file_watcher.rs:80:17
|
80 | warn!("Failed to open meta file {:?}: {:?}", path, e);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
80 - warn!("Failed to open meta file {:?}: {:?}", path, e);
80 + warn!("Failed to open meta file {path:?}: {e:?}");
|
|
|
variables can be used directly in the `format!` string:
src/directory/file_watcher.rs#L56
warning: variables can be used directly in the `format!` string
--> src/directory/file_watcher.rs:56:29
|
56 | ... info!("Meta file {:?} was modified", path);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
56 - info!("Meta file {:?} was modified", path);
56 + info!("Meta file {path:?} was modified");
|
|
|
variables can be used directly in the `format!` string:
src/directory/directory.rs#L67
warning: variables can be used directly in the `format!` string
--> src/directory/directory.rs:67:13
|
67 | error!("Failed to remove the lock file. {:?}", e);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
67 - error!("Failed to remove the lock file. {:?}", e);
67 + error!("Failed to remove the lock file. {e:?}");
|
|
|
variables can be used directly in the `format!` string:
src/directory/mmap_directory.rs#L470
warning: variables can be used directly in the `format!` string
--> src/directory/mmap_directory.rs:470:9
|
470 | debug!("Atomic Write {:?}", path);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
470 - debug!("Atomic Write {:?}", path);
470 + debug!("Atomic Write {path:?}");
|
|
|
variables can be used directly in the `format!` string:
src/directory/mmap_directory.rs#L417
warning: variables can be used directly in the `format!` string
--> src/directory/mmap_directory.rs:417:9
|
417 | debug!("Open Write {:?}", path);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
417 - debug!("Open Write {:?}", path);
417 + debug!("Open Write {path:?}");
|
|
|
variables can be used directly in the `format!` string:
src/directory/mmap_directory.rs#L372
warning: variables can be used directly in the `format!` string
--> src/directory/mmap_directory.rs:372:9
|
372 | debug!("Open Read {:?}", path);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
372 - debug!("Open Read {:?}", path);
372 + debug!("Open Read {path:?}");
|
|
|
this can be `std::io::Error::other(_)`:
src/directory/mmap_directory.rs#L32
warning: this can be `std::io::Error::other(_)`
--> src/directory/mmap_directory.rs:32:5
|
32 | io::Error::new(io::ErrorKind::Other, msg)
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
32 - io::Error::new(io::ErrorKind::Other, msg)
32 + io::Error::other(msg)
|
|
|
using `clone` on type `f32` which implements the `Copy` trait:
src/collector/top_score_collector.rs#L883
warning: using `clone` on type `f32` which implements the `Copy` trait
--> src/collector/top_score_collector.rs:883:30
|
883 | .map(|d| d.feature.clone())
| ^^^^^^^^^^^^^^^^^ help: try removing the `clone` call: `d.feature`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#clone_on_copy
|
|
using `clone` on type `f32` which implements the `Copy` trait:
src/collector/top_score_collector.rs#L873
warning: using `clone` on type `f32` which implements the `Copy` trait
--> src/collector/top_score_collector.rs:873:30
|
873 | .map(|d| d.feature.clone())
| ^^^^^^^^^^^^^^^^^ help: try removing the `clone` call: `d.feature`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#clone_on_copy
= note: `#[warn(clippy::clone_on_copy)]` on by default
|
|
large size difference between variants:
src/aggregation/intermediate_agg_result.rs#L249
warning: large size difference between variants
--> src/aggregation/intermediate_agg_result.rs:249:1
|
249 | / pub enum IntermediateAggregationResult {
250 | | /// Bucket variant
251 | | Bucket(IntermediateBucketResult),
| | -------------------------------- the second-largest variant contains at least 56 bytes
252 | | /// Metric variant
253 | | Metric(IntermediateMetricResult),
| | -------------------------------- the largest variant contains at least 376 bytes
254 | | }
| |_^ the entire enum is at least 376 bytes
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#large_enum_variant
= note: `#[warn(clippy::large_enum_variant)]` on by default
help: consider boxing the large fields to reduce the total size of the enum
|
253 - Metric(IntermediateMetricResult),
253 + Metric(Box<IntermediateMetricResult>),
|
|
|
this can be `std::io::Error::other(_)`:
src/aggregation/bucket/term_agg.rs#L521
warning: this can be `std::io::Error::other(_)`
--> src/aggregation/bucket/term_agg.rs:521:40
|
521 | .map_err(|err| io::Error::new(io::ErrorKind::Other, err))?;
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
521 - .map_err(|err| io::Error::new(io::ErrorKind::Other, err))?;
521 + .map_err(|err| io::Error::other(err))?;
|
|
|
this can be `std::io::Error::other(_)`:
src/tokenizer/tokenized_string.rs#L48
warning: this can be `std::io::Error::other(_)`
--> src/tokenizer/tokenized_string.rs:48:17
|
48 | Err(io::Error::new(
| _________________^
49 | | io::ErrorKind::Other,
50 | | "Failed to parse string data as PreTokenizedString.",
51 | | ))
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
48 ~ Err(io::Error::other(
49 ~ "Failed to parse string data as PreTokenizedString.",
|
|
|
this can be `std::io::Error::other(_)`:
src/tokenizer/tokenized_string.rs#L35
warning: this can be `std::io::Error::other(_)`
--> src/tokenizer/tokenized_string.rs:35:17
|
35 | Err(io::Error::new(
| _________________^
36 | | io::ErrorKind::Other,
37 | | "Failed to dump PreTokenizedString to json.",
38 | | ))
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
= note: `#[warn(clippy::io_other_error)]` on by default
help: use `std::io::Error::other`
|
35 ~ Err(io::Error::other(
36 ~ "Failed to dump PreTokenizedString to json.",
|
|
|
variables can be used directly in the `format!` string:
src/indexer/segment_writer.rs#L877
warning: variables can be used directly in the `format!` string
--> src/indexer/segment_writer.rs:877:13
|
877 | assert_eq!(cols[0].column_type(), typ, "{}", field);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
877 - assert_eq!(cols[0].column_type(), typ, "{}", field);
877 + assert_eq!(cols[0].column_type(), typ, "{field}");
|
|
|
variables can be used directly in the `format!` string:
src/indexer/segment_writer.rs#L876
warning: variables can be used directly in the `format!` string
--> src/indexer/segment_writer.rs:876:13
|
876 | assert_eq!(cols.len(), 1, "{}", field);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
876 - assert_eq!(cols.len(), 1, "{}", field);
876 + assert_eq!(cols.len(), 1, "{field}");
|
|
|
variables can be used directly in the `format!` string:
src/indexer/segment_updater.rs#L594
warning: variables can be used directly in the `format!` string
--> src/indexer/segment_updater.rs:594:17
|
594 | / warn!(
595 | | "Starting the merge failed for the following reason. This is not fatal. {}",
596 | | err
597 | | );
| |_________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
|
|
this expression creates a reference which is immediately dereferenced by the compiler:
src/indexer/segment_updater.rs#L492
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/indexer/segment_updater.rs:492:17
|
492 | &previous_metas,
| ^^^^^^^^^^^^^^^ help: change this to: `previous_metas`
|
= 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
|
|
this `if` statement can be collapsed:
src/indexer/merger.rs#L382
warning: this `if` statement can be collapsed
--> src/indexer/merger.rs:382:13
|
382 | / if cnt % 1000 == 0 {
383 | | if self.cancel.wants_cancel() {
384 | | return Err(crate::TantivyError::Cancelled);
385 | | }
386 | | }
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#collapsible_if
help: collapse nested if block
|
382 ~ if cnt % 1000 == 0
383 ~ && self.cancel.wants_cancel() {
384 | return Err(crate::TantivyError::Cancelled);
385 ~ }
|
|
|
variables can be used directly in the `format!` string:
src/indexer/index_writer.rs#L692
warning: variables can be used directly in the `format!` string
--> src/indexer/index_writer.rs:692:9
|
692 | info!("Prepared commit {}", commit_opstamp);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
692 - info!("Prepared commit {}", commit_opstamp);
692 + info!("Prepared commit {commit_opstamp}");
|
|
|
variables can be used directly in the `format!` string:
src/indexer/index_writer.rs#L395
warning: variables can be used directly in the `format!` string
--> src/indexer/index_writer.rs:395:13
|
395 | error!("Some merging thread failed {:?}", e);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
395 - error!("Some merging thread failed {:?}", e);
395 + error!("Some merging thread failed {e:?}");
|
|
|
this `if` statement can be collapsed:
src/indexer/index_writer.rs#L127
warning: this `if` statement can be collapsed
--> src/indexer/index_writer.rs:127:17
|
127 | / if *segment_id == segment_reader.segment_id() {
128 | | if doc_opstamps.is_deleted(*doc_id, *opstamp) {
129 | | alive_bitset.remove(*doc_id);
130 | | might_have_changed = true;
131 | | }
132 | | }
| |_________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#collapsible_if
= note: `#[warn(clippy::collapsible_if)]` on by default
help: collapse nested if block
|
127 ~ if *segment_id == segment_reader.segment_id()
128 ~ && doc_opstamps.is_deleted(*doc_id, *opstamp) {
129 | alive_bitset.remove(*doc_id);
130 | might_have_changed = true;
131 ~ }
|
|
|
you should consider adding a `Default` implementation for `DeleteQueue`:
src/indexer/delete_queue.rs#L33
warning: you should consider adding a `Default` implementation for `DeleteQueue`
--> src/indexer/delete_queue.rs:33:5
|
33 | / pub fn new() -> DeleteQueue {
34 | | DeleteQueue {
35 | | inner: Arc::default(),
36 | | }
37 | | }
| |_____^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#new_without_default
= note: `#[warn(clippy::new_without_default)]` on by default
help: try adding this
|
31 + impl Default for DeleteQueue {
32 + fn default() -> Self {
33 + Self::new()
34 + }
35 + }
|
|
|
variables can be used directly in the `format!` string:
src/core/executor.rs#L70
warning: variables can be used directly in the `format!` string
--> src/core/executor.rs:70:37
|
70 | / ... error!(
71 | | ... "Failed to send search task. It probably means all search \
72 | | ... threads have panicked. {:?}",
73 | | ... err
74 | | ... );
| |_______________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
= note: `#[warn(clippy::uninlined_format_args)]` on by default
|
|
variables can be used directly in the `format!` string:
src/reader/mod.rs#L89
warning: variables can be used directly in the `format!` string
--> src/reader/mod.rs:89:25
|
89 | / error!(
90 | | "Error while loading searcher after commit was detected. {:?}",
91 | | err
92 | | );
| |_________________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
|
|
variables can be used directly in the `format!` string:
src/reader/warming.rs#L155
warning: variables can be used directly in the `format!` string
--> src/reader/warming.rs:155:21
|
155 | error!("Panic in Warmer GC {:?}", err);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
155 - error!("Panic in Warmer GC {:?}", err);
155 + error!("Panic in Warmer GC {err:?}");
|
|
|
this can be `std::io::Error::other(_)`:
src/termdict/mod.rs#L96
warning: this can be `std::io::Error::other(_)`
--> src/termdict/mod.rs:96:24
|
96 | return Err(io::Error::new(
| ________________________^
97 | | io::ErrorKind::Other,
98 | | format!(
99 | | "Unsupported dictionary type, compiled tantivy with {CURRENT_TYPE:?}, but got \
100 | | {dict_type:?}",
101 | | ),
102 | | ));
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
96 ~ return Err(io::Error::other(
97 ~ format!(
|
|
|
this can be `std::io::Error::other(_)`:
src/termdict/mod.rs#L89
warning: this can be `std::io::Error::other(_)`
--> src/termdict/mod.rs:89:13
|
89 | / io::Error::new(
90 | | io::ErrorKind::Other,
91 | | format!("Unsupported dictionary type, found {dict_type}"),
92 | | )
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
89 ~ io::Error::other(
90 ~ format!("Unsupported dictionary type, found {dict_type}"),
|
|
|
this can be `std::io::Error::other(_)`:
src/termdict/fst_termdict/termdict.rs#L131
warning: this can be `std::io::Error::other(_)`
--> src/termdict/fst_termdict/termdict.rs:131:24
|
131 | return Err(io::Error::new(
| ________________________^
132 | | io::ErrorKind::Other,
133 | | format!("Unsupported fst version, expected {version}, found {FST_VERSION}",),
134 | | ));
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
131 ~ return Err(io::Error::other(
132 ~ format!("Unsupported fst version, expected {version}, found {FST_VERSION}",),
|
|
|
this can be `std::io::Error::other(_)`:
src/termdict/fst_termdict/termdict.rs#L16
warning: this can be `std::io::Error::other(_)`
--> src/termdict/fst_termdict/termdict.rs:16:5
|
16 | io::Error::new(io::ErrorKind::Other, e)
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
16 - io::Error::new(io::ErrorKind::Other, e)
16 + io::Error::other(e)
|
|
|
method `next` can be confused for the standard trait method `std::iter::Iterator::next`:
src/termdict/fst_termdict/streamer.rs#L288
warning: method `next` can be confused for the standard trait method `std::iter::Iterator::next`
--> src/termdict/fst_termdict/streamer.rs:288:5
|
288 | / pub fn next(&mut self) -> Option<(&[u8], &TermInfo, A::State)> {
289 | | if self.advance() {
290 | | let state = self.current_state.take().unwrap(); // always Some(_) after advance
291 | | Some((self.key(), self.value(), state))
... |
295 | | }
| |_____^
|
= help: consider implementing the trait `std::iter::Iterator` or choosing a less ambiguous method name
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#should_implement_trait
= note: `#[warn(clippy::should_implement_trait)]` on by default
|
|
this can be `std::io::Error::other(_)`:
src/store/store_compressor.rs#L238
warning: this can be `std::io::Error::other(_)`
--> src/store/store_compressor.rs:238:25
|
238 | .map_err(|_err| io::Error::new(io::ErrorKind::Other, "Compressing thread panicked."))?
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
238 - .map_err(|_err| io::Error::new(io::ErrorKind::Other, "Compressing thread panicked."))?
238 + .map_err(|_err| io::Error::other("Compressing thread panicked."))?
|
|
|
this can be `std::io::Error::other(_)`:
src/store/store_compressor.rs#L235
warning: this can be `std::io::Error::other(_)`
--> src/store/store_compressor.rs:235:24
|
235 | .ok_or_else(|| io::Error::new(io::ErrorKind::Other, "Thread already joined."))?;
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
235 - .ok_or_else(|| io::Error::new(io::ErrorKind::Other, "Thread already joined."))?;
235 + .ok_or_else(|| io::Error::other("Thread already joined."))?;
|
|
|
this can be `std::io::Error::other(_)`:
src/store/store_compressor.rs#L218
warning: this can be `std::io::Error::other(_)`
--> src/store/store_compressor.rs:218:24
|
218 | return Err(io::Error::new(io::ErrorKind::Other, "Unidentified error."));
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
218 - return Err(io::Error::new(io::ErrorKind::Other, "Unidentified error."));
218 + return Err(io::Error::other("Unidentified error."));
|
|
|
variables can be used directly in the `format!` string:
src/store/reader.rs#L55
warning: variables can be used directly in the `format!` string
--> src/store/reader.rs:55:21
|
55 | format!("Invalid doc store version {}", v),
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
55 - format!("Invalid doc store version {}", v),
55 + format!("Invalid doc store version {v}"),
|
|
|
this can be `std::io::Error::other(_)`:
src/schema/document/se.rs#L279
warning: this can be `std::io::Error::other(_)`
--> src/schema/document/se.rs:279:24
|
279 | return Err(io::Error::new(
| ________________________^
280 | | io::ErrorKind::Other,
281 | | format!(
282 | | "Unexpected number of entries written to serializer, expected {} entries, got \
... |
285 | | ),
286 | | ));
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
279 ~ return Err(io::Error::other(
280 ~ format!(
|
|
|
this can be `std::io::Error::other(_)`:
src/schema/document/se.rs#L218
warning: this can be `std::io::Error::other(_)`
--> src/schema/document/se.rs:218:24
|
218 | return Err(io::Error::new(
| ________________________^
219 | | io::ErrorKind::Other,
220 | | format!(
221 | | "Unexpected number of entries written to serializer, expected {} entries, got \
... |
224 | | ),
225 | | ));
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
218 ~ return Err(io::Error::other(
219 ~ format!(
|
|
|
this can be `std::io::Error::other(_)`:
src/schema/document/se.rs#L58
warning: this can be `std::io::Error::other(_)`
--> src/schema/document/se.rs:58:24
|
58 | return Err(io::Error::new(
| ________________________^
59 | | io::ErrorKind::Other,
60 | | format!(
61 | | "Unexpected number of entries written to serializer, expected \
62 | | {num_field_values} entries, got {actual_length} entries",
63 | | ),
64 | | ));
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
58 ~ return Err(io::Error::other(
59 ~ format!(
|
|
|
item uses `packed` representation without ABI-qualification:
src/schema/document/default_document.rs#L485
warning: item uses `packed` representation without ABI-qualification
--> src/schema/document/default_document.rs:485:1
|
483 | #[repr(packed)]
| ------ `packed` representation set here
484 | /// The value type and the address to its payload in the container.
485 | / struct ValueAddr {
486 | | type_id: ValueType,
487 | | /// This is the address to the value in the vec, except for bool and null, which are inlined
488 | | val_addr: Addr,
489 | | }
| |_^
|
= warning: unqualified `#[repr(packed)]` defaults to `#[repr(Rust, packed)]`, which has no stable ABI
= help: qualify the desired ABI explicitly via `#[repr(C, packed)]` or `#[repr(Rust, packed)]`
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#repr_packed_without_abi
|
|
item uses `packed` representation without ABI-qualification:
src/schema/document/default_document.rs#L21
warning: item uses `packed` representation without ABI-qualification
--> src/schema/document/default_document.rs:21:1
|
18 | #[repr(packed)]
| ------ `packed` representation set here
...
21 | / struct FieldValueAddr {
22 | | pub field: u16,
23 | | pub value_addr: ValueAddr,
24 | | }
| |_^
|
= warning: unqualified `#[repr(packed)]` defaults to `#[repr(Rust, packed)]`, which has no stable ABI
= help: qualify the desired ABI explicitly via `#[repr(C, packed)]` or `#[repr(Rust, packed)]`
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#repr_packed_without_abi
= note: `#[warn(clippy::repr_packed_without_abi)]` on by default
|
|
this can be `std::io::Error::other(_)`:
src/query/set_query.rs#L55
warning: this can be `std::io::Error::other(_)`
--> src/query/set_query.rs:55:26
|
55 | .map_err(|e| std::io::Error::new(std::io::ErrorKind::Other, e))?;
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
55 - .map_err(|e| std::io::Error::new(std::io::ErrorKind::Other, e))?;
55 + .map_err(|e| std::io::Error::other(e))?;
|
|
|
variables can be used directly in the `format!` string:
src/query/range_query/range_query_fastfield.rs#L258
warning: variables can be used directly in the `format!` string
--> src/query/range_query/range_query_fastfield.rs:258:9
|
258 | / panic!(
259 | | "internal error: couldn't cast to numerical_type: {:?}",
260 | | col_type
261 | | )
| |_________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
|
|
variables can be used directly in the `format!` string:
src/query/range_query/range_query_fastfield.rs#L195
warning: variables can be used directly in the `format!` string
--> src/query/range_query/range_query_fastfield.rs:195:62
|
195 | return Err(TantivyError::InvalidArgument(format!(
| ______________________________________________________________^
196 | | "Expected term with u64, i64, f64 or date, but got {:?}",
197 | | term
198 | | )));
| |_____________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
|
|
variables can be used directly in the `format!` string:
src/query/range_query/range_query_fastfield.rs#L178
warning: variables can be used directly in the `format!` string
--> src/query/range_query/range_query_fastfield.rs:178:13
|
178 | / assert!(
179 | | maps_to_u64_fastfield(field_type.value_type()),
180 | | "{:?}",
181 | | field_type
182 | | );
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
|
|
variables can be used directly in the `format!` string:
src/query/phrase_query/regex_phrase_weight.rs#L83
warning: variables can be used directly in the `format!` string
--> src/query/phrase_query/regex_phrase_weight.rs:83:65
|
83 | return Err(crate::TantivyError::InvalidArgument(format!(
| _________________________________________________________________^
84 | | "Phrase query exceeded max expansions {}",
85 | | num_terms
86 | | )));
| |_________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
|
|
variables can be used directly in the `format!` string:
src/query/phrase_query/regex_phrase_query.rs#L125
warning: variables can be used directly in the `format!` string
--> src/query/phrase_query/regex_phrase_query.rs:125:57
|
125 | return Err(crate::TantivyError::SchemaError(format!(
| _________________________________________________________^
126 | | "RegexPhraseQuery can only be used with a field of type text currently, but got \
127 | | {:?}",
128 | | field_type
129 | | )));
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
|
|
this call to `as_ref` does nothing:
src/positions/borrowed_position_reader.rs#L67
warning: this call to `as_ref` does nothing
--> src/positions/borrowed_position_reader.rs:67:31
|
67 | let num_bits: usize = self.bit_widths.as_ref()[..num_blocks]
| ^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `self.bit_widths`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_asref
= note: `#[warn(clippy::useless_asref)]` on by default
|
|
this expression creates a reference which is immediately dereferenced by the compiler:
src/index/segment_reader.rs#L557
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/index/segment_reader.rs:557:37
|
557 | CompositeFile::open(&positions_file)
| ^^^^^^^^^^^^^^^ help: change this to: `positions_file`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
|
this expression creates a reference which is immediately dereferenced by the compiler:
src/index/segment_reader.rs#L248
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/index/segment_reader.rs:248:41
|
248 | CompositeFile::open(&positions_file)
| ^^^^^^^^^^^^^^^ help: change this to: `positions_file`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
|
variables can be used directly in the `format!` string:
src/directory/watch_event_router.rs#L93
warning: variables can be used directly in the `format!` string
--> src/directory/watch_event_router.rs:93:13
|
93 | / error!(
94 | | "Failed to spawn thread to call watch callbacks. Cause: {:?}",
95 | | err
96 | | );
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
|
|
this can be `std::io::Error::other(_)`:
src/directory/ram_directory.rs#L194
warning: this can be `std::io::Error::other(_)`
--> src/directory/ram_directory.rs:194:36
|
194 | io_error: Arc::new(io::Error::new(io::ErrorKind::Other, e.to_string())),
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
194 - io_error: Arc::new(io::Error::new(io::ErrorKind::Other, e.to_string())),
194 + io_error: Arc::new(io::Error::other(e.to_string())),
|
|
|
variables can be used directly in the `format!` string:
src/directory/managed_directory.rs#L191
warning: variables can be used directly in the `format!` string
--> src/directory/managed_directory.rs:191:33
|
191 | ... error!("Failed to delete {:?}", file_to_delete);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
191 - error!("Failed to delete {:?}", file_to_delete);
191 + error!("Failed to delete {file_to_delete:?}");
|
|
|
variables can be used directly in the `format!` string:
src/directory/managed_directory.rs#L178
warning: variables can be used directly in the `format!` string
--> src/directory/managed_directory.rs:178:21
|
178 | info!("Deleted {:?}", file_to_delete);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
178 - info!("Deleted {:?}", file_to_delete);
178 + info!("Deleted {file_to_delete:?}");
|
|
|
variables can be used directly in the `format!` string:
src/directory/file_watcher.rs#L80
warning: variables can be used directly in the `format!` string
--> src/directory/file_watcher.rs:80:17
|
80 | warn!("Failed to open meta file {:?}: {:?}", path, e);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
80 - warn!("Failed to open meta file {:?}: {:?}", path, e);
80 + warn!("Failed to open meta file {path:?}: {e:?}");
|
|
|
variables can be used directly in the `format!` string:
src/directory/file_watcher.rs#L56
warning: variables can be used directly in the `format!` string
--> src/directory/file_watcher.rs:56:29
|
56 | ... info!("Meta file {:?} was modified", path);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
56 - info!("Meta file {:?} was modified", path);
56 + info!("Meta file {path:?} was modified");
|
|
|
variables can be used directly in the `format!` string:
src/directory/directory.rs#L67
warning: variables can be used directly in the `format!` string
--> src/directory/directory.rs:67:13
|
67 | error!("Failed to remove the lock file. {:?}", e);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
67 - error!("Failed to remove the lock file. {:?}", e);
67 + error!("Failed to remove the lock file. {e:?}");
|
|
|
variables can be used directly in the `format!` string:
src/directory/mmap_directory.rs#L470
warning: variables can be used directly in the `format!` string
--> src/directory/mmap_directory.rs:470:9
|
470 | debug!("Atomic Write {:?}", path);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
470 - debug!("Atomic Write {:?}", path);
470 + debug!("Atomic Write {path:?}");
|
|
|
variables can be used directly in the `format!` string:
src/directory/mmap_directory.rs#L417
warning: variables can be used directly in the `format!` string
--> src/directory/mmap_directory.rs:417:9
|
417 | debug!("Open Write {:?}", path);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
417 - debug!("Open Write {:?}", path);
417 + debug!("Open Write {path:?}");
|
|
|
variables can be used directly in the `format!` string:
src/directory/mmap_directory.rs#L372
warning: variables can be used directly in the `format!` string
--> src/directory/mmap_directory.rs:372:9
|
372 | debug!("Open Read {:?}", path);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
372 - debug!("Open Read {:?}", path);
372 + debug!("Open Read {path:?}");
|
|
|
this can be `std::io::Error::other(_)`:
src/directory/mmap_directory.rs#L32
warning: this can be `std::io::Error::other(_)`
--> src/directory/mmap_directory.rs:32:5
|
32 | io::Error::new(io::ErrorKind::Other, msg)
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
32 - io::Error::new(io::ErrorKind::Other, msg)
32 + io::Error::other(msg)
|
|
|
using `clone` on type `f32` which implements the `Copy` trait:
src/collector/top_score_collector.rs#L883
warning: using `clone` on type `f32` which implements the `Copy` trait
--> src/collector/top_score_collector.rs:883:30
|
883 | .map(|d| d.feature.clone())
| ^^^^^^^^^^^^^^^^^ help: try removing the `clone` call: `d.feature`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#clone_on_copy
|
|
using `clone` on type `f32` which implements the `Copy` trait:
src/collector/top_score_collector.rs#L873
warning: using `clone` on type `f32` which implements the `Copy` trait
--> src/collector/top_score_collector.rs:873:30
|
873 | .map(|d| d.feature.clone())
| ^^^^^^^^^^^^^^^^^ help: try removing the `clone` call: `d.feature`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#clone_on_copy
= note: `#[warn(clippy::clone_on_copy)]` on by default
|
|
large size difference between variants:
src/aggregation/intermediate_agg_result.rs#L249
warning: large size difference between variants
--> src/aggregation/intermediate_agg_result.rs:249:1
|
249 | / pub enum IntermediateAggregationResult {
250 | | /// Bucket variant
251 | | Bucket(IntermediateBucketResult),
| | -------------------------------- the second-largest variant contains at least 56 bytes
252 | | /// Metric variant
253 | | Metric(IntermediateMetricResult),
| | -------------------------------- the largest variant contains at least 376 bytes
254 | | }
| |_^ the entire enum is at least 376 bytes
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#large_enum_variant
= note: `#[warn(clippy::large_enum_variant)]` on by default
help: consider boxing the large fields to reduce the total size of the enum
|
253 - Metric(IntermediateMetricResult),
253 + Metric(Box<IntermediateMetricResult>),
|
|
|
this can be `std::io::Error::other(_)`:
src/aggregation/bucket/term_agg.rs#L521
warning: this can be `std::io::Error::other(_)`
--> src/aggregation/bucket/term_agg.rs:521:40
|
521 | .map_err(|err| io::Error::new(io::ErrorKind::Other, err))?;
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
521 - .map_err(|err| io::Error::new(io::ErrorKind::Other, err))?;
521 + .map_err(|err| io::Error::other(err))?;
|
|
|
this can be `std::io::Error::other(_)`:
src/tokenizer/tokenized_string.rs#L48
warning: this can be `std::io::Error::other(_)`
--> src/tokenizer/tokenized_string.rs:48:17
|
48 | Err(io::Error::new(
| _________________^
49 | | io::ErrorKind::Other,
50 | | "Failed to parse string data as PreTokenizedString.",
51 | | ))
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
48 ~ Err(io::Error::other(
49 ~ "Failed to parse string data as PreTokenizedString.",
|
|
|
this can be `std::io::Error::other(_)`:
src/tokenizer/tokenized_string.rs#L35
warning: this can be `std::io::Error::other(_)`
--> src/tokenizer/tokenized_string.rs:35:17
|
35 | Err(io::Error::new(
| _________________^
36 | | io::ErrorKind::Other,
37 | | "Failed to dump PreTokenizedString to json.",
38 | | ))
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
= note: `#[warn(clippy::io_other_error)]` on by default
help: use `std::io::Error::other`
|
35 ~ Err(io::Error::other(
36 ~ "Failed to dump PreTokenizedString to json.",
|
|
|
variables can be used directly in the `format!` string:
src/indexer/segment_updater.rs#L594
warning: variables can be used directly in the `format!` string
--> src/indexer/segment_updater.rs:594:17
|
594 | / warn!(
595 | | "Starting the merge failed for the following reason. This is not fatal. {}",
596 | | err
597 | | );
| |_________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
|
|
this expression creates a reference which is immediately dereferenced by the compiler:
src/indexer/segment_updater.rs#L492
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/indexer/segment_updater.rs:492:17
|
492 | &previous_metas,
| ^^^^^^^^^^^^^^^ help: change this to: `previous_metas`
|
= 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
|
|
this `if` statement can be collapsed:
src/indexer/merger.rs#L382
warning: this `if` statement can be collapsed
--> src/indexer/merger.rs:382:13
|
382 | / if cnt % 1000 == 0 {
383 | | if self.cancel.wants_cancel() {
384 | | return Err(crate::TantivyError::Cancelled);
385 | | }
386 | | }
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#collapsible_if
help: collapse nested if block
|
382 ~ if cnt % 1000 == 0
383 ~ && self.cancel.wants_cancel() {
384 | return Err(crate::TantivyError::Cancelled);
385 ~ }
|
|
|
variables can be used directly in the `format!` string:
src/indexer/index_writer.rs#L692
warning: variables can be used directly in the `format!` string
--> src/indexer/index_writer.rs:692:9
|
692 | info!("Prepared commit {}", commit_opstamp);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
692 - info!("Prepared commit {}", commit_opstamp);
692 + info!("Prepared commit {commit_opstamp}");
|
|
|
variables can be used directly in the `format!` string:
src/indexer/index_writer.rs#L395
warning: variables can be used directly in the `format!` string
--> src/indexer/index_writer.rs:395:13
|
395 | error!("Some merging thread failed {:?}", e);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
help: change this to
|
395 - error!("Some merging thread failed {:?}", e);
395 + error!("Some merging thread failed {e:?}");
|
|
|
you should consider adding a `Default` implementation for `DeleteQueue`:
src/indexer/delete_queue.rs#L33
warning: you should consider adding a `Default` implementation for `DeleteQueue`
--> src/indexer/delete_queue.rs:33:5
|
33 | / pub fn new() -> DeleteQueue {
34 | | DeleteQueue {
35 | | inner: Arc::default(),
36 | | }
37 | | }
| |_____^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#new_without_default
= note: `#[warn(clippy::new_without_default)]` on by default
help: try adding this
|
31 + impl Default for DeleteQueue {
32 + fn default() -> Self {
33 + Self::new()
34 + }
35 + }
|
|
|
variables can be used directly in the `format!` string:
src/core/executor.rs#L70
warning: variables can be used directly in the `format!` string
--> src/core/executor.rs:70:37
|
70 | / ... error!(
71 | | ... "Failed to send search task. It probably means all search \
72 | | ... threads have panicked. {:?}",
73 | | ... err
74 | | ... );
| |_______________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
= note: `#[warn(clippy::uninlined_format_args)]` on by default
|
|
manually reimplementing `div_ceil`:
columnar/src/columnar/writer/column_operation.rs#L247
warning: manually reimplementing `div_ceil`
--> columnar/src/columnar/writer/column_operation.rs:247:5
|
247 | (msb + 7) / 8
| ^^^^^^^^^^^^^ help: consider using `.div_ceil()`: `msb.div_ceil(8)`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_div_ceil
|
|
accessing first element with `header
.read_bytes()?
.as_slice().get(0)`:
columnar/src/column_values/u64_based/mod.rs#L198
warning: accessing first element with `header
.read_bytes()?
.as_slice().get(0)`
--> columnar/src/column_values/u64_based/mod.rs:198:33
|
198 | let codec_type: CodecType = header
| _________________________________^
199 | | .read_bytes()?
200 | | .as_slice()
201 | | .get(0)
| |_______________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#get_first
= note: `#[warn(clippy::get_first)]` on by default
help: try
|
198 ~ let codec_type: CodecType = header
199 + .read_bytes()?
200 + .as_slice().first()
|
|
|
manually reimplementing `div_ceil`:
columnar/src/column_values/u64_based/linear.rs#L121
warning: manually reimplementing `div_ceil`
--> columnar/src/column_values/u64_based/linear.rs:121:19
|
121 | + (num_bits as u64 * stats.num_rows as u64 + 7) / 8,
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using `.div_ceil()`: `(num_bits as u64 * stats.num_rows as u64).div_ceil(8)`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_div_ceil
|
|
manually reimplementing `div_ceil`:
columnar/src/column_values/u64_based/bitpacked.rs#L113
warning: manually reimplementing `div_ceil`
--> columnar/src/column_values/u64_based/bitpacked.rs:113:34
|
113 | Some(stats.num_bytes() + (stats.num_rows as u64 * (num_bits_per_value as u64) + 7) / 8)
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using `.div_ceil()`: `(stats.num_rows as u64 * (num_bits_per_value as u64)).div_ceil(8)`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_div_ceil
= note: `#[warn(clippy::manual_div_ceil)]` on by default
|
|
this `repeat().take()` can be written more concisely:
columnar/src/column_index/merge/stacked.rs#L110
warning: this `repeat().take()` can be written more concisely
--> columnar/src/column_index/merge/stacked.rs:110:22
|
110 | Box::new(std::iter::repeat(1u32).take(optional_index.num_non_nulls() as usize))
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using `repeat_n()` instead: `std::iter::repeat_n(1u32, optional_index.num_non_nulls() as usize)`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_repeat_n
|
|
this `repeat().take()` can be written more concisely:
columnar/src/column_index/merge/stacked.rs#L108
warning: this `repeat().take()` can be written more concisely
--> columnar/src/column_index/merge/stacked.rs:108:39
|
108 | ColumnIndex::Full => Box::new(std::iter::repeat(1u32).take(num_docs as usize)),
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using `repeat_n()` instead: `std::iter::repeat_n(1u32, num_docs as usize)`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_repeat_n
= note: `#[warn(clippy::manual_repeat_n)]` on by default
|
|
manually reimplementing `div_ceil`:
sstable/src/sstable_index_v3.rs#L441
warning: manually reimplementing `div_ceil`
--> sstable/src/sstable_index_v3.rs:441:12
|
441 | if (range_end_addr + self.range_start_nbits as usize + 7) / 8 > data.len() {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using `.div_ceil()`: `(range_end_addr + self.range_start_nbits as usize).div_ceil(8)`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_div_ceil
= note: `#[warn(clippy::manual_div_ceil)]` on by default
|
|
this can be `std::io::Error::other(_)`:
sstable/src/sstable_index_v3.rs#L397
warning: this can be `std::io::Error::other(_)`
--> sstable/src/sstable_index_v3.rs:397:47
|
397 | tantivy_fst::Error::Fst(fst_error) => io::Error::new(io::ErrorKind::Other, fst_error),
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
help: use `std::io::Error::other`
|
397 - tantivy_fst::Error::Fst(fst_error) => io::Error::new(io::ErrorKind::Other, fst_error),
397 + tantivy_fst::Error::Fst(fst_error) => io::Error::other(fst_error),
|
|
|
method `next` can be confused for the standard trait method `std::iter::Iterator::next`:
sstable/src/streamer.rs#L389
warning: method `next` can be confused for the standard trait method `std::iter::Iterator::next`
--> sstable/src/streamer.rs:389:5
|
389 | / pub fn next(&mut self) -> Option<(&[u8], &TSSTable::Value, A::State)> {
390 | | if self.advance() {
391 | | let st = self.current_state.clone().unwrap();
392 | | Some((self.key(), self.value(), st))
... |
396 | | }
| |_____^
|
= help: consider implementing the trait `std::iter::Iterator` or choosing a less ambiguous method name
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#should_implement_trait
= note: `#[warn(clippy::should_implement_trait)]` on by default
|
|
this can be `std::io::Error::other(_)`:
sstable/src/dictionary.rs#L309
warning: this can be `std::io::Error::other(_)`
--> sstable/src/dictionary.rs:309:28
|
309 | return Err(io::Error::new(
| ____________________________^
310 | | io::ErrorKind::Other,
311 | | format!("Unsupported sstable version, expected one of [2, 3], found {version}"),
312 | | ))
| |_________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
= note: `#[warn(clippy::io_other_error)]` on by default
help: use `std::io::Error::other`
|
309 ~ return Err(io::Error::other(
310 ~ format!("Unsupported sstable version, expected one of [2, 3], found {version}"),
|
|
|
variables can be used directly in the `format!` string:
stacker/src/fastcpy.rs#L12
warning: variables can be used directly in the `format!` string
--> stacker/src/fastcpy.rs:12:9
|
12 | / panic!(
13 | | "source slice length ({}) does not match destination slice length ({})",
14 | | src_len, dst_len,
15 | | );
| |_________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
= note: `#[warn(clippy::uninlined_format_args)]` on by default
|
|
calling .bytes() is very inefficient when data is not in memory:
common/src/vint.rs#L226
warning: calling .bytes() is very inefficient when data is not in memory
--> common/src/vint.rs:226:25
|
226 | let mut bytes = reader.bytes();
| ^^^^^^^^^^^^^^
|
= help: consider using `BufReader`
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unbuffered_bytes
|
|
calling .bytes() is very inefficient when data is not in memory:
common/src/vint.rs#L62
warning: calling .bytes() is very inefficient when data is not in memory
--> common/src/vint.rs:62:25
|
62 | let mut bytes = reader.bytes();
| ^^^^^^^^^^^^^^
|
= help: consider using `BufReader`
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unbuffered_bytes
|
|
calling .bytes() is very inefficient when data is not in memory:
common/src/vint.rs#L32
warning: calling .bytes() is very inefficient when data is not in memory
--> common/src/vint.rs:32:25
|
32 | let mut bytes = reader.bytes();
| ^^^^^^^^^^^^^^
|
= help: consider using `BufReader`
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unbuffered_bytes
= note: `#[warn(clippy::unbuffered_bytes)]` on by default
|
|
this can be `std::io::Error::other(_)`:
common/src/file_slice.rs#L351
warning: this can be `std::io::Error::other(_)`
--> common/src/file_slice.rs:351:27
|
351 | Err(e) => Err(io::Error::new(io::ErrorKind::Other, e.to_string())),
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error
= note: `#[warn(clippy::io_other_error)]` on by default
help: use `std::io::Error::other`
|
351 - Err(e) => Err(io::Error::new(io::ErrorKind::Other, e.to_string())),
351 + Err(e) => Err(io::Error::other(e.to_string())),
|
|
|
manually reimplementing `div_ceil`:
common/src/bitset.rs#L186
warning: manually reimplementing `div_ceil`
--> common/src/bitset.rs:186:5
|
186 | (max_val + 63u32) / 64u32
| ^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using `.div_ceil()`: `max_val.div_ceil(64u32)`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_div_ceil
= note: `#[warn(clippy::manual_div_ceil)]` on by default
|
|
manually reimplementing `div_ceil`:
bitpacker/src/bitpacker.rs#L141
warning: manually reimplementing `div_ceil`
--> bitpacker/src/bitpacker.rs:141:29
|
141 | let end_byte_read = (end_bit_read + 7) / 8;
| ^^^^^^^^^^^^^^^^^^^^^^ help: consider using `.div_ceil()`: `end_bit_read.div_ceil(8)`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_div_ceil
|
|
check
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
check
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
check
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
check
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
check
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
check
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
check
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
check
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
test-all
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
test-all
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
test-all
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
test-all
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
test-quickwit
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
test-quickwit
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
test-quickwit
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
test-quickwit
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|