- 06 8月, 2017 11 次提交
-
-
由 Benjamin Sago 提交于
This merges in exa’s own new options parser, which has the following features: - You can specify an option twice and it’ll use the second one, making aliases usable for defaults (fixes #144) - Lets arguments be specified more than once (fixes #125) Strict mode is not done yet; I just wanted to merge this in because it’s been a while, and there’s work that needs to be done on master so I don’t want them drifting apart any further. It’s likely that you’ll find cases where multiple arguments doesn’t work or where the wrong value is being used. There aren’t tests for *everything* yet, and it still uses global environment variables. # Conflicts: # src/options/view.rs
-
由 Benjamin Sago 提交于
Now I just need to remember to update it. Was suggested in #197.
-
由 Benjamin Sago 提交于
The term_size crate introduced in #237 did things *slightly* differently than exa: it tried to get the terminal width of stdout, stderr, and stdin. This broke some tests that only redirected stdout.
-
由 Benjamin Sago 提交于
Use term_size crate to fetch terminal size.
-
由 Benjamin Sago 提交于
Update filetype.rs
-
由 Benjamin Sago 提交于
-
由 Benjamin Sago 提交于
Now it’s more like help. There aren’t any other fields in its struct at the moment, but there will be in the future (listing the features, and extremely colourful vanity mode)
-
由 Benjamin Sago 提交于
-
由 Benjamin Sago 提交于
Now, building the two result vectors doesn’t have to go through at least one field.
-
由 Benjamin Sago 提交于
Originally, both the matched flags and the list of free strings were returned from the parsing function and then passed around to every type that had a ‘deduce’ method. This worked, but the list of free strings was carried around with it, never used. Now, only the flags are passed around. They’re in a new struct which has the methods the Matches had. Both of Matches’s fields are now just data, and all of the methods on MatchedFlags don’t ignore any fields, so it’s more cohesive, at least I think that’s the word. Building up the MatchedFlags is a bit more annoying though because the vector is now hidden behind a field.
-
由 Benjamin Sago 提交于
-
- 05 8月, 2017 5 次提交
-
-
由 Benjamin Sago 提交于
-
由 Benjamin Sago 提交于
-
由 Benjamin Sago 提交于
One of the previous tests started to fail, because it was working when it shouldn’t have! It worked up until now because I forgot to flag --level as taking an argument, and “--level 4” still worked with 4 as a filename. So there’s now an early check for that functionality that got lost somewhere.
-
由 Benjamin Sago 提交于
-
由 PJB3005 提交于
Should be cross-platform, one step closer to Windows support.
-
- 04 8月, 2017 2 次提交
-
-
由 pskuza 提交于
Added a few video and audio extensions
-
由 Benjamin Sago 提交于
Testing that a recent PR to rust-users (https://github.com/ogham/rust-users/issues/12) didn’t cause any regressions after updating to v0.5.3 of it.
-
- 27 7月, 2017 8 次提交
-
-
由 Benjamin Sago 提交于
Apparently I forgot to give the --time flag an argument, and this wasn’t actually covered by any of the xtests! Well, it’s tested now. I’m not sure how to handle multiple --time arguments.
-
由 Benjamin Sago 提交于
-
由 Benjamin Sago 提交于
-
由 Benjamin Sago 提交于
Fixes #193. --all was treated the same as --all --all; now it’s treated differently.
-
由 Benjamin Sago 提交于
-
由 Benjamin Sago 提交于
**
-
由 Benjamin Sago 提交于
-
由 Benjamin Sago 提交于
This commit removes the dependency on the ‘getopts’ crate entirely, and re-writes all its uses to use the new options parser instead. As expected there are casualties galore: - We now need to collect the options into a vector at the start, so we can use references to them, knowing they’ll be stored *somewhere*. - Because OsString isn’t Display, its Debug impl gets used instead. (This is hopefully temporary) - Options that take values (such as ‘sort’ or ‘time-style’) now parse those values with ‘to_string_lossy’. The ‘lossy’ part means “I’m at a loss for what to do here” - Error messages got a lot worse, but “--tree --all --all” is now a special case of error rather than just another Misfire::Useless. - Some tests had to be re-written to deal with the fact that the parser works with references. - ParseError loses its lifetime and owns its contents, to avoid having to attach <'a> to Misfire. - The parser now takes an iterator instead of a slice. - OsStrings can’t be ‘match’ patterns, so the code devolves to using long Eq chains instead. - Make a change to the xtest that assumed an input argument with invalid UTF-8 in was always an error to stderr, when that now in fact works! - Fix a bug in Vagrant where ‘exa’ and ‘rexa’ didn’t properly escape filenames with spaces in.
-
- 24 7月, 2017 1 次提交
-
-
由 Benjamin Sago 提交于
This commit moves the definitions of Filter and DirAction from the options module to the fs module, but leaves the parts that actually have to do with option parsing alone. Now, the options module shouldn’t define any types that get used elsewhere in the program: it only adds functionality to types that already exist.
-
- 23 7月, 2017 1 次提交
-
-
由 Benjamin Sago 提交于
Even though these can’t actually be viewed with `cargo doc` yet, they’re still good to have around.
-
- 13 7月, 2017 9 次提交
-
-
由 Benjamin Sago 提交于
-
由 Benjamin Sago 提交于
Casualty here was that you can’t have static values reference one another directly, so the static args slice had to be turned into a slice *of references* rather than of values. No big deal, just have to write & a few more times.
-
由 Benjamin Sago 提交于
-
由 Benjamin Sago 提交于
-
由 Benjamin Sago 提交于
-
由 Benjamin Sago 提交于
-
由 Benjamin Sago 提交于
Every time looking up an argument fails, it returns an error. We might as well just move this into the lookup function.
-
由 Benjamin Sago 提交于
-
由 Benjamin Sago 提交于
-
- 12 7月, 2017 1 次提交
-
-
由 Benjamin Sago 提交于
All the tests pass, but only half the functionality is there right now.
-
- 10 7月, 2017 2 次提交
-
-
由 Benjamin Sago 提交于
This creates a new type that holds file extensions in preparation for #116. It doesn’t do anything yet, but it will!
-
由 Benjamin Sago 提交于
The FileExtensions in the FileName is now a reference to the one in the original FileStyle, which gets put there in the options module. This allows the extensions to be derived from the user, somehow, in the future when that part’s done.
-