aboutsummaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorDavid Robillard <d@drobilla.net>2016-03-15 23:21:34 -0400
committerDavid Robillard <d@drobilla.net>2018-05-27 18:18:30 +0200
commit9b3d8263604ba11ae008c8651a1fe2063296d6b1 (patch)
tree9747d428f34f73e5d2e9e72bd8edef50b05ddad9 /src
parenta0c483d9f1f3955499560ef1536fdb760df274fb (diff)
downloadserd-9b3d8263604ba11ae008c8651a1fe2063296d6b1.tar.gz
serd-9b3d8263604ba11ae008c8651a1fe2063296d6b1.tar.bz2
serd-9b3d8263604ba11ae008c8651a1fe2063296d6b1.zip
Use SerdBuffer for mutable buffers
This avoids const violations from abusing SerdChunk as a mutable buffer for string sinks.
Diffstat (limited to 'src')
-rw-r--r--src/node.c17
-rw-r--r--src/uri.c10
-rw-r--r--src/writer.c14
3 files changed, 21 insertions, 20 deletions
diff --git a/src/node.c b/src/node.c
index 10df10d9..1472ad13 100644
--- a/src/node.c
+++ b/src/node.c
@@ -183,27 +183,28 @@ serd_node_new_file_uri(const uint8_t* path,
evil ? "/" : "");
}
- SerdChunk chunk = { uri, uri_len };
+ SerdBuffer buffer = { uri, uri_len };
for (size_t i = 0; i < path_len; ++i) {
if (evil && path[i] == '\\') {
- serd_chunk_sink("/", 1, &chunk);
+ serd_buffer_sink("/", 1, &buffer);
} else if (path[i] == '%') {
- serd_chunk_sink("%%", 2, &chunk);
+ serd_buffer_sink("%%", 2, &buffer);
} else if (!escape || is_uri_path_char(path[i])) {
- serd_chunk_sink(path + i, 1, &chunk);
+ serd_buffer_sink(path + i, 1, &buffer);
} else {
char escape_str[4] = { '%', 0, 0, 0 };
snprintf(escape_str + 1, sizeof(escape_str) - 1, "%X", path[i]);
- serd_chunk_sink(escape_str, 3, &chunk);
+ serd_buffer_sink(escape_str, 3, &buffer);
}
}
- serd_chunk_sink_finish(&chunk);
+ serd_buffer_sink_finish(&buffer);
if (out) {
- serd_uri_parse(chunk.buf, out);
+ serd_uri_parse((const uint8_t*)buffer.buf, out);
}
- return serd_node_from_substring(SERD_URI, chunk.buf, chunk.len);
+ return serd_node_from_substring(
+ SERD_URI, (const uint8_t*)buffer.buf, buffer.len);
}
SERD_API
diff --git a/src/uri.c b/src/uri.c
index 7b172e42..b13a1e6f 100644
--- a/src/uri.c
+++ b/src/uri.c
@@ -72,27 +72,27 @@ serd_file_uri_parse(const uint8_t* uri, uint8_t** hostname)
++path;
}
- SerdChunk chunk = { NULL, 0 };
+ SerdBuffer buffer = { NULL, 0 };
for (const uint8_t* s = path; *s; ++s) {
if (*s == '%') {
if (*(s + 1) == '%') {
- serd_chunk_sink("%", 1, &chunk);
+ serd_buffer_sink("%", 1, &buffer);
++s;
} else if (is_hexdig(*(s + 1)) && is_hexdig(*(s + 2))) {
const uint8_t code[3] = { *(s + 1), *(s + 2), 0 };
uint32_t num;
sscanf((const char*)code, "%X", &num);
const uint8_t c = num;
- serd_chunk_sink(&c, 1, &chunk);
+ serd_buffer_sink(&c, 1, &buffer);
s += 2;
} else {
s += 2; // Junk escape, ignore
}
} else {
- serd_chunk_sink(s, 1, &chunk);
+ serd_buffer_sink(s, 1, &buffer);
}
}
- return serd_chunk_sink_finish(&chunk);
+ return serd_buffer_sink_finish(&buffer);
}
SERD_API
diff --git a/src/writer.c b/src/writer.c
index 603ca65e..26bf5e5a 100644
--- a/src/writer.c
+++ b/src/writer.c
@@ -976,19 +976,19 @@ serd_file_sink(const void* buf, size_t len, void* stream)
SERD_API
size_t
-serd_chunk_sink(const void* buf, size_t len, void* stream)
+serd_buffer_sink(const void* buf, size_t len, void* stream)
{
- SerdChunk* chunk = (SerdChunk*)stream;
- chunk->buf = (uint8_t*)realloc((uint8_t*)chunk->buf, chunk->len + len);
- memcpy((uint8_t*)chunk->buf + chunk->len, buf, len);
- chunk->len += len;
+ SerdBuffer* buffer = (SerdBuffer*)stream;
+ buffer->buf = (char*)realloc(buffer->buf, buffer->len + len);
+ memcpy((uint8_t*)buffer->buf + buffer->len, buf, len);
+ buffer->len += len;
return len;
}
SERD_API
uint8_t*
-serd_chunk_sink_finish(SerdChunk* stream)
+serd_buffer_sink_finish(SerdBuffer* stream)
{
- serd_chunk_sink("", 1, stream);
+ serd_buffer_sink("", 1, stream);
return (uint8_t*)stream->buf;
}