expand: replace getopts with clap

expand has one odd behavior that allows two format for tabstop

From expand --help
```
-t, --tabs=N     have tabs N characters apart, not 8
-t, --tabs=LIST  use comma separated list of tab positions
```

This patch use one `value_name("N, LIST")` for tabstop and
deal with above behavior in `parse_tabstop`.

Close #1795

src/uu/expand/Cargo.toml

@@ -15,7 +15,7 @@ edition = "2018"
 path = "src/expand.rs"
 
 [dependencies]
-getopts = "0.2.18"
+clap = "2.33"
 unicode-width = "0.1.5"
 uucore = { version=">=0.0.7", package="uucore", path="../../uucore" }
 uucore_procs = { version=">=0.0.5", package="uucore_procs", path="../../uucore_procs" }

src/uu/expand/src/expand.rs

@@ -12,19 +12,25 @@
 #[macro_use]
 extern crate uucore;
 
+use clap::{App, Arg, ArgMatches};
 use std::fs::File;
 use std::io::{stdin, stdout, BufRead, BufReader, BufWriter, Read, Write};
 use std::iter::repeat;
 use std::str::from_utf8;
 use unicode_width::UnicodeWidthChar;
 
-static SYNTAX: &str = "[OPTION]... [FILE]...";
+static VERSION: &str = env!("CARGO_PKG_VERSION");
-static SUMMARY: &str = "Convert tabs in each FILE to spaces, writing to standard output.
+static ABOUT: &str = "Convert tabs in each FILE to spaces, writing to standard output.
  With no FILE, or when FILE is -, read standard input.";
+
 static LONG_HELP: &str = "";
 
 static DEFAULT_TABSTOP: usize = 8;
 
+fn get_usage() -> String {
+    format!("{0} [OPTION]... [FILE]...", executable!())
+}
+
 fn tabstops_parse(s: String) -> Vec<usize> {
     let words = s.split(',');
 
@@ -58,14 +64,14 @@ struct Options {
 }
 
 impl Options {
-    fn new(matches: getopts::Matches) -> Options {
+    fn new(matches: &ArgMatches) -> Options {
-        let tabstops = match matches.opt_str("t") {
+        let tabstops = match matches.value_of("tabstops") {
+            Some(s) => tabstops_parse(s.to_string()),
             None => vec![DEFAULT_TABSTOP],
-            Some(s) => tabstops_parse(s),
         };
 
-        let iflag = matches.opt_present("i");
+        let iflag = matches.is_present("iflag");
-        let uflag = !matches.opt_present("U");
+        let uflag = !matches.is_present("uflag");
 
         // avoid allocations when dumping out long sequences of spaces
         // by precomputing the longest string of spaces we will ever need
@@ -80,10 +86,9 @@ impl Options {
             .unwrap(); // length of tabstops is guaranteed >= 1
         let tspaces = repeat(' ').take(nspaces).collect();
 
-        let files = if matches.free.is_empty() {
+        let files: Vec<String> = match matches.values_of("files") {
-            vec!["-".to_owned()]
+            Some(s) => s.map(|v| v.to_string()).collect(),
-        } else {
+            None => vec!["-".to_owned()],
-            matches.free
         };
 
         Options {
@@ -97,31 +102,34 @@ impl Options {
 }
 
 pub fn uumain(args: impl uucore::Args) -> i32 {
-    let args = args.collect_str();
+    let usage = get_usage();
-
+    let matches = App::new(executable!())
-    let matches = app!(SYNTAX, SUMMARY, LONG_HELP)
+        .version(VERSION)
-        .optflag("i", "initial", "do not convert tabs after non blanks")
+        .about(ABOUT)
-        .optopt(
+        .usage(&usage[..])
-            "t",
+        .after_help(LONG_HELP)
-            "tabs",
+        .arg(
-            "have tabs NUMBER characters apart, not 8",
+            Arg::with_name("iflag")
-            "NUMBER",
+                .long("initial")
+                .short("i")
+                .help("do not convert tabs after non blanks"),
         )
-        .optopt(
+        .arg(
-            "t",
+            Arg::with_name("tabstops")
-            "tabs",
+                .long("tabs")
-            "use comma separated list of explicit tab positions",
+                .short("t")
-            "LIST",
+                .value_name("N, LIST")
+                .takes_value(true)
+                .help("have tabs N characters apart, not 8 or use comma separated list of explicit tab positions"),
         )
-        .optflag(
+        .arg(
-            "U",
+            Arg::with_name("uflags").long("no-utf8").short("U").help("interpret input file as 8-bit ASCII rather than UTF-8"),
-            "no-utf8",
+        ).arg(
-            "interpret input file as 8-bit ASCII rather than UTF-8",
+            Arg::with_name("files").multiple(true).hidden(true).takes_value(true)
         )
-        .parse(args);
+        .get_matches_from(args);
-
-    expand(Options::new(matches));
 
+    expand(Options::new(&matches));
     0
 }