Commit f9a4f23e authored by trinity-1686a's avatar trinity-1686a
Browse files

remove most usage of SystemTime::now

parent c98a2f6f

clippy.toml

0 → 100644
+3 −0
Original line number Diff line number Diff line 
disallowed-methods = [

    { path = "std::time::SystemTime::now", reason = "prefere using SleepProvider::wallclock instead when possible" },

]

crates/arti-bench/src/main.rs

+1 −0
Original line number Diff line number Diff line
@@ -35,6 +35,7 @@ 
// This file uses `unwrap()` a fair deal, but this is fine in test/bench code

// because it's OK if tests and benchmarks simply crash if things go wrong.

#![allow(clippy::unwrap_used)]

#![allow(clippy::disallowed_methods)]



use anyhow::{anyhow, Result};

use arti_client::{TorAddr, TorClient, TorClientConfig};

crates/arti-client/src/status.rs

+5 −1
Original line number Diff line number Diff line
@@ -36,7 +36,10 @@ impl BootstrapStatus { 
    /// 0 is defined as "just started"; 1 is defined as "ready to use."

    pub fn as_frac(&self) -> f32 {

        // Coefficients chosen arbitrarily.

        self.conn_status.frac() * 0.15 + self.dir_status.frac_at(SystemTime::now()) * 0.85

        #[allow(clippy::disallowed_methods)]

        let res =

            self.conn_status.frac() * 0.15 + self.dir_status.frac_at(SystemTime::now()) * 0.85;

        res

    }



    /// Return true if the status indicates that the client is ready for
@@ -45,6 +48,7 @@ impl BootstrapStatus { 
    /// For the purposes of this function, the client is "ready for traffic" if,

    /// as far as we know, we can start acting on a new client request immediately.

    pub fn ready_for_traffic(&self) -> bool {

        #[allow(clippy::disallowed_methods)]

        let now = SystemTime::now();

        self.conn_status.usable() && self.dir_status.usable_at(now)

    }

crates/tor-checkable/src/lib.rs

+1 −0
Original line number Diff line number Diff line
@@ -120,6 +120,7 @@ pub trait Timebound<T>: Sized { 


    /// Unwrap this Timebound object if it is valid now.

    fn check_valid_now(self) -> Result<T, Self::Error> {

        #[allow(clippy::disallowed_methods)]

        self.check_valid_at(&time::SystemTime::now())

    }

crates/tor-circmgr/src/build.rs

+10 −0
Original line number Diff line number Diff line
@@ -244,6 +244,11 @@ impl<R: Runtime, C: Buildable + Sync + Send + 'static> Builder<R, C> { 
            Err(e) => Err(e),

        }

    }



    /// Return a reference to this Builder runtime.

    pub(crate) fn runtime(&self) -> &R {

        &self.runtime

    }

}



/// A factory object to build circuits.
@@ -367,6 +372,11 @@ impl<R: Runtime> CircuitBuilder<R> { 
    pub(crate) fn guardmgr(&self) -> &tor_guardmgr::GuardMgr<R> {

        &self.guardmgr

    }



    /// REturn a reference to this builder's runtime

    pub(crate) fn runtime(&self) -> &R {

        self.builder.runtime()

    }

}



/// Helper function: spawn a future as a background task, and run it with