summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Robillard <d@drobilla.net>2022-06-28 14:31:05 -0400
committerDavid Robillard <d@drobilla.net>2022-06-28 15:10:15 -0400
commit088fee1f4b4586e96c3b29314678d05a53eed9c4 (patch)
treeb9f4a7a691275b92058d268e2f3f181f7358dcb8
parentb5a4d1f58efd8e7976349ac2592b5ae2dbb86fc2 (diff)
downloadzix-088fee1f4b4586e96c3b29314678d05a53eed9c4.tar.gz
zix-088fee1f4b4586e96c3b29314678d05a53eed9c4.tar.bz2
zix-088fee1f4b4586e96c3b29314678d05a53eed9c4.zip
Fix incorrect function attributes
The ring accessors are pure, not const, because they read pointed-to data (the ring) that may change between invocations. The BTree iter comparison is const because it only compares the values passed as parameters (although they contain pointers, they aren't dereferenced).
-rw-r--r--include/zix/btree.h2
-rw-r--r--include/zix/ring.h6
2 files changed, 4 insertions, 4 deletions
diff --git a/include/zix/btree.h b/include/zix/btree.h
index af10afb..3f3e28b 100644
--- a/include/zix/btree.h
+++ b/include/zix/btree.h
@@ -180,7 +180,7 @@ ZixBTreeIter
zix_btree_end(const ZixBTree* ZIX_NULLABLE t);
/// Return true iff `lhs` is equal to `rhs`
-ZIX_PURE_API
+ZIX_CONST_API
bool
zix_btree_iter_equals(ZixBTreeIter lhs, ZixBTreeIter rhs);
diff --git a/include/zix/ring.h b/include/zix/ring.h
index d7d9713..dc3d4ce 100644
--- a/include/zix/ring.h
+++ b/include/zix/ring.h
@@ -67,17 +67,17 @@ void
zix_ring_reset(ZixRing* ZIX_NONNULL ring);
/// Return the number of bytes of space available for reading
-ZIX_CONST_API
+ZIX_PURE_API
uint32_t
zix_ring_read_space(const ZixRing* ZIX_NONNULL ring);
/// Return the number of bytes of space available for writing
-ZIX_CONST_API
+ZIX_PURE_API
uint32_t
zix_ring_write_space(const ZixRing* ZIX_NONNULL ring);
/// Return the capacity (i.e. total write space when empty)
-ZIX_CONST_API
+ZIX_PURE_API
uint32_t
zix_ring_capacity(const ZixRing* ZIX_NONNULL ring);