Merge pull request #555 from JakeRoggenbuck/add-forbid-unsafe_code-st…

…atic-check-attr

src/args.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 use std::{thread, time};
 use structopt::StructOpt;
 

src/bin/main.rs

@@ -23,6 +23,7 @@
 // https://github.com/autoclockspeed
 
 #![allow(clippy::uninlined_format_args)]
+#![forbid(unsafe_code)]
 
 use autoclockspeed::args::parse_args;
 use autoclockspeed::config::get_config;

src/config.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 use super::daemon::State;
 use super::{print_done, warn_user};
 use crate::print_error;

src/cpu.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 use efcl::{bold, color, Color};
 use rand::Rng;
 use std::fmt;

src/csv.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 //! # CSV logging
 //! This file is reponsible for logging data about the running daemon.
 //! CSV Logging is by default disabled, however it can be enabled by passing a parameter to the

src/daemon.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 //! The daemon handles the running auto clock speed instance
 //!
 //! # Modes
@@ -57,7 +58,6 @@ use crate::system::{
     check_available_governors, check_cpu_freq, check_cpu_temperature, check_cpu_usage,
     get_highest_temp, inside_docker, inside_wsl, list_cpus,
 };
-use crate::terminal::terminal_width;
 use crate::warn_user;
 
 /// Describes the state of the machine
@@ -447,7 +447,7 @@ impl Checker for Daemon {
             self.graph = self.grapher.update_one(&mut self.grapher.vals.clone());
         }
 
-        let term_width = terminal_width();
+        let (term_width, _term_height) = termion::terminal_size().unwrap();
 
         // Render two sections of the output
         // Rendering before screen is cleared reduces the time between clear and print
@@ -473,7 +473,12 @@ impl Checker for Daemon {
         // Print all pre-rendered items
         print!("{}", preprint);
 
-        print_turbo_status(cores, self.settings.animation, term_width, delay_in_millis);
+        print_turbo_status(
+            cores,
+            self.settings.animation,
+            term_width.into(),
+            delay_in_millis,
+        );
 
         // Print more pre-rendered items
         print!("{}", postprint);

src/display.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 use efcl::{color, Color};
 use std::fmt::Display;
 use std::thread;

src/error.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 use std::fmt;
 
 const BUG_REPORT: &str = "\nGive us a bug report by opening an issue at

src/gov.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 use std::fmt;
 
 /// Governor

src/graph.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 use rasciigraph::{plot, Config};
 use std::default::Default;
 use std::fmt;

src/interactive.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 use super::config::{get_config, Config};
 use super::interface::{DaemonControl, DaemonController, Get, Getter, Interface, Set, Setter};
 use super::settings::Settings;

src/interface.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 ///! The interface creates an API for getting and setting values with ACS
 ///!
 ///! Note: this is not an API for all of the functions in ACS, it's only the ones that get used in

src/lib.rs

@@ -17,5 +17,4 @@ pub mod settings;
 pub mod setup;
 pub mod sysfs;
 pub mod system;
-pub mod terminal;
 pub mod thermal;

src/network.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 use serde::{Deserialize, Serialize};
 
 use crate::logger::Log;

src/network/hook.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 use crate::network::{log_to_daemon, logger, Daemon, Packet};
 use std::io::{BufRead, BufReader, Write};
 use std::os::unix::net::UnixStream;

src/network/listen.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 use crate::logger;
 use crate::logger::Interface;
 use crate::network::log_to_daemon;

src/network/send.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 use std::{
     io::{BufRead, BufReader, Write},
     os::unix::net::UnixStream,

src/power.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 use std::fs;
 use std::path::Path;
 

src/power/battery.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 use crate::error::Error;
 use crate::sysfs;
 use std::any::Any;

src/power/lid.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 use crate::error::Error;
 use std::cmp::PartialEq;
 use std::fmt;

src/proc.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 use std::fs::read_to_string;
 
 use crate::error::Error;

src/settings.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 use super::graph::GraphType;
 use std::default::Default;
 

src/setup.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 use super::config::config_dir_exists;
 use super::system::{inside_docker, inside_wsl};
 use super::warn_user;

src/sysfs.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 use crate::error::Error;
 use globset::Glob;
 use std::fs;

src/system.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 use cached::proc_macro::once;
 use std::fs::{self, read_dir};
 use std::path::Path;

src/thermal.rs

@@ -1,3 +1,4 @@
+#![forbid(unsafe_code)]
 use super::system::{read_int, read_str};
 use crate::error::Error;
 use efcl::{color, Color};