Skip to content

add clippy linting CI target #1051

add clippy linting CI target

add clippy linting CI target #1051

Triggered via pull request November 30, 2023 06:54
Status Failure
Total duration 5m 40s
Artifacts

ci.yml

on: pull_request
Matrix: build
Fit to window
Zoom out
Zoom in

Annotations

1 error and 8 warnings
lint
Clippy had exited with the 101 exit code
the following explicit lifetimes could be elided: 'b: capnp/src/data_list.rs#L65
warning: the following explicit lifetimes could be elided: 'b --> capnp/src/data_list.rs:65:21 | 65 | pub fn reborrow<'b>(&'b self) -> Reader<'b> { | ^^ ^^ ^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_lifetimes = note: `#[warn(clippy::needless_lifetimes)]` on by default help: elide the lifetimes | 65 - pub fn reborrow<'b>(&'b self) -> Reader<'b> { 65 + pub fn reborrow(&self) -> Reader<'_> { |
unsafe function's docs miss `# Safety` section: capnp/src/private/arena.rs#L40
warning: unsafe function's docs miss `# Safety` section --> capnp/src/private/arena.rs:40:5 | 40 | / unsafe fn check_offset( 41 | | &self, 42 | | segment_id: u32, 43 | | start: *const u8, 44 | | offset_in_words: i32, 45 | | ) -> Result<*const u8>; | |___________________________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#missing_safety_doc = note: `#[warn(clippy::missing_safety_doc)]` on by default
very complex type used. Consider factoring parts into `type` definitions: capnp/src/private/capability.rs#L40
warning: very complex type used. Consider factoring parts into `type` definitions --> capnp/src/private/capability.rs:40:10 | 40 | ) -> Option<( | __________^ 41 | | u32, 42 | | crate::capability::Promise<(), crate::Error>, 43 | | Box<dyn PipelineHook>, 44 | | )>; | |______^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#type_complexity = note: `#[warn(clippy::type_complexity)]` on by default
unsafe function's docs miss `# Safety` section: capnp/src/private/layout.rs#L162
warning: unsafe function's docs miss `# Safety` section --> capnp/src/private/layout.rs:162:5 | 162 | pub unsafe fn target(ptr: *const Self) -> *const u8 { | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#missing_safety_doc
this function has too many arguments (10/7): capnp/src/private/layout.rs#L2149
warning: this function has too many arguments (10/7) --> capnp/src/private/layout.rs:2149:5 | 2149 | / pub unsafe fn copy_pointer( 2150 | | dst_arena: &mut dyn BuilderArena, 2151 | | dst_segment_id: u32, 2152 | | dst_cap_table: CapTableBuilder, ... | 2159 | | canonicalize: bool, 2160 | | ) -> Result<SegmentAnd<*mut u8>> { | |____________________________________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments = note: `#[warn(clippy::too_many_arguments)]` on by default
unsafe function's docs miss `# Safety` section: capnp/src/private/layout.rs#L2885
warning: unsafe function's docs miss `# Safety` section --> capnp/src/private/layout.rs:2885:5 | 2885 | pub unsafe fn get_root_unchecked<'b>(location: *const u8) -> PointerReader<'b> { | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#missing_safety_doc
unsafe function's docs miss `# Safety` section: capnp/src/private/layout.rs#L3888
warning: unsafe function's docs miss `# Safety` section --> capnp/src/private/layout.rs:3888:5 | 3888 | / pub unsafe fn is_canonical( 3889 | | &self, 3890 | | read_head: &Cell<*const u8>, 3891 | | reff: *const WirePointer, 3892 | | ) -> Result<bool> { | |_____________________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#missing_safety_doc
the following explicit lifetimes could be elided: 'b: capnp/src/text_list.rs#L65
warning: the following explicit lifetimes could be elided: 'b --> capnp/src/text_list.rs:65:21 | 65 | pub fn reborrow<'b>(&'b self) -> Reader<'b> { | ^^ ^^ ^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_lifetimes help: elide the lifetimes | 65 - pub fn reborrow<'b>(&'b self) -> Reader<'b> { 65 + pub fn reborrow(&self) -> Reader<'_> { |