aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* 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
| |
* | Move 24bit setup into config.fishGravatar Fabian Homborg2016-04-18
| | | | | | | | Fixes #2941.
* | number `dirh` output to make prevd/nextd easierGravatar Kurtis Rader2016-04-16
| | | | | | | | Fixes #2786
* | Merge branch 'Integration_2.3.0' of https://github.com/fish-shell/fish-shellGravatar Kurtis Rader2016-04-15
|\|
| * fix off by one errorGravatar Kurtis Rader2016-04-15
| | | | | | | | (cherry picked from commit ea3d9c36a52dcf6bcde0ff41af8ca19daca2267e)
| * fish handling of readlink()Gravatar Kurtis Rader2016-04-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The readlink() function does not null terminate the path it returns. Remove the OS X code that deals with a path buffer that is too short. For one thing a loop isn't needed since we're told how big of a buffer is required if the first _NSGetExecutablePath() call fails. But more important it is so unlikely that the path will be longer than PATH_MAX that if it is we should just give up. Fixes 2931. (cherry picked from commit 8e103c231e74a3dda40d0a0e5a844ea8b6aa1e4b)
| * Add completions for 'pacaur' (#2934)Gravatar Yauhen Kirylau2016-04-15
| |
| * Correct typo in valgrind completionsGravatar Laurence McGlashan2016-04-15
| |
| * Remove the default self-insert binding in vi-default modeGravatar Fabian Homborg2016-04-15
| | | | | | | | Fixes #2832.
* | Add completions for 'pacaur' (#2934)Gravatar Yauhen Kirylau2016-04-15
| |
* | provide a better experience when user presses \cCGravatar Kurtis Rader2016-04-14
| | | | | | | | Fixes #2904
* | tell git to ignore test failure artifactsGravatar Kurtis Rader2016-04-14
| |
* | Correct typo in valgrind completionsGravatar Laurence McGlashan2016-04-13
| |
* | fix off by one errorGravatar Kurtis Rader2016-04-13
| |
* | improve the style.fish scriptGravatar Kurtis Rader2016-04-12
| | | | | | | | | | | | | | | | | | | | If there are uncommitted changes use `git-clang-format` to limit the style fixups to the lines being modified. Refuse to do a `make style-all` if there are uncommitted changes. Include a fix for the parsing of `git status` output that was recently incorporated into the lint.fish script.
* | fish handling of readlink()Gravatar Kurtis Rader2016-04-12
| | | | | | | | | | | | | | | | | | | | | | | | The readlink() function does not null terminate the path it returns. Remove the OS X code that deals with a path buffer that is too short. For one thing a loop isn't needed since we're told how big of a buffer is required if the first _NSGetExecutablePath() call fails. But more important it is so unlikely that the path will be longer than PATH_MAX that if it is we should just give up. Fixes 2931.
* | revert inadvertent file inclusion in prior commitGravatar Kurtis Rader2016-04-12
| | | | | | | | | | Commit 6431c0de16937622a2664c417f517d002220e017 was not meant to include changes to fish.cpp.
* | fix bug in lint.fish helper scriptGravatar Kurtis Rader2016-04-12
| | | | | | | | | | | | | | I just noticed that depending on the state of your working tree there can be one or more spaces after the modification token and the file name. If there is more than one space that causes the `string split` to produce unexpected output.
* | Remove the default self-insert binding in vi-default modeGravatar Fabian Homborg2016-04-12
| | | | | | | | Fixes #2832.
* | retry flakey tests on failureGravatar Kurtis Rader2016-04-11
| | | | | | | | Fixes #2926
* | another wcstring to wchar_t * changeGravatar Kurtis Rader2016-04-10
| | | | | | | | | | | | | | | | | | | | | | In keeping with the change made by @ridiculousfish earlier today modify the `keyword_description()` function to return a const wchar_t pointer. Also, simplify the `token_type_description()` function to use the recently introduced mapping array. This changes the wording of many of the token type descriptions. However, I can't see this as being a problem since the original descriptions (e.g., "token_redirection") are no clearer to someone not acquainted with the implementation.
* | enhance fish_indent to normalize keywordsGravatar Kurtis Rader2016-04-10
| | | | | | | | | | | | | | | | | | | | | | Fish keywords can be quoted and split across lines. Prior to this change `fish_indent` would retain such odd, obfuscated, formatting. This change results in all keywords being converted to their canonical form. This required fixing a bug: the keyword member of parse_node_t wasn't being populated. This hadn't been noticed prior to now because it wasn't used. Fixes #2921
* | Change parser_token_types from wcstring to const wchar_t *Gravatar ridiculousfish2016-04-10
|/ | | | Reduces allocations and startup time
* Fix #2919 by removing the entire block.Gravatar Aaron Gyes2016-04-09
| | | | | | | | | | | | | | This code represents only risk and does nothing useful for anything that can compile fish. In C++ situations where __STDC_VERSION__ is unset (as it should be), fish was assuming we are on < C99 and setting it to __FUNCTION__. Basically always, __PRETTY_FUNCTION__ ends up reaplaced by __FUNCTION__, this hurt error message usefulness and richness. __PRETTY_FUNCTION__: const thing::sub(int) __FUNCTION__: sub
generated by cgit v1.2.3 (git 2.39.1) at 2024-07-03 02:35:51 +0000