summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorRichard Braun <rbraun@sceen.net>2018-12-27 21:16:52 +0100
committerRichard Braun <rbraun@sceen.net>2018-12-27 21:16:52 +0100
commitfc51b4a0b2f4f3e0546054b8262afded30e1f1c1 (patch)
treec00bc5f3e19c017195971d9146ff236bf8a79cee /src
parent562033bdc83fcd0ceff958faf1ce070ca1c48eec (diff)
cbuf: remove range checking function
There isn't a single case where the range check cannot be replaced with an index check, which is slightly lighter.
Diffstat (limited to 'src')
-rw-r--r--src/cbuf.c4
-rw-r--r--src/cbuf.h8
2 files changed, 2 insertions, 10 deletions
diff --git a/src/cbuf.c b/src/cbuf.c
index 2baad37..a176b2a 100644
--- a/src/cbuf.c
+++ b/src/cbuf.c
@@ -143,13 +143,13 @@ cbuf_write(struct cbuf *cbuf, size_t index, const void *buf, size_t size)
uint8_t *start, *end, *buf_end;
size_t new_end, skip;
- if (!cbuf_range_valid(cbuf, index, cbuf->end)) {
+ if (!cbuf_index_valid(cbuf, index)) {
return EINVAL;
}
new_end = index + size;
- if (!cbuf_range_valid(cbuf, cbuf->start, new_end)) {
+ if (!cbuf_index_valid(cbuf, new_end)) {
cbuf->end = new_end;
if (size > cbuf_capacity(cbuf)) {
diff --git a/src/cbuf.h b/src/cbuf.h
index 354cc34..6b6a844 100644
--- a/src/cbuf.h
+++ b/src/cbuf.h
@@ -89,14 +89,6 @@ cbuf_index_valid(const struct cbuf *cbuf, size_t index)
&& ((cbuf->end - index) <= cbuf_size(cbuf));
}
-static inline bool
-cbuf_range_valid(const struct cbuf *cbuf, size_t start, size_t end)
-{
- return (((end - start) <= cbuf_size(cbuf))
- && ((start - cbuf->start) <= cbuf_size(cbuf))
- && ((cbuf->end - end) <= cbuf_size(cbuf)));
-}
-
/*
* Initialize a circular buffer.
*