| Commit message (Collapse) | Author | Age |
|---|
| ... | |
| | |
|
| |
|
|
| |
This prevents undesired implicit conversions
|
| |
|
|
| |
It doesn't use any properties of parser_t
|
| |
|
|
| |
Errors are now unconditionally shown
|
| |
|
|
| |
It was never fully implemented and wasn't used for anything
|
| |
|
|
| |
Fixes #2772
|
| |
|
|
|
| |
Previously there had to be some variable text, now you can
tab complete from just a naked $.
|
| | |
|
| | |
|
| |
|
|
| |
completeable.
|
| |
|
|
| |
Ought to fix the wildcard expansion test on Linux
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
pcre2_substitute() now sets the output buffer length to PCRE2_UNSET (~0)
if the output buffer is determined to be too small. This change keeps
track of the buffer size separately where pcre2 can't touch it.
A better fix would be to let pcre2 tell fish what size buffer it needs.
This can be done with PCRE2_SUBSTITUTE_OVERFLOW_LENGTH, but this
requires pcre2 10.21 or later (released January 12), which may be too
new to introduce as a dependency at this point.
Fixes #2743
|
| |
|
|
| |
Noticed by cppcheck
|
| |
|
|
| |
out_suggested_cdpath is no longer required from is_potential_path
|
| | |
|
| | |
|
| | |
|
| |
|
|
| |
This avoids confusion between getcwd() and $PWD
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
| |
This will simplify some code and make the cd autosuggestion smarter
|
| | |
|
| |
|
|
|
|
| |
get_is_interactive can crash if proc_init has not been called.
Fixes #2280
|
| |
|
|
| |
Fix a few typos too
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Expand globs to zero arguments (nullglob) only for set, for and count.
The warning about failing globs, and setting the accompanying $status,
now happens regardless of mode, interactive or not.
It is assumed that the above commands are the common cases where
nullglob behaviour is desirable.
More importantly, doing this with `set` is a real feature enabler,
since the resulting empty array can be passed on to any command.
The previous behaviour was actually all nullglob (since commit
cab115c8b9933ae7db9412c66d452c0ccb2d7152), but this was undocumented;
the failglob warning was still printed in interactive mode,
and the documentation was bragging about failglob behaviour.
|
| | |
|
| | |
|
| |
|
|
| |
The argv argument may be modified on calls to exchange within the function and should not be const qualified (it's not true from the caller's point of view).
|
| |
|
|
|
|
| |
On arm, wchar_t is unsigned, and C++11 and newer disallow implicit
narrowing conversions inside braces. Use an explicit conversion to
fix the build on GCC 6 and up, which defaults to C++11.
|
| |
|
|
|
| |
Breaks up a big ugly function into separable stages
with a common interface.
|
| | |
|
| |
|
|
| |
Remove some unnecessary variables, make some things not pointers
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| | |
This changes the default escape timeout for the default keybindings (emacs
mode) to 300ms and the default for vi keybindings to 10ms.
I couldn't resist fixing a few nits in the fish_vi_key_bindings.fish file
since I was touching it to set the escape timeout.
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
All versions of fish prior to this change silently discarded anything written
to stderr while source a config.fish file. Apparently just to avoid having
the source command display an error if the file did not exist. This can mask
real problems. So instead this change explicitly checks whether the file is
readable and silently skips sourcing it if it isn't.
Resolves issue #2702.
|
| | |
| |
| |
| | |
Fixes #2672
|
| | |
| |
| |
| | |
Partial fix for #2672
|
| | |
| |
| |
| | |
deque is a memory hog last I checked
|
| |\ \ |
|
| | | |
| | |
| | |
| | | |
Fixes #2696
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Introduce a "fish_escape_delay_ms" variable to allow the user to configure the
delay used when seeing a bare escape before assuming no other characters will
be received that might match a bound character sequence. This is primarily
useful for vi mode so that a bare escape character (i.e., keystroke) can
switch to vi "insert" to "normal" mode in a timely fashion.
|
| | | |
| | |
| | |
| | |
| | | |
Rather than passing a triple of short, long, is_old, pass an option
as a string and then a type
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
Rather than storing short and long options separately, using
a complicated set of invariants, store them in a single string
and use an explicit type complete_option_type_t to track how they
are interpreted.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This was a "cache" of dubious value that was also very confusing.
The idea was to express in one place all of the short options that
were allowed for a command, in a big string. But it's simpler to
just construct that on-demand by walking the list of
complete_entry_opt_t.
Also remove some other dead code as part of cleanup.
|
ridiculousfish
Aaron Gyes
Michael Steed
Andreas Nordal
Jon Eyolfson
Andy Lutomirski
Kurtis Rader