diff options
| author | David Robillard <d@drobilla.net> | 2023-12-01 20:39:44 -0500 |
|---|---|---|
| committer | David Robillard <d@drobilla.net> | 2023-12-02 18:49:08 -0500 |
| commit | 02d56e83931e53e1cde57247c64d56fda3804f77 (patch) | |
| tree | 2d1ac467bc56f4f4f3570497427be32d7e36bd1a /src/nodes.c | |
| parent | d094448c095a59117febc8bd4687df071ce9759a (diff) | |
| download | serd-02d56e83931e53e1cde57247c64d56fda3804f77.tar.gz serd-02d56e83931e53e1cde57247c64d56fda3804f77.tar.bz2 serd-02d56e83931e53e1cde57247c64d56fda3804f77.zip | |
[WIP] Tighten up reader node management
[WIP] Broken on 32-bit
This makes the reader stack manipulations stricter, to make the code more
regular and avoid redundant work and bad cache activity. Now, functions that
push node headers and their bodies are responsible for (more or less)
immediately pushing any trailing null bytes required for termination and
alignment.
This makes the writes to the node in the stack more local, ensures nodes are
terminated as early as possible (to reduce the risk of using non-terminated
strings), and avoids the need to calculate aligned stack allocations.
Diffstat (limited to 'src/nodes.c')
| -rw-r--r-- | src/nodes.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/nodes.c b/src/nodes.c index e353b9aa..dae67077 100644 --- a/src/nodes.c +++ b/src/nodes.c @@ -198,7 +198,7 @@ node_equals_spec(const SerdNode* const node, const NodeSpec* const spec) !strcmp(serd_node_string_i(serd_node_meta_c(node)), spec->meta.data)); } -static bool +ZIX_PURE_FUNC static bool nodes_meta_equal(const SerdNode* const a, const SerdNode* const b) { assert(a->flags & meta_mask); @@ -213,7 +213,7 @@ nodes_meta_equal(const SerdNode* const a, const SerdNode* const b) !memcmp(serd_node_string_i(am), serd_node_string_i(bm), am->length); } -static bool +ZIX_PURE_FUNC static bool nodes_equal(const SerdNode* const a, const SerdNode* const b) { return (a == b) || |