| Commit message (Collapse) | Author | Age |
... | |
|\ |
|
| |
| |
| |
| | |
Fixes #718
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The u_int typedef fails to compile on all platforms (e.g. Windows). It
is part of the code imported from tmux.
Update it to the SUS-standard uid_t.
Closes #2821.
|
|/
|
|
|
|
|
|
|
| |
The u_int typedef fails to compile on all platforms (e.g. Windows). It
is part of the code imported from tmux.
Update it to the SUS-standard uid_t.
Closes #2821.
|
|
|
|
| |
My IDE (Kdevelop 4.7.3) didn't tackle them.
|
|
|
|
|
|
|
|
|
| |
Address the feedback from the prior commit:
- Change the sense of return value testing to match more common
comparison idiom
- Test result of fchmod as well as fchown
- Change sense of return value testing around wrename as well
- Include errno where possible in error message
|
|
|
|
|
|
|
|
|
|
|
| |
The function fchown is annotated with warn_unused_result. As
formerly used in the code, it would emit a compiler warning
```warning: ignoring return value of ‘fchown’, declared with
attribute warn_unused_result [-Wunused-result]```
This commit notes the return value and emits appropriate error/logging
messages if the call fails, creating more traceable results and
satisfying the compiler.
|
| |
|
|
|
|
|
|
|
| |
instead add a bit of information on how fish's configuration works for
the admin to etc/config.fish.
This means that fish is fully functional without /etc, which might be nice for "stateless" systems.
|
|
|
|
|
|
|
|
|
|
|
| |
There is no longer a good reason to detect whether or not getopt_long()
is available. All UNIX implementations we're likely to run on have it. And
if we ever find one that doesn't the right thing to do is not fallback to
getopt() but to include the getopt_long() source in our package like we
do with the pcre2 library. Since it's licensed under LGPL we can legally
do so if it becomes necessary.
This partially addresses issue #2790.
|
|
|
|
| |
Fixes #2534 as backward-kill-path-component is now documented.
|
|
|
|
| |
Closes #2798.
|
|
|
|
|
| |
This is an unwise assumption, both for en_US and UTF-8; the fallback C
locale should be used instead.
|
|
|
|
|
|
| |
Closes #2190.
[ci skip]
|
|
|
|
| |
Fixes #2796
|
| |
|
|
|
|
|
| |
If envp contains duplicate environment variables, use the
first value, not the last value. Fixes #2784.
|
| |
|
|
|
|
| |
The test is unnecessary.
|
|
|
|
|
| |
This is important information but the Set button is the wrong place to
put it.
|
|
|
|
|
|
|
|
| |
Previously, when decoding UTF-8, we would first run through the
array to compute the correct size, then allocate a buffer of that size,
then run through the array again to fill the buffer, and then copy it
into a std::wstring. With this fix we can copy it into the string
directly, reducing allocations and only requiring a single pass.
|
|
|
|
|
|
|
|
|
| |
We silently upgrade existing abbreviations and change the separator when
saving.
This does not yet warn when the user is using the old syntax.
Resolves #2051
|
|
|
|
|
|
|
|
| |
These are an additional source of information
Thanks to @sysbot (#2313) for the inspiration.
This also stringifies the ssh_config reading
|
| |
|
| |
|
|
|
|
| |
This would be a lot of useless options for those who block via hosts.
|
|
|
|
| |
Pass the input to 'string', it was accidentally removed in fcdc6a48c0bbdc796975db8d0b7f32434d86d249.
|
|
|
|
|
|
|
|
|
| |
This narrows the range of Unicode codepoints fish reserves for its own
use from U+E000 thru U+F8FE (6399 codepoints) to U+F600 thru U+F73F (320
codepoints). This is still not ideal since fish shouldn't be using any
Unicode private-use codepoints but it's a step in the right direction.
This partially addresses issue #2684.
|
|
|
|
| |
See #2777
|
| |
|
|
|
|
| |
See #2773
|
|
|
|
|
|
|
|
| |
Turns out some shells will alias which to be something function-aware,
but doing this on fish would blow up because it would call type which
would then call which which would then call type....
Fixes #2775
|
|
|
|
|
|
|
|
|
|
| |
This was used to cache a narrow string representation
of commands, so that if certain system calls returned errors
after fork, we could output error messages without allocating
memory. But in practice these errors are very uncommon, as are
commands that have wide characters. It is simpler to do a best-effort
output of the wide string, instead of caching a narrow string
unconditionally.
|
|
|
|
|
|
|
| |
Prior to this fix, read_ni would use parse_util_detect_errors
to lint the script to run, and then parser_t::eval() to execute it.
Both functions would parse the script into a parse tree. This allows
us to re-use the parse tree, improving perfomance.
|
|
|
|
|
|
| |
Introduces a new template moved_ref which is like an rvalue reference.
This allows passing around objects while being explicit that the
receiver may acquire ownership. This will help reduce some allocations.
|
| |
|
| |
|
| |
|
|
|
|
| |
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 $.
|
| |
|
| |
|
|
|
|
|
|
| |
Much better to only encode the characters that are not URL-safe. This
also doesn't involve any forking, and it even handles newlines and NULs
in the input.
|
|
|
|
|
|
| |
This is a file under version control, there's no reason it should be
listed here. Having it in .gitignore was causing tools like `ag` to
avoid looking at share/config.fish.
|
|
|
|
|
|
| |
I had an old /etc/paths.d/TeX file that listed a path /usr/texbin that
doesn't exist anymore. This was causing the `set PATH` to print a
warning.
|
| |
|