aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* config clang-format to ignore oclint pragmasGravatar Kurtis Rader2016-05-04
|
* config oclint to allow longer var namesGravatar Kurtis Rader2016-05-04
|
* another oclint rule to ignoreGravatar Kurtis Rader2016-05-04
|
* lint cleanup: eliminate "redundant" errorsGravatar Kurtis Rader2016-05-04
| | | | | This removes some pointless parentheses but the primary focus is removing redundancies like unnecessary "else" clauses.
* Remove useless case completionsGravatar Fabian Homborg2016-05-04
| | | | | It doesn't take options and what it takes (arbitrary strings) we can't sensibly complete.
* eliminate "useless parentheses" lint errorsGravatar Kurtis Rader2016-05-03
| | | | | | | Some `oclint` errors regarding "useless parentheses" are meaningfull. But the vast majority are bogus in as much as removing the parentheses reduces readability. So fix a few of the egregious uses and otherwise suppress that error.
* limit `make style-all` to fish scripts in shareGravatar Kurtis Rader2016-05-03
| | | | | | | | | I noticed that if I've previous done `make test` that a subsequent `make style-all` attempts to restyle all the fish scripts in the *test* directory. Those files are transient and not part of the git repository. Limit restyling all fish scripts just to those in the *share* directory tree. There are a couple elsewhere in the repo (e.g., *build_tools*) but they can be handled on an individual basis.
* add floating point output to `math` commandGravatar Kurtis Rader2016-05-03
| | | | | | | | | | | This makes it easy for the user to request floating point output with the desired number of digits after the decimal point (not to be confused with significant digits). Note that this is just a thin wrapper so someone can say `math -s3 10 / 3` rather than `math "scale=3; 10 /3"`. Resolves #1643
* add missing fallback declarationsGravatar Kurtis Rader2016-05-03
| | | | Fixes #2993
* restyle switch blocks to match project styleGravatar Kurtis Rader2016-05-03
| | | | | | | | | | | | | | | I missed restyling a few "switch" blocks to make them consistent with the rest of the code base. This fixes that oversight. This should be the final step in restyling the C++ code to have a consistent style. This also includes a few trivial cleanups elsewhere. I also missed restyling the "complete" module when working my way from a to z so this final change includes restyling that module. Total lint errors decreased 36%. Cppcheck errors went from 47 to 24. Oclint P2 errors went from 819 to 778. Oclint P3 errors went from 3252 to 1842. Resolves #2902.
* restyle remaining modules to match project styleGravatar Kurtis Rader2016-05-03
| | | | | | | | | For this change I decided to bundle the remaining modules that need to be resytyled because only two were large enough to warrant doing on their own. Reduces lint errors from 225 to 162 (-28%). Line count from 3073 to 2465 (-20%). Another step in resolving issue #2902.
* restyle utf8 module to match project styleGravatar Kurtis Rader2016-05-03
| | | | | | Reduces lint errors from 63 to 57 (-10%). Line count from 518 to 418 (-19%). Another step in resolving issue #2902.
* restyle tokenizer module to match project styleGravatar Kurtis Rader2016-05-03
| | | | | | Reduces lint errors from 70 to 46 (-34%). Line count from 1158 to 936 (-19%). Another step in resolving issue #2902.
* restyle signal module to match project styleGravatar Kurtis Rader2016-05-03
| | | | | | Reduces lint errors from 15 to 15 (-0%). Line count from 754 to 438 (-42%). Another step in resolving issue #2902.
* restyle sanity & screen module to match project styleGravatar Kurtis Rader2016-05-03
| | | | | | Reduces lint errors from 163 to 112 (-31%). Line count from 1866 to 1493 (-20%). Another step in resolving issue #2902.
* restyle reader module to match project styleGravatar Kurtis Rader2016-05-03
| | | | | | Reduces lint errors from 338 to 205 (-39%). Line count from 4650 to 3654 (-21%). Another step in resolving issue #2902.
* Bind btab also in vi-bindingsGravatar Fabian Homborg2016-05-03
| | | | | | It wasn't inherited. Fixes #2964.
* Suppress another pointless IWYU warningGravatar Kurtis Rader2016-05-02
|
* restyle proc module to match project styleGravatar Kurtis Rader2016-05-02
| | | | | | Reduces lint errors from 134 to 101 (-25%). Line count from 1994 to 1466 (-26%). Another step in resolving issue #2902.
* restyle postfork module to match project styleGravatar Kurtis Rader2016-05-02
| | | | | | Reduces lint errors from 37 to 20 (-46%). Line count from 670 to 566 (-15%). Another step in resolving issue #2902.
* restyle path module to match project styleGravatar Kurtis Rader2016-05-02
| | | | | | Reduces lint errors from 30 to 21 (-30%). Line count from 597 to 481 (-19%). Another step in resolving issue #2902.
* restyle parser module to match project styleGravatar Kurtis Rader2016-05-02
| | | | | | Reduces lint errors from 72 to 44 (-43%). Line count from 1698 to 1313 (-23%). Another step in resolving issue #2902.
* restyle parse_util module to match project styleGravatar Kurtis Rader2016-05-02
| | | | | | Reduces lint errors from 187 to 91 (-51%). Line count from 1754 to 1477 (-16%). Another step in resolving issue #2902.
* restyle parse_tree module to match project styleGravatar Kurtis Rader2016-05-02
| | | | | | Reduces lint errors from 163 to 52 (-68%). Line count from 2012 to 1904 (-5%). Another step in resolving issue #2902.
* restyle parse_productions module to match project styleGravatar Kurtis Rader2016-05-02
| | | | | | Reduces lint errors from 33 to 9 (-73%). Line count from 551 to 534 (-3%). Another step in resolving issue #2902.
* remove inadvertently added key_reader binaryGravatar Kurtis Rader2016-05-02
|
* restyle parse_execution module to match project styleGravatar Kurtis Rader2016-05-02
| | | | | | Reduces lint errors from 184 to 84 (-54%). Line count from 2139 to 1943 (-9%). Another step in resolving issue #2902.
* restyle pager & lru module to match project styleGravatar Kurtis Rader2016-05-02
| | | | | | Reduces lint errors from 65 to 25 (-63%). Line count from 1439 to 1218 (-15%). Another step in resolving issue #2902.
* restyle output module to match project styleGravatar Kurtis Rader2016-05-01
| | | | | | Reduces lint errors from 34 to 31 (-9%). Line count from 712 to 535 (-25%). Another step in resolving issue #2902.
* restyle kill module to match project styleGravatar Kurtis Rader2016-05-01
| | | | | | Reduces lint errors from 10 to 9 (-10%). Line count from 242 to 175 (-28%). Another step in resolving issue #2902.
* restyle iothread module to match project styleGravatar Kurtis Rader2016-05-01
| | | | | | Reduces lint errors from 41 to 26 (-37%). Line count from 444 to 423 (-5%). Another step in resolving issue #2902.
* restyle io module to match project styleGravatar Kurtis Rader2016-05-01
| | | | | | Reduces lint errors from 15 to 10 (-33%). Line count from 637 to 489 (-23%). Another step in resolving issue #2902.
* restyle intern module to match project styleGravatar Kurtis Rader2016-05-01
| | | | | | Reduces lint errors from 8 to 6 (-25%). Line count from 112 to 83 (-26%). Another step in resolving issue #2902.
* restyle input_common module to match project styleGravatar Kurtis Rader2016-05-01
| | | | | | Reduces lint errors from 27 to 24 (-11%). Line count from 466 to 378 (-19%). Another step in resolving issue #2902.
* Add missing color definitions to __fish_init_1_50_0 reset. (#2987)Gravatar Jorge Bucaran2016-05-01
| | | | | | | | | | | | | | * Add missing color definitions to __fish_init_1_50_0 reset. The values where determined by inspecting the values of: * fish_color_end * fish_color_user * fish_color_host after resetting the color theme via fish_config. * Add documentation for fish_color_user and fish_color_host.
* restyle input module to match project styleGravatar Kurtis Rader2016-04-30
| | | | | | Reduces lint errors from 69 to 48 (-30%). Line count from 1270 to 1044 (-18%). Another step in resolving issue #2902.
* restyle highlight module to match project styleGravatar Kurtis Rader2016-04-30
| | | | | | Reduces lint errors from 176 to 69 (-61%). Line count from 1627 to 1426 (-12%). Another step in resolving issue #2902.
* restyle function module to match project styleGravatar Kurtis Rader2016-04-30
| | | | | | Reduces lint errors from 39 to 27 (-31%). Line count from 619 to 498 (-20%). Another step in resolving issue #2902.
* restyle fish modules to match project styleGravatar Kurtis Rader2016-04-30
| | | | | | Reduces lint errors from 60 to 60 (-0%). Line count from 5599 to 4925 (-12%). Another step in resolving issue #2902.
* restyle fallback module to match project styleGravatar Kurtis Rader2016-04-30
| | | | | | | | | | Reduces lint errors from 36 to 33 (-8%). Line count from 1910 to 1476 (-23%). Another step in resolving issue #2902. This also fixes a stupid mistake from an earlier commit where I didn't realize that osx/config.h was meant to be included as a semi-static file in the repository.
* Remove use of __environGravatar ridiculousfish2016-04-30
| | | | It has apparently never worked. Fixes #2988
* don't use colors when writing the ^C indicatorGravatar Kurtis Rader2016-04-30
| | | | There was an extended discussion in https://github.com/fish-shell/fish-shell/issues/2904 about using a bright yellow background to make the cancelled command indicator, ^C, standout. The upshot was that standout (i.e., reversing fg/bg colors) mode should be used until themes are agumented with proper support for background colors and special characters.
* Remove using_command from netctl completionsGravatar Fabian Homborg2016-04-30
| | | | | | This allows `; and netctl` to work. First step towards #2705.
* git completion: Allow optional "+" for pushGravatar Fabian Homborg2016-04-30
| | | | | | | | This signifies a force-push. To avoid cluttering, only complete branches if a + is already given. Fixes #2879.
* Add repository/refspec completion to gitGravatar Fabian Homborg2016-04-30
| | | | | | | A few commands (fetch, pull and push at least) take a "repository" (aka "remote") and then a "refspec" (we currently do branches here). Fixes #2525 (seems that man is still alive)
* Erase the autosuggestion in fish_cancel_commandline by clearing to EOLGravatar ridiculousfish2016-04-29
|
* Remove an errant newline in the fish_cancel_commandline outputGravatar ridiculousfish2016-04-29
| | | | | Now the next line appears immediately after the cancelled line, without an intervening newline
* Remove some unused macros from builtin_set_color.cppGravatar ridiculousfish2016-04-29
|
* Add some more files missing from Xcode compile stepGravatar ridiculousfish2016-04-29
| | | | | Previously the .cpp files were #included by common.cpp Now they get compiled separately
* Issue a \r in fish_title, except when executing it for the promptGravatar ridiculousfish2016-04-29
| | | | | | | | | | | fish_title currently outputs some escaped text, which can confuse the line driver (#2453). Issue a carriage return so the line driver knows we are at the beginning of the line, unless we are writing the title as part of the prompt. In that case, we may have text from the previous command still on the line and we don't want to move the cursor. Fixes #2453