[pbs-devel] applied: [PATCH backup v3 1/2] backup-proxy: avoid block in if condition

[Fabian Grünbichler]

On February 13, 2024 1:43 pm, Maximiliano Sandoval wrote:
> Fixes the clippy lint:
> 
> ```
> warning: in a `match` scrutinee, avoid complex blocks or closures with blocks; instead, move the block or closure higher and bind it with a `let`
>    --> src/bin/proxmox-backup-proxy.rs:874:58
>     |
> 874 |           let stats = match tokio::task::spawn_blocking(|| {
>     |  __________________________________________________________^
> 875 | |             let hoststats = collect_host_stats_sync();
> 876 | |             let (hostdisk, datastores) = collect_disk_stats_sync();
> 877 | |             Arc::new((hoststats, hostdisk, datastores))
> 878 | |         })
>     | |_________^
>     |
>     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#blocks_in_conditions
>     = note: `#[warn(clippy::blocks_in_conditions)]` on by default
> ```
> 
> Signed-off-by: Maximiliano Sandoval <m.sandoval at proxmox.com>
> ---
> Differences from v2:
>  - Rename stats_res to stats_fut
>  - Port other uses of single branched `match` to if-let
> 
>  src/bin/proxmox-backup-proxy.rs | 7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/src/bin/proxmox-backup-proxy.rs b/src/bin/proxmox-backup-proxy.rs
> index 9c49026b..a2c48f16 100644
> --- a/src/bin/proxmox-backup-proxy.rs
> +++ b/src/bin/proxmox-backup-proxy.rs
> @@ -871,13 +871,12 @@ async fn run_stat_generator() {
>      loop {
>          let delay_target = Instant::now() + Duration::from_secs(10);
>  
> -        let stats = match tokio::task::spawn_blocking(|| {
> +        let stats_fut = tokio::task::spawn_blocking(|| {
>              let hoststats = collect_host_stats_sync();
>              let (hostdisk, datastores) = collect_disk_stats_sync();
>              Arc::new((hoststats, hostdisk, datastores))
> -        })
> -        .await
> -        {
> +        });
> +        let stats = match stats_fut.await {
>              Ok(res) => res,
>              Err(err) => {
>                  log::error!("collecting host stats panicked: {err}");
> -- 
> 2.39.2
> 
> 
> 
> _______________________________________________
> pbs-devel mailing list
> pbs-devel at lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel
> 
> 
>