Skip to content

Implement streaming #1299

Implement streaming

Implement streaming #1299

Triggered via pull request September 16, 2024 18:36
@dwrenshadwrensha
synchronize #517
streaming
Status Failure
Total duration 6m 30s
Artifacts

ci.yml

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

Annotations

1 error and 10 warnings
formatting
Process completed with exit code 1.
type parameter `T` goes unused in function definition: target/debug/build/fill_random_values-dfe7938719c32a86/out/fill_capnp.rs#L2431
warning: type parameter `T` goes unused in function definition --> /home/runner/work/capnproto-rust/capnproto-rust/target/debug/build/fill_random_values-dfe7938719c32a86/out/fill_capnp.rs:2431:28 | 2431 | pub fn get_field_types<T>(index: u16) -> ::capnp::introspect::Type where T: ::capnp::traits::Owned { | ^ | = help: consider removing the parameter = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#extra_unused_type_parameters = note: `#[warn(clippy::extra_unused_type_parameters)]` on by default
type parameter `T` goes unused in function definition: target/debug/build/fill_random_values-dfe7938719c32a86/out/fill_capnp.rs#L2434
warning: type parameter `T` goes unused in function definition --> /home/runner/work/capnproto-rust/capnproto-rust/target/debug/build/fill_random_values-dfe7938719c32a86/out/fill_capnp.rs:2434:33 | 2434 | pub fn get_annotation_types<T>(child_index: Option<u16>, index: u32) -> ::capnp::introspect::Type where T: ::capnp::traits::Owned { | ^ | = help: consider removing the parameter = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#extra_unused_type_parameters
unneeded `return` statement: capnp-rpc/src/flow_control.rs#L130
warning: unneeded `return` statement --> capnp-rpc/src/flow_control.rs:130:21 | 130 | return Promise::ok(()); | ^^^^^^^^^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return = note: `#[warn(clippy::needless_return)]` on by default help: remove `return` | 130 - return Promise::ok(()); 130 + Promise::ok(()) |
unneeded `return` statement: capnp-rpc/src/flow_control.rs#L134
warning: unneeded `return` statement --> capnp-rpc/src/flow_control.rs:134:21 | 134 | / return Promise::from_future(async { 135 | | match rcv.await { 136 | | Ok(r) => r, 137 | | Err(e) => Err(crate::canceled_to_error(e)), 138 | | } 139 | | }); | |______________________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return help: remove `return` | 134 ~ Promise::from_future(async { 135 + match rcv.await { 136 + Ok(r) => r, 137 + Err(e) => Err(crate::canceled_to_error(e)), 138 + } 139 ~ }) |
unneeded `return` statement: capnp-rpc/src/flow_control.rs#L142
warning: unneeded `return` statement --> capnp-rpc/src/flow_control.rs:142:37 | 142 | State::Failed(ref e) => return Promise::err(e.clone()), | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return help: remove `return` | 142 | State::Failed(ref e) => Promise::err(e.clone()), | ~~~~~~~~~~~~~~~~~~~~~~~
returning the result of a `let` binding from a block: capnp-rpc/src/rpc.rs#L1864
warning: returning the result of a `let` binding from a block --> capnp-rpc/src/rpc.rs:1864:9 | 1857 | / let flow_promise = flow.send( 1858 | | message, 1859 | | Promise::from_future(async move { 1860 | | let _ = promise.await?; 1861 | | Ok(()) 1862 | | }), 1863 | | ); | |__________- unnecessary `let` binding 1864 | flow_promise | ^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_and_return = note: `#[warn(clippy::let_and_return)]` on by default help: return the expression directly | 1857 ~ 1858 ~ flow.send( 1859 + message, 1860 + Promise::from_future(async move { 1861 + let _ = promise.await?; 1862 + Ok(()) 1863 + }), 1864 + ) |
this match could be replaced by its body itself: target/debug/build/capnp-rpc-test-ac09535fe77af25d/out/test_capnp.rs#L6419
warning: this match could be replaced by its body itself --> /home/runner/work/capnproto-rust/capnproto-rust/target/debug/build/capnp-rpc-test-ac09535fe77af25d/out/test_capnp.rs:6419:7 | 6419 | / match method_id { 6420 | | _ => { ::capnp::capability::DispatchCallResult::new(::capnp::capability::Promise::err(::capnp::Error::unimplemented("Method not i... 6421 | | } | |_______^ help: consider using the match body instead: `::capnp::capability::DispatchCallResult::new(::capnp::capability::Promise::err(::capnp::Error::unimplemented("Method not implemented.".to_string())), false)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#match_single_binding = note: `#[warn(clippy::match_single_binding)]` on by default
type parameters go unused in function definition: Foo, Bar: target/debug/build/capnpc-test-edition-2018-aba021fcc757334a/out/test_capnp.rs#L19583
warning: type parameters go unused in function definition: Foo, Bar --> /home/runner/work/capnproto-rust/capnproto-rust/target/debug/build/capnpc-test-edition-2018-aba021fcc757334a/out/test_capnp.rs:19583:33 | 19583 | pub fn get_annotation_types<Foo,Bar>(child_index: Option<u16>, index: u32) -> ::capnp::introspect::Type where Foo: ::capnp::traits::O... | ^^^ ^^^ | = help: consider removing the parameters = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#extra_unused_type_parameters = note: `#[warn(clippy::extra_unused_type_parameters)]` on by default
type parameters go unused in function definition: Foo, Bar: target/debug/build/capnpc-test-edition-2018-aba021fcc757334a/out/test_capnp.rs#L19854
warning: type parameters go unused in function definition: Foo, Bar --> /home/runner/work/capnproto-rust/capnproto-rust/target/debug/build/capnpc-test-edition-2018-aba021fcc757334a/out/test_capnp.rs:19854:35 | 19854 | pub fn get_annotation_types<Foo,Bar>(child_index: Option<u16>, index: u32) -> ::capnp::introspect::Type where Foo: ::capnp::traits:... | ^^^ ^^^ | = help: consider removing the parameters = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#extra_unused_type_parameters
type parameters go unused in function definition: Foo, Bar, Baz: target/debug/build/capnpc-test-edition-2018-aba021fcc757334a/out/test_capnp.rs#L20226
warning: type parameters go unused in function definition: Foo, Bar, Baz --> /home/runner/work/capnproto-rust/capnproto-rust/target/debug/build/capnpc-test-edition-2018-aba021fcc757334a/out/test_capnp.rs:20226:35 | 20226 | pub fn get_annotation_types<Foo,Bar,Baz>(child_index: Option<u16>, index: u32) -> ::capnp::introspect::Type where Foo: ::capnp::tra... | ^^^ ^^^ ^^^ | = help: consider removing the parameters = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#extra_unused_type_parameters