app.rs now configures a given serial device with a given baud rate. If not defined, 4800bps are set by default, but this will be surely changed in the future.

I was messing up pretty badly with command line arguments, so they have been declared as constants in cmdline.rs, whereas CMD_LINE_ARGS is now private.
Uncommented  "extern crate serial". Question: why does need to be included on main?

1 files changed, 74 insertions, 13 deletions

diff --git a/src/app.rs b/src/app.rs
index 92490b1..a772989 100644
--- a/src/app.rs
+++ b/src/app.rs
@@ -6,11 +6,24 @@ use std::{string::String, io, collections::HashMap};
 /// if configured by command line parameters.
 pub fn app(arg_hash: HashMap<String, String>) -> io::Result<()> {
 
-    match arg_hash.get(&String::from("--tcp")) {
+    use cmdline;
+
+    let addr = arg_hash.get(&String::from(cmdline::TCP_ARG));
+
+    match addr {
         None => println!("No TCP address specified"),
         Some(addr) => setup_tcp(addr)?
     };
 
+    // Since this should never return None, always unwrap() it.
+    let port_name = arg_hash.get(&String::from(cmdline::PORT_NAME_ARG)).unwrap();
+
+    // Extract baud rate from command line parameters,
+    // but don't process it yet.
+    let baud_rate = arg_hash.get(&String::from(cmdline::BAUDRATE_ARG));
+
+    serial_comm(addr, port_name, baud_rate)?;
+
     Ok(())
 }
 
@@ -22,19 +35,67 @@ fn setup_tcp(tcp_addr : &String) -> io::Result<()> {
 
     println!("Awaiting for connection on address {}", tcp_addr);
 
-    for stream in listener.incoming() {
-        match stream {
-            Ok(s) => {
-                // do something with the TcpStream
-            }
-            Err(ref e) if e.kind() == io::ErrorKind::WouldBlock => {
-                // wait until network socket is ready, typically implemented
-                // via platform-specific APIs such as epoll or IOCP
-                continue;
+    //~ for stream in listener.incoming() {
+        //~ match stream {
+            //~ Ok(s) => {
+                //~ // do something with the TcpStream
+            //~ }
+            //~ Err(ref e) if e.kind() == io::ErrorKind::WouldBlock => {
+                //~ // wait until network socket is ready, typically implemented
+                //~ // via platform-specific APIs such as epoll or IOCP
+                //~ continue;
+            //~ }
+            //~ Err(e) => panic!("encountered IO error: {}", e),
+        //~ }
+    //~ }
+
+    Ok(())
+}
+
+fn serial_comm(addr : Option<&String>, port_name : &String, baud_rate : Option<&String>) -> io::Result<()> {
+    serial_init(addr, port_name, baud_rate).unwrap();
+
+    Ok(())
+}
+
+fn serial_init(addr : Option<&String>, port_name : &String, baud_rate : Option<&String>) -> io::Result<serial::SystemPort> {
+    use serial::SerialPort;
+
+    let port = serial::open(port_name);
+
+    let mut port_unwrapped;
+
+    let baud =  match baud_rate {
+        None => serial::Baud4800,
+        Some(b) => {
+            match b.parse() {
+                Ok(4800) => serial::Baud4800,
+                Ok(9600) => serial::Baud9600,
+                Err(_) | Ok(_) => return Err(io::Error::new(io::ErrorKind::Other, "Invalid baudrate")),
             }
-            Err(e) => panic!("encountered IO error: {}", e),
         }
-    }
+    };
 
-    Ok(())
+    match port {
+       Err(_) => {
+           println!();
+           return Err(io::Error::new(io::ErrorKind::NotFound, "Could not open serial device"));
+       },
+
+       Ok(p) => {
+           port_unwrapped = p;
+       }
+    };
+
+    let settings : serial::PortSettings = serial::PortSettings {
+                baud_rate: baud,
+                char_size: serial::Bits8,
+                parity: serial::ParityOdd,
+                stop_bits: serial::Stop1,
+                flow_control: serial::FlowNone
+            };
+
+    port_unwrapped.configure(&settings)?;
+
+    Ok(port_unwrapped)
 }