aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* 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
* Fix the Xcode buildGravatar ridiculousfish2016-04-29
| | | | Add missing files to link phase
* git takes --help even when it needs a command (#2984)Gravatar Sanne Wouda2016-04-29
| | | `git --help` is a valid command and fish should complete it as such
* fix bind unit testsGravatar Kurtis Rader2016-04-28
| | | | | | In my rush to get the fix for the wrong default Vi mode escape delay merged (commit 3e24ae80b3739ded7d4066349fc87d13f2b70727) I neglected to update the unit test. This change corrects that oversight.
* clarify fish_vi_mode deprecation warningGravatar Kurtis Rader2016-04-28
| | | | | | | Also, correct the Vi mode default escape timeout. I intended it to be 100 ms in my previous change but it ended up 10 ms which is far too short. A 10 ms delay will continue to cause problems for people running fish inside `screen`, `tmux`, or over high latency connections.
* restyle expand module to match project styleGravatar Kurtis Rader2016-04-28
| | | | | | Reduces lint errors from 183 to 126 (-31%). Line count from 2231 to 1787 (-20%). Another step in resolving issue #2902.
* restyle exec module to match project styleGravatar Kurtis Rader2016-04-28
| | | | | | Reduces lint errors from 121 to 59 (-51%). Line count from 1578 to 1290 (-18%). Another step in resolving issue #2902.
* retyle event module to match project styleGravatar Kurtis Rader2016-04-28
| | | | | | Reduces lint errors from 39 to 30 (-23%). Line count from 915 to 670 (-27%). Another step in resolving issue #2902.
* restyle env_universal module to match project styleGravatar Kurtis Rader2016-04-28
| | | | | | Reduces lint errors from 121 to 52 (-57%). Line count from 1916 to 1671 (-13%). Another step in resolving issue #2902.
* restyle env module to match project styleGravatar Kurtis Rader2016-04-28
| | | | | | Reduces lint errors from 90 to 72 (-20%). Line count from 1719 to 1298 (-24%). Another step in resolving issue #2902.
* document how to config editors for this projectGravatar Kurtis Rader2016-04-28
|
* provide a realpath implementationGravatar Kurtis Rader2016-04-28
| | | | | | | Not all distros have a `realpath` command. Provide a function that uses the real command if available else use the fish builtin. Fixes #2932
* git completion: Only show unmerged branches for cherry-pickGravatar Fabian Homborg2016-04-28
|
* switch to newer flag --set-upstream-to for git (#2982)Gravatar Cody Scott2016-04-28
| | | | | | | set-upstream was deprecated in git 1.8.0 as stated in [1] in favor of set-upstream-to. this patch replaces the old flag in fish completions [1]: https://raw.githubusercontent.com/git/git/master/Documentation/RelNotes/1.8.0.txt
* restyle common module to match project styleGravatar Kurtis Rader2016-04-27
| | | | | | Reduces lint errors from 194 to 142 (-27%). Line count from 3352 to 2645 (-21%). Another step in resolving issue #2902.
* restyle color module to match project styleGravatar Kurtis Rader2016-04-27
| | | | | | Reduces lint errors from 16 to 14 (-13%). Line count from 558 to 463 (-17%). Another step in resolving issue #2902.
* compile fallback.cpp and util.cpp seperatelyGravatar Kurtis Rader2016-04-27
| | | | | | | Don't `#include "*.cpp"` modules in other cpp modules. I already took care of all the builtin_*.cpp modules in my previous change where I restyled the builtin code. This change fixes the two remaining instances of this anti-pattern.
* env_universal_common.cpp: pass correct flag to fcntlGravatar David Adam2016-04-27
| | | | | | Closes #2955. (cherry picked from commit d5797c543982a0fbc8201ae21db5009e9447f75d)
* Make busctl completions usefulGravatar Fabian Homborg2016-04-27
| | | | | | | | | - More accurate - Fast enough to be usable (previously, this would sometimes take a few seconds) - A bit smaller
* restyle builtin modules to match project styleGravatar Kurtis Rader2016-04-26
| | | | | | | | | | | Now that the IWYU cleanup has been merged compile all, not just a couple, of the builtin modules independent of builtin.cpp. That is, no longer `#include builtin_NAME.cpp` in builtin.cpp. This is more consistent, more in line with what developers expect, and is likely to reduce mistakes. Reduces lint errors from 384 to 336 (-13%). Line count from 6307 to 4988 (-21%). Another step in resolving issue #2902.
* Update usage of pcre2_substitute() for pcre2-10.21Gravatar Michael Steed2016-04-27
| | | | | | | - Set PCRE2_SUBSTITUTE_OVERFLOW_LENGTH to get the required buffer length from pcre2 instead of guessing - Set PCRE2_SUBSTITUTE_EXTENDED to enable extra goodies in the replacement string
* configure: require at least pcre2-10.21Gravatar Michael Steed2016-04-27
|
* add better support for IWYU and fix thingsGravatar Kurtis Rader2016-04-26
| | | | | | | | | Remove the "make iwyu" build target. Move the functionality into the recently introduced lint.fish script. Fix a lot, but not all, of the include-what-you-use errors. Specifically, it fixes all of the IWYU errors on my OS X server but only removes some of them on my Ubuntu 14.04 server. Fixes #2957
* Allow setting key bindings universallyGravatar Fabian Homborg2016-04-26
| | | | As always, we default to setting globally.
* Remove named arg in fish_default_key_bindingsGravatar Fabian Homborg2016-04-26
| | | | This wasn't actually used anywhere.
* Tests: Add fish_mode_prompt to except_promptGravatar Fabian Homborg2016-04-26
| | | | | Without this, the interactive tests fail when they receive a mode_prompt in vi-mode.
* Set fish_key_bindings globally in binding functionsGravatar Fabian Homborg2016-04-26
| | | | This should fix the tests.
* Deprecate fish_vi_modeGravatar Fabian Homborg2016-04-26
| | | | | This was never mentioned in the documentation as the way to switch to vi-mode, and now does nothing of value anymore.
* Let the binding functions set the binding variableGravatar Fabian Homborg2016-04-26
| | | | | This ensures they can just be called and "the right thing" will happen - fish_user_key_bindings will be executed, the variable will reflect the bindings.
* Remove $__fish_vi_modeGravatar Fabian Homborg2016-04-26
| | | | | | | This makes fish_mode_prompt rely on $fish_key_bindings instead. fish_bind_mode is also set in default mode (only always "default"), so it can't be used as the indicator.
* more doxygen lexicon changes to eliminate errorsGravatar Kurtis Rader2016-04-24
| | | | | | I noticed that Doxygen was also complaining about the "<asis>" and "<bs>" tags. So convert those to the backslash form like we did for "<outp>" in the previous commit.
* replace <outp> command with \outp in docsGravatar Kurtis Rader2016-04-23
| | | | | | | | | | Doxygen has been warning that `<outp>` and `</outp>` are not valid XML/HTML commands since commit cb6d5d76 on 20016-04-04. That's primarily because there is at present no way to tell Doxygen to recognize new XML/HTML tags. The actual errors look like this: ``` .../string.doxygen:187: warning: Unsupported xml/html tag </outp> found ``` I hate build errors since they a) cause needless concern, and b) make it harder to notice when I've introduced a new error. So switch from XML/C## style markup to Doxygen style markup for the "outp" annotation.
* trivial fixes to make doxygen happyGravatar Kurtis Rader2016-04-23
|
* build_tools/make_pkg: fixups for versioning and Xcode preferencesGravatar David Adam2016-04-20
| | | | [ci skip]
* add way to comment/uncomment a commandGravatar Kurtis Rader2016-04-19
| | | | Fixes #2375
* make comments Xcode friendlyGravatar Kurtis Rader2016-04-19
| | | | | | The OS X Xcode IDE has a weird requirement that block comments preceding a function or class definition must begin with three slashes rather than two if you want the comment displayed in the "Quick Help" window.
* Merge branch 'Integration_2.3.0'Gravatar David Adam2016-04-19
|\ | | | | | | | | | | Merge the pcre2 10.20 to 10.21 update. Closes #2943.
| * pcre2: add maintainer mode and disable by defaultGravatar David Adam2016-04-19
| |
| * Apply pcre2 svn rev 489 patch to fix CVE-2016-3191Gravatar Aaron Gyes2016-04-19
| |
| * Update Xcode build for new pcre2 versionGravatar David Adam2016-04-19
| |
* | restyle autoload module to match project styleGravatar Kurtis Rader2016-04-18
| | | | | | | | | | | | Reduces lint errors from 38 to 19 (-50%). Line count from 506 to 426 (-16%). Another step in resolving issue #2902.
* | restyle builtin module to match project styleGravatar Kurtis Rader2016-04-18
| | | | | | | | | | | | Reduces lint errors from 271 to 215 (-21%). Line count from 4304 to 3242 (-25%). Another step in resolving issue #2902.
| * Update to pcre2 10.21Gravatar Aaron Gyes2016-04-19
| | | | | | | | Point build tools at 10.21
* | restyle history code to match project styleGravatar Kurtis Rader2016-04-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make the history code conform to the new style guide. Every change was produced by clang-format (e.g., `make style`) with the exception of comments which were manually reformatted. That has to be done by hand since clang-format leaves comments alone other than to reflow comment lines to get them below the allowed line length. The total number of lines is reduced by 313 lines (13%) in the two affected files. Line count is generally a poor metric but in this case it reflects an increase in information density without a loss in readability. Furthermore, the standardization of braces, whitespace, and comment style will make it easier for people to read the code. This reduces the number of warnings by `make lint` from 168 to 87 (a 48% decrease). Making it much easier to focus on the substantive lint issues. Further improvements are possible. For example, many comments are not very helpful (e.g., they point out the obvious) or provide insufficient detail. But those are beyond the scope of this change. This is the first step in resolving issue #2902.
| * Allow overriding fish_term24bit on launchGravatar Fabian Homborg2016-04-18
| | | | | | | | (cherry picked from commit ba1008b75033e9ff8f4ad17fa2de25eed115de5c)
| * Move 24bit setup into config.fishGravatar Fabian Homborg2016-04-18
| | | | | | | | | | | | Fixes #2941. (cherry picked from commit 8558561650995b330a1ae5ee87c592382856cc40)
* | Allow overriding fish_term24bit on launchGravatar Fabian Homborg2016-04-18
| |