Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | alloc: Switch arenas from a freelist to a bitmap allocator | Tavian Barnes | 2024-11-04 | 2 | -90/+262 |
| | |||||
* | diag: New bfs_unreachable() and bfs_assume() macros | Tavian Barnes | 2024-11-04 | 1 | -0/+32 |
| | |||||
* | alloc: Don't require the old size in varena_realloc() | Tavian Barnes | 2024-11-02 | 7 | -42/+68 |
| | | | | Instead, just look up which arena contains the pointer. | ||||
* | sanity: Don't mark memory uninit in sanitize_{alloc,free}() | Tavian Barnes | 2024-11-02 | 2 | -9/+11 |
| | | | | | We might want to change the size of an allocated region without changing which bytes are initialized. | ||||
* | alloc: Stop supporting pathological flexible array ABIs | Tavian Barnes | 2024-11-02 | 2 | -18/+29 |
| | |||||
* | alloc: Add macro versions of alignment utils | Tavian Barnes | 2024-11-02 | 1 | -3/+12 |
| | |||||
* | trie: Fix varena_free() with wrong size | Tavian Barnes | 2024-11-02 | 1 | -8/+8 |
| | | | | | | | In trie_remove(), clearing the bit before trie_node_collapse() causes us to free the old node with size 1 instead of 2, putting it on the wrong freelist. This is technically safe with the current arena implementation, but not intentional. | ||||
* | alloc: Don't require size % align == 0 | Tavian Barnes | 2024-11-02 | 2 | -30/+19 |
| | | | | | | | | | Allowing unaligned sizes will allow us to allocate aligned slabs with additional metadata in the tail without ballooning the allocation size for large alignments. Link: https://www.open-std.org/jtc1/sc22/wg14/www/docs/n2244.htm#dr_460 Link: https://www.open-std.org/jtc1/sc22/wg14/www/docs/n2072.htm | ||||
* | color: Handle bfs_check_capabilities error code (-1) in cpath_has_capabilities | Jody Frankowski | 2024-10-31 | 1 | -1/+1 |
| | | | | | | | | | | | On non-Linux systems, bfs_check_capabilities always returned -1, so cpath_has_capabilities would always return true. If bfs was called with an LS_COLORS containing the `ca` statement, e.g. `ca=30;41`, bfs would color ALL the files. While including the `ca` statement in LS_COLORS on non-Linux systems is unusual, this is not an error and is a valid use-case (e.g. Using GNU ls or lsd). Fixes: 1520d4d3 ("color: Don't color files like directories on ENOTDIR") | ||||
* | bit: Add bswap() overloads for every primitive type | Tavian Barnes | 2024-10-27 | 1 | -9/+34 |
| | | | | Fixes: https://github.com/tavianator/bfs/issues/145 | ||||
* | dstring: Add an infallible dstrshrink() function | Tavian Barnes | 2024-10-17 | 4 | -9/+47 |
| | | | | And mark the fallible ones with _nodiscard. | ||||
* | color: Don't color files like directories on ENOTDIR | Tavian Barnes | 2024-10-16 | 1 | -114/+230 |
| | |||||
* | color: Fix an infinite loop colorizing some invalid paths | Tavian Barnes | 2024-10-13 | 1 | -8/+11 |
| | | | | | | | | | | | | | | Previously, given $ touch file $ ln -s file/file notdir $ bfs notdir/file bfs would loop forever when printing the error message, since it expected stripping the trailing slash from "notdir/" to fix the ENOTDIR error, but the broken symlink still gave the same error. Fixes: b4c3201 ("color: Only highlight the trailing slash on ENOTDIR") | ||||
* | sighook: Don't forget to reset list->tail on the last sigpop() | Tavian Barnes | 2024-10-10 | 1 | -0/+2 |
| | | | | | | | This was causing a UAF if we ever unregistered the last hook for a signal and then re-registered one. Fixes: 75b7f70 ("sighook: Make sigunhook() O(1)") | ||||
* | eval: Use xtimer_start() for status bar updates | Tavian Barnes | 2024-10-10 | 1 | -36/+76 |
| | |||||
* | xtime: Add a wrapper for timer_create()/setitimer() | Tavian Barnes | 2024-10-10 | 2 | -0/+120 |
| | | | | | | setitimer() is obsolescent in POSIX 2008 and removed from POSIX 2024. However, at least macOS doesn't implement the new timer_create() API, so we still need the setitimer() fallback. | ||||
* | Adjust in/out parameter docs | Tavian Barnes | 2024-10-10 | 5 | -11/+11 |
| | |||||
* | Fix some -Wundef-prefix warnings | Tavian Barnes | 2024-10-10 | 4 | -6/+14 |
| | |||||
* | trie: Switch the tag bits around | Tavian Barnes | 2024-10-08 | 1 | -31/+28 |
| | | | | | | First of all, almost all checks were !trie_is_leaf(), so it makes sense to use trie_is_node() instead. Secondly, using the tag bit for internal nodes allows us to remove some NULL checks. | ||||
* | trie: Add some extra bounds checking | Tavian Barnes | 2024-10-08 | 1 | -13/+24 |
| | |||||
* | opt: Remove duplicate assignment in canonicalize_not() | Tavian Barnes | 2024-10-04 | 1 | -1/+0 |
| | |||||
* | color: Fix an assertion failure with embedded NUL bytes | Tavian Barnes | 2024-10-02 | 1 | -0/+10 |
| | | | | | | | | | | | | | Leading NUL bytes (e.g. `*\0.gz=...`) could cause us to insert a non-prefix-free set of strings into the case-insensitive extension trie, which would lead to crashes like bfs: trie_split@src/trie.c:538: Assertion failed: `key_nibble != rep_nibble` and OOB accesses to trie leaf keys. Fix it by ignoring those extensions, since filenames cannot contain NUL bytes. Fixes: 08030aea ("color: Delay the case_sensitive decision") | ||||
* | Write @arg instead of @param arg in doc comments | Tavian Barnes | 2024-10-01 | 27 | -367/+367 |
| | |||||
* | build: Add -Wundef-prefix=BFS_ | Tavian Barnes | 2024-10-01 | 2 | -0/+12 |
| | |||||
* | list: Make SLIST_REMOVE() more type-safe | Tavian Barnes | 2024-09-30 | 1 | -25/+20 |
| | | | | | | This avoids what might be a strict aliasing violation in some models. Link: https://github.com/llvm/llvm-project/pull/108385#issuecomment-2352938645 | ||||
* | Fix spelling | Tavian Barnes | 2024-09-23 | 6 | -6/+6 |
| | |||||
* | list: New drain_slist() macro | Tavian Barnes | 2024-09-19 | 4 | -39/+49 |
| | |||||
* | opt: #include "xspawn.h" before #ifdef POSIX_SPAWN_SETRLIMIT | Tavian Barnes | 2024-09-17 | 1 | -0/+1 |
| | | | | Fixes: 65a7814 ("opt: Don't raise RLIMIT_NOFILE if it would prevent using posix_spawn()") | ||||
* | opt: Fix an invalid assertion in sink_not_comma() | Tavian Barnes | 2024-09-17 | 1 | -4/+3 |
| | | | | | | | When optimizing -not \( -a , -b \), the child is a comma expression, not the parent. Fixes: 4a36bb9 ("expr: Make expressions variadic") | ||||
* | parse: Add a "logo" to bfs --version | Tavian Barnes | 2024-09-14 | 1 | -1/+32 |
| | |||||
* | dstring: New dstrepeat() function | Tavian Barnes | 2024-09-14 | 2 | -0/+22 |
| | |||||
* | opt: Add missing newline to debug statement | Tavian Barnes | 2024-09-12 | 1 | -1/+1 |
| | | | | Fixes: 9b8cf46 ("opt: Warn about all ignored tests") | ||||
* | Fix includes | Tavian Barnes | 2024-08-29 | 7 | -4/+6 |
| | |||||
* | build: Move feature test macros to prelude.h | Tavian Barnes | 2024-08-28 | 46 | -69/+57 |
| | | | | | | This replaces the explicit CPPFLAGS list in flags.mk with just `-include src/prelude.h`, shortening our compiler command lines and allowing them to be easily documented. | ||||
* | prelude: Split bfs-specific utilities into new bfs.h header | Tavian Barnes | 2024-08-28 | 50 | -194/+339 |
| | |||||
* | prelude: Rely more on __has_include() | Tavian Barnes | 2024-08-28 | 7 | -96/+31 |
| | | | | | Rather than a bunch of manual fallback macros, just provide a fallback definition that returns false. | ||||
* | build: Move tsan/target_clones conflict to the prelude | Tavian Barnes | 2024-08-28 | 1 | -1/+3 |
| | |||||
* | prelude: Define __SANITIZE_*__ | Tavian Barnes | 2024-08-28 | 6 | -24/+21 |
| | |||||
* | diag: Get rid of bfs_static_assert() | Tavian Barnes | 2024-08-28 | 3 | -13/+2 |
| | | | | | In most cases, it's not too annoying to specify a message. For tests/bit.c, we can manually polyfill the 1-argument version. | ||||
* | prelude: Define thread_local | Tavian Barnes | 2024-08-28 | 2 | -11/+17 |
| | |||||
* | bftw: Handle a hypothetical platform where AT_FDCWD == -1 | Tavian Barnes | 2024-08-28 | 1 | -2/+2 |
| | |||||
* | prelude: Stub out __has_builtin() if necessary | Tavian Barnes | 2024-08-23 | 1 | -0/+4 |
| | | | | Fixes: c964524 ("atomic: Add a spin_loop() hint") | ||||
* | eval: Don't warn about suppressed errors without -noerror | Tavian Barnes | 2024-08-19 | 1 | -1/+1 |
| | |||||
* | sighook: Fix typo in comment | Tavian Barnes | 2024-08-17 | 1 | -1/+1 |
| | |||||
* | eval: Capitalize -noerror warning message | Tavian Barnes | 2024-08-17 | 1 | -1/+1 |
| | |||||
* | docs: Document the new -mount vs. -xdev behaviour | Tavian Barnes | 2024-08-16 | 1 | -2/+1 |
| | | | | Fixes: 33b85e1 ("Implement POSIX 2024's -mount") | ||||
* | New -noerror option to suppress error messages | Tavian Barnes | 2024-08-16 | 3 | -1/+29 |
| | | | | Closes: https://github.com/tavianator/bfs/issues/142 | ||||
* | eval: Remove info_hook from callback_args | Tavian Barnes | 2024-08-16 | 1 | -4/+2 |
| | |||||
* | opt: Warn about all ignored tests | Tavian Barnes | 2024-08-15 | 1 | -31/+34 |
| | | | | | This lets us warn about `bfs -delete -empty`, even though -empty is impure. | ||||
* | expr: Tell expressions what kind of expression they are | Tavian Barnes | 2024-08-15 | 5 | -200/+205 |
| |