Memory safety enhancements

Cargo.toml

@@ -14,9 +14,11 @@ telemetry = ["winapi"]
 bitflags = "1.2"
 chrono = "0.4"
 encoding_rs = "0.8"
+libc = { version = "0.2", optional = true }
 serde = {version = "1.0", features = ["derive"] }
 serde_yaml = "0.8"
-winapi = {version = "0.3.9", features = ["std","memoryapi","winnt","errhandlingapi","synchapi","handleapi"], optional = true }
+thiserror = "1.0"
+winapi = {version = "0.3", features = ["std","memoryapi","winnt","errhandlingapi","synchapi","handleapi"], optional = true }
 
 [package.metadata.docs.rs]
 default-target = "x86_64-pc-windows-msvc"

examples/dump_sample.rs

@@ -3,7 +3,7 @@ use serde_yaml::to_string;
 
 pub fn main() {
     let conn = Connection::new().expect("Unable to open telemetry");
-    let telem = conn.telemetry().expect("Telem");
+    let telem = conn.telemetry();
 
     print!("{}", to_string(&telem.all()).unwrap());
 }

examples/get_telemetry.rs

@@ -4,8 +4,7 @@ use std::convert::TryInto;
 use std::time::Duration;
 
 pub fn main() {
-    let mut conn = Connection::new().expect("Unable to open telemetry");
-
+    let conn = Connection::new().expect("Unable to open telemetry");
     let session = conn.session_info().expect("Invalid Session");
 
     let shift_up_rpm = session.drivers.shift_light_shift_rpm;

examples/view_session.rs

@@ -1,7 +1,7 @@
 use iracing::telemetry::Connection;
 
 pub fn main() {
-    let mut conn = Connection::new().expect("Unable to open telemetry");
+    let conn = Connection::new().expect("Unable to open telemetry");
     let session = conn.session_info().expect("Invalid session data");
 
     print!("{:#?}", session);

src/replay.rs

@@ -3,7 +3,6 @@ use std::io;
 use std::io::Read;
 use std::io::Result as IOResult;
 use std::io::{Error as IOError, ErrorKind};
-use std::u32;
 
 /// Magic number found at the start of replay files
 pub const FILE_MAGIC: &[u8] = b"YLPR";
@@ -170,7 +169,7 @@ fn read_str<R: Read>(mut reader: R, length: usize) -> IOResult<String> {
         .position(|&b| b == 0)
         .expect("Given string does not terminate within given length");
 
-    Ok(String::from_utf8((&raw_string_bytes[..nul]).to_vec()).unwrap())
+    Ok(String::from_utf8((raw_string_bytes[..nul]).to_vec()).unwrap())
 }
 
 impl<R: Read> Replay<R> {