From 5ee4bb58cd55a77e0e10ca3d5867017b169f6be3 Mon Sep 17 00:00:00 2001 From: Floris Bruynooghe Date: Sun, 31 Jan 2021 12:30:36 +0100 Subject: [PATCH] Tidy up imports --- src/context.rs | 11 ++++------- src/events.rs | 4 ++-- src/scheduler.rs | 26 ++++++++++++++------------ 3 files changed, 20 insertions(+), 21 deletions(-) diff --git a/src/context.rs b/src/context.rs index 2bf0a2e66..61bdc0c5c 100644 --- a/src/context.rs +++ b/src/context.rs @@ -1,15 +1,13 @@ //! Context module +use std::collections::{BTreeMap, HashMap}; use std::ffi::OsString; use std::ops::Deref; -use std::{ - collections::{BTreeMap, HashMap}, - time::Instant, -}; +use std::time::{Instant, SystemTime}; use anyhow::{bail, ensure, Result}; use async_std::{ - channel::{bounded as channel, Receiver, Sender}, + channel::{self, Receiver, Sender}, path::{Path, PathBuf}, sync::{Arc, Mutex, RwLock}, task, @@ -27,7 +25,6 @@ use crate::message::{self, MsgId}; use crate::scheduler::Scheduler; use crate::securejoin::Bob; use crate::sql::Sql; -use std::time::SystemTime; #[derive(Clone, Debug)] pub struct Context { @@ -225,7 +222,7 @@ impl Context { s.ongoing_running = true; s.shall_stop_ongoing = false; - let (sender, receiver) = channel(1); + let (sender, receiver) = channel::bounded(1); s.cancel_sender = Some(sender); Ok(receiver) diff --git a/src/events.rs b/src/events.rs index 577d8abe6..d6abd8e3b 100644 --- a/src/events.rs +++ b/src/events.rs @@ -2,7 +2,7 @@ use std::ops::Deref; -use async_std::channel::{bounded as channel, Receiver, Sender, TrySendError}; +use async_std::channel::{self, Receiver, Sender, TrySendError}; use async_std::path::PathBuf; use strum::EnumProperty; @@ -18,7 +18,7 @@ pub struct Events { impl Default for Events { fn default() -> Self { - let (sender, receiver) = channel(1_000); + let (sender, receiver) = channel::bounded(1_000); Self { receiver, sender } } diff --git a/src/scheduler.rs b/src/scheduler.rs index b910c907c..35419baf4 100644 --- a/src/scheduler.rs +++ b/src/scheduler.rs @@ -1,14 +1,16 @@ use async_std::prelude::*; use async_std::{ - channel::{bounded as channel, Receiver, Sender}, + channel::{self, Receiver, Sender}, task, }; +use crate::config::Config; use crate::context::Context; use crate::dc_tools::maybe_add_time_based_warnings; use crate::imap::Imap; use crate::job::{self, Thread}; -use crate::{config::Config, message::MsgId, smtp::Smtp}; +use crate::message::MsgId; +use crate::smtp::Smtp; pub(crate) struct StopToken; @@ -277,12 +279,12 @@ impl Scheduler { let (smtp, smtp_handlers) = SmtpConnectionState::new(); let (inbox, inbox_handlers) = ImapConnectionState::new(); - let (inbox_start_send, inbox_start_recv) = channel(1); - let (mvbox_start_send, mvbox_start_recv) = channel(1); + let (inbox_start_send, inbox_start_recv) = channel::bounded(1); + let (mvbox_start_send, mvbox_start_recv) = channel::bounded(1); let mut mvbox_handle = None; - let (sentbox_start_send, sentbox_start_recv) = channel(1); + let (sentbox_start_send, sentbox_start_recv) = channel::bounded(1); let mut sentbox_handle = None; - let (smtp_start_send, smtp_start_recv) = channel(1); + let (smtp_start_send, smtp_start_recv) = channel::bounded(1); let inbox_handle = { let ctx = ctx.clone(); @@ -503,9 +505,9 @@ pub(crate) struct SmtpConnectionState { impl SmtpConnectionState { fn new() -> (Self, SmtpConnectionHandlers) { - let (stop_sender, stop_receiver) = channel(1); - let (shutdown_sender, shutdown_receiver) = channel(1); - let (idle_interrupt_sender, idle_interrupt_receiver) = channel(1); + let (stop_sender, stop_receiver) = channel::bounded(1); + let (shutdown_sender, shutdown_receiver) = channel::bounded(1); + let (idle_interrupt_sender, idle_interrupt_receiver) = channel::bounded(1); let handlers = SmtpConnectionHandlers { connection: Smtp::new(), @@ -551,9 +553,9 @@ pub(crate) struct ImapConnectionState { impl ImapConnectionState { /// Construct a new connection. fn new() -> (Self, ImapConnectionHandlers) { - let (stop_sender, stop_receiver) = channel(1); - let (shutdown_sender, shutdown_receiver) = channel(1); - let (idle_interrupt_sender, idle_interrupt_receiver) = channel(1); + let (stop_sender, stop_receiver) = channel::bounded(1); + let (shutdown_sender, shutdown_receiver) = channel::bounded(1); + let (idle_interrupt_sender, idle_interrupt_receiver) = channel::bounded(1); let handlers = ImapConnectionHandlers { connection: Imap::new(idle_interrupt_receiver),