Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | Makefile: Run tests with --verbose during distcheck | Tavian Barnes | 2020-11-28 | 1 | -1/+3 | |
| | ||||||
* | parse: Don't pass an uninitialized regex_t to regerror() | Tavian Barnes | 2020-11-28 | 1 | -1/+1 | |
| | ||||||
* | color: Add an intermediate buffer | Tavian Barnes | 2020-11-28 | 2 | -68/+104 | |
| | | | | | | | cfprintf() on an unbuffered file like stderr was causing a lot of write() calls since every format specifier resulted in a separate fwrite(). This led to inefficiency and bad interleaving. Add our own dstring buffer to smooth it out. | |||||
* | dstring: New dstrdcat(), dstrcatf(), dstrvcatf() functions | Tavian Barnes | 2020-11-28 | 2 | -23/+97 | |
| | ||||||
* | eval: Make -hidden behave consistently | Tavian Barnes | 2020-11-28 | 3 | -1/+19 | |
| | | | | | | | | | | | | | | | Previously there was an unexpected difference between $ bfs .hidden -hidden and $ bfs ./.hidden -hidden ./.hidden The intent of the code was to avoid considering `.`, the default starting point, as hidden and thus pruning a whole search with -nohidden. Fix it to do that explicitly, and handle `..` too. | |||||
* | stat: #include <unistd.h> for syscall() when needed | Tavian Barnes | 2020-11-18 | 1 | -0/+1 | |
| | ||||||
* | Include what I use | Tavian Barnes | 2020-11-12 | 29 | -36/+31 | |
| | | | | Thanks to https://github.com/include-what-you-use/include-what-you-use | |||||
* | ctx: Add missing #include <limits.h> | Tavian Barnes | 2020-11-12 | 1 | -0/+1 | |
| | | | | Spotted at https://github.com/freebsd/freebsd-ports/commit/4d80040cb4c10271b375f3b5d70d2b29dae7fc6f | |||||
* | Release 2.12.1 | Tavian Barnes | 2020-11-11 | 3 | -2/+12 | |
| | ||||||
* | tests: Don't squelch stderr with --verbose | Tavian Barnes | 2020-11-11 | 1 | -48/+64 | |
| | ||||||
* | tests: Test LS_COLORS extension lowercasing | Tavian Barnes | 2020-11-10 | 1 | -2/+2 | |
| | ||||||
* | Makefile: New gcov target | Tavian Barnes | 2020-11-10 | 2 | -1/+9 | |
| | ||||||
* | opt: Assert that we don't do disabled optimizations | Tavian Barnes | 2020-11-10 | 1 | -0/+2 | |
| | ||||||
* | tests: Improve test coverage a bit | Tavian Barnes | 2020-11-10 | 13 | -0/+350 | |
| | ||||||
* | tests: Add missing expectations from dd3bbb9 | Tavian Barnes | 2020-11-09 | 2 | -0/+38 | |
| | ||||||
* | opt: Predicates aren't true when they're false | Tavian Barnes | 2020-11-09 | 2 | -2/+14 | |
| | | | | | | | | | | | | This unfortunate typo was mostly harmless; since the predicates were always assumed to be true, they wouldn't conflict. The exception is -user/-group, which set -nouser/-nogroup to false for users/groups that exist. Even -O0 wasn't enough to suppress the bug, due to a missing optlevel check fixed in the previous commit. Fixes: 305ee902874b49351f4916e303c293523f11570b | |||||
* | opt: Check optlevel before removing unreachable expressions | Tavian Barnes | 2020-11-09 | 1 | -2/+4 | |
| | ||||||
* | Use two newlines for all pre-eval warnings | Tavian Barnes | 2020-11-09 | 2 | -2/+2 | |
| | ||||||
* | eval: Fix the status bar to only print the parent again | Tavian Barnes | 2020-11-04 | 1 | -5/+7 | |
| | ||||||
* | eval: Make a crude attempt to handle double-width chars in the status bar | Tavian Barnes | 2020-11-04 | 1 | -11/+49 | |
| | ||||||
* | Enable -Wsign-compare to catch bugs like 726d7801 | Tavian Barnes | 2020-11-04 | 6 | -11/+11 | |
| | ||||||
* | spawn: Fix error pipe write failure handling | Tavian Barnes | 2020-11-04 | 1 | -1/+8 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A short history lesson: this code (from exec.c) used to just be write(...); In 649d85b, an empty if block was added to placate _FORTIFY_SOURCE's warn_unused_result: if (write(...) != sizeof(error)) { // ... } This is fine since the parent doesn't rely on receiving those writes (e.g. the child could die before writing), and the small write to a pipe won't fail anyway. But when bfs_spawn() was created, the implementation changed to this: while (write(...) < sizeof(error)); The while loop was presumably added to mirror musl's posix_spawn() implementation, which handles EINTR. But musl's is actually correct: while (write(...) < 0); whereas mine has a subtle bug: since sizeof(error) is unsigned, the bfs code did an unsigned comparison, meaning -1 from write() would *not* restart the loop. Fix it by comparing with 0 like musl, since short writes are impossible here. Perhaps it's time for -Wsign-compare, despite the other 18 instances being false positives. | |||||
* | Makefile: Fail early on sanitizer errors | Tavian Barnes | 2020-11-03 | 1 | -0/+8 | |
| | ||||||
* | New -status option to display a status bar | Tavian Barnes | 2020-11-03 | 5 | -14/+122 | |
| | ||||||
* | bar: Implement terminal status bars | Tavian Barnes | 2020-11-03 | 3 | -0/+320 | |
| | ||||||
* | dstring: New dstrvprintf() function | Tavian Barnes | 2020-11-03 | 2 | -9/+28 | |
| | ||||||
* | Update the project URL | Tavian Barnes | 2020-10-14 | 2 | -2/+2 | |
| | ||||||
* | Release 2.02.0 | Tavian Barnes | 2020-10-14 | 3 | -3/+7 | |
| | ||||||
* | util: New BFS_FLEX_SIZEOF() macro for more precise flexible array allocations | Tavian Barnes | 2020-10-13 | 4 | -4/+28 | |
| | | | | | See http://www.open-std.org/jtc1/sc22/wg14/www/docs/dr_282.htm for all the fun behind this. | |||||
* | pwcache: Adjust some naming conventions | Tavian Barnes | 2020-10-06 | 3 | -14/+14 | |
| | ||||||
* | mtab: Adjust some naming conventions | Tavian Barnes | 2020-10-06 | 3 | -7/+7 | |
| | ||||||
* | expr, eval: Clean up header dependencies | Tavian Barnes | 2020-10-06 | 3 | -19/+23 | |
| | ||||||
* | exec: Adjust some calling conventions | Tavian Barnes | 2020-10-06 | 3 | -11/+11 | |
| | ||||||
* | printf: Adjust some calling conventions | Tavian Barnes | 2020-10-06 | 3 | -22/+22 | |
| | ||||||
* | ctx: Don't include color.h | Tavian Barnes | 2020-10-06 | 1 | -4/+4 | |
| | ||||||
* | diag: New bfs_perror() function | Tavian Barnes | 2020-10-05 | 6 | -37/+53 | |
| | ||||||
* | parse: Fail if -color is passed and the colors couldn't be parsed | Tavian Barnes | 2020-10-04 | 3 | -0/+13 | |
| | ||||||
* | parse: More accurate error reporting for cfdup() | Tavian Barnes | 2020-10-04 | 1 | -1/+7 | |
| | ||||||
* | parse: Report errors when failing to add a root | Tavian Barnes | 2020-10-04 | 1 | -1/+5 | |
| | ||||||
* | diag: Factor debug_flag string conversion into its own function | Tavian Barnes | 2020-10-04 | 1 | -25/+21 | |
| | ||||||
* | time: Don't call tzset() on every x{local,gm}time() | Tavian Barnes | 2020-10-01 | 1 | -2/+11 | |
| | | | | It turns out tzset() checks /etc/localtime every time you call it. | |||||
* | dstring: Try to avoid calling vsnprintf() twice in dstrprintf() | Tavian Barnes | 2020-10-01 | 1 | -10/+26 | |
| | ||||||
* | main: Preserve errno over close() in redirect() | Tavian Barnes | 2020-10-01 | 1 | -6/+7 | |
| | ||||||
* | util: Move redirect() and isopen() to main.c | Tavian Barnes | 2020-10-01 | 3 | -48/+22 | |
| | ||||||
* | util: Don't rely on bftw | Tavian Barnes | 2020-09-30 | 4 | -40/+39 | |
| | | | | And rename format_mode() to xstrmode() while I'm at it. | |||||
* | ctx: Perserve errno better in bfs_ctx_open() | Tavian Barnes | 2020-09-28 | 1 | -0/+7 | |
| | ||||||
* | Rename struct cmdline to bfs_ctx | Tavian Barnes | 2020-09-27 | 18 | -562/+786 | |
| | | | | | The API remains similar, with some added accessor functions for lazy initialization of the pwcache and mtab. | |||||
* | exec: Output a human-readable description of terminating signals | Tavian Barnes | 2020-09-23 | 1 | -1/+5 | |
| | ||||||
* | bftw: Fix bftw_cached_stat() with BFS_STAT_TRYFOLLOW | Tavian Barnes | 2020-09-20 | 1 | -1/+5 | |
| | ||||||
* | tests: Add missing ground truth | Tavian Barnes | 2020-09-20 | 1 | -0/+12 | |
| |