Age | Commit message (Collapse) | Author | Files | Lines | |
---|---|---|---|---|---|
10 days | Fix spellingHEADmain | David Robillard | 6 | -7/+7 | |
12 days | Suppress new warnings in clang-tidy | David Robillard | 1 | -0/+1 | |
12 days | Avoid potentially returning a reference to a local | David Robillard | 1 | -10/+6 | |
This case shouldn't happen anyway since it's handled at the caller, which is best not only because of this issue, but because having set_property() maybe not set the property is confusing. | |||||
12 days | Use std::min() and std::max() | David Robillard | 4 | -16/+7 | |
12 days | Put local utility function in anonymous namespace | David Robillard | 1 | -0/+4 | |
12 days | Fully parenthesize expressions | David Robillard | 9 | -16/+16 | |
12 days | Clean up includes and forward declarations | David Robillard | 62 | -158/+38 | |
12 days | Avoid using fallback for dependencies without a subproject | David Robillard | 1 | -1/+0 | |
This screws things up with older meson versions, particularly with include_type 'system'. I don't think it had any purpose since there's no corresponding subproject inside this one anyway. | |||||
2025-01-20 | Use "system" include type for all dependencies | David Robillard | 2 | -0/+2 | |
Things get confused when these flags differ across projects, so universally use "system" for external dependencies. | |||||
2024-12-19 | Order dependencies to prioritize flags of lower level libraries | David Robillard | 1 | -2/+4 | |
Specifically, to make the "system" include type for LV2 work and suppress C warnings there. Otherwise, the settings from other projects that depend on it (like lilv) take priority. This seems broken to me, and didn't used to happen, so I'm not really sure what's changed, but this fixes it. | |||||
2024-12-02 | Fix potential use of uninitialized value | David Robillard | 1 | -1/+1 | |
According to GCC's -Wmaybe-uninitialized, this is a potential use of an uninitialized value. I'm not seeing it, but the pattern with get_property() everywhere else is to store the return value as a reference, which makes the warning go away, so do that. | |||||
2024-11-24 | Remove IWYU pragmas that no longer seem necessary | David Robillard | 19 | -34/+20 | |
In a few cases here, we could get away with just a forward declaration but IWYU can't figure it out. Since this isn't a huge deal, just do what the tool says, since it's overall less hassle to just do what it says. | |||||
2024-11-24 | Use angle brackets for library includes | David Robillard | 189 | -1263/+1392 | |
2024-10-26 | Fix line wrapping in comments | David Robillard | 5 | -8/+14 | |
2024-10-12 | Fix configuration directory creation error handling | David Robillard | 1 | -2/+4 | |
2024-10-11 | Add missing include | David Robillard | 1 | -0/+1 | |
Apparently things are defined in different headers in different JACK headers. | |||||
2024-10-11 | Use std::transform() | David Robillard | 2 | -18/+29 | |
2024-10-11 | Use std::find_if() | David Robillard | 4 | -51/+58 | |
2024-10-11 | Use std::count_if() | David Robillard | 2 | -16/+10 | |
2024-10-11 | Use std::any_of() | David Robillard | 8 | -65/+54 | |
2024-10-11 | Clarify atom forge buffer dereferencing code | David Robillard | 1 | -2/+3 | |
2024-10-11 | Handle realloc failure and avoid potential null pointer arithmetic | David Robillard | 4 | -8/+29 | |
2024-10-11 | Move AtomForge implementation out of public headers | David Robillard | 2 | -0/+117 | |
2024-10-11 | Pass strings and large structures by const reference | David Robillard | 2 | -5/+5 | |
2024-10-11 | Call std::terminate() directly instead of rethrowing nothing | David Robillard | 1 | -0/+13 | |
2024-10-11 | Remove redundant method override | David Robillard | 1 | -7/+0 | |
2024-10-11 | Add missing const qualifiers | David Robillard | 6 | -20/+20 | |
2024-10-11 | Fix confusing iteration over collections of pointers | David Robillard | 2 | -5/+5 | |
2024-10-11 | Reduce variable scopes | David Robillard | 4 | -15/+15 | |
2024-10-11 | Make more single-argument constructors explicit | David Robillard | 9 | -12/+19 | |
2024-10-11 | Simplify PortType | David Robillard | 16 | -105/+99 | |
2024-10-06 | Fix inconsistent C-style cast | David Robillard | 1 | -1/+1 | |
2024-10-06 | Avoid using uninitialized va_list variables | David Robillard | 1 | -14/+20 | |
2024-10-06 | Explicitly destroy the world in LV2 plugin | David Robillard | 1 | -0/+1 | |
Makes no real difference, but avoids a seemingly unused variable. | |||||
2024-10-06 | Explicitly disallow copying or moving of App | David Robillard | 1 | -0/+5 | |
2024-10-06 | Avoid inefficient use of substr() to set strings to a prefix | David Robillard | 6 | -7/+7 | |
2024-10-06 | Remove dead code | David Robillard | 2 | -15/+0 | |
2024-10-06 | Remove redundant conditional clause | David Robillard | 1 | -1/+1 | |
2024-10-06 | Remove redundant assignment | David Robillard | 1 | -1/+0 | |
2024-07-17 | Add missing namespace comment | David Robillard | 1 | -1/+1 | |
2024-07-17 | Avoid use of jack_frame_time() | David Robillard | 2 | -22/+33 | |
This seems to be broken on the pipewire implementation of Jack, but we use FrameTimer for this on other drivers anyway, so just use that for Jack as well and avoid the issue entirely. Conveniently also exercises the clock and timestamp DLL code, which as it turns out, was itself broken. | |||||
2024-07-17 | Use uppercase integer literal suffixes | David Robillard | 5 | -8/+6 | |
2024-07-17 | Clean up includes and forward declarations | David Robillard | 34 | -25/+18 | |
According to include-what-you-use 0.22 on LLVM 18, anyway. Most of the changes seem vaguely reasonable, so as usual, just go with what the tool says because it's most useful when reports are typically clean. | |||||
2024-07-17 | Remove redundant inline specifier | David Robillard | 2 | -2/+1 | |
2024-07-17 | Avoid C-style casts and some size type conversions | David Robillard | 23 | -49/+55 | |
Aside from the syntactic cast changes, reduces some size types to 32-bits, since they can never be so large in practice. This eliminates some type conversions and shaves a few bytes. | |||||
2024-07-17 | Remove redundant casts | David Robillard | 14 | -22/+17 | |
2024-07-17 | Use empty() method instead of checking length | David Robillard | 4 | -5/+3 | |
2024-07-17 | Avoid return with void value | David Robillard | 3 | -3/+2 | |
2024-07-17 | Remove unused local variable | David Robillard | 2 | -2/+0 | |
2024-07-17 | Suppress new warnings in clang and clang-tidy 18 | David Robillard | 3 | -0/+8 | |