Use char* for strings in public API

The constant casting just makes user code a mess, for no benefit.

1 files changed, 9 insertions, 12 deletions

diff --git a/src/env.c b/src/env.c
index a9ae6af2..9df2c41e 100644
--- a/src/env.c
+++ b/src/env.c
@@ -17,7 +17,6 @@
 #include "serd/serd.h"
 
 #include <stdbool.h>
-#include <stdint.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
@@ -101,7 +100,7 @@ serd_env_set_base_uri(SerdEnv* const env, const SerdNode* const uri)
 
 SERD_PURE_FUNC static SerdPrefix*
 serd_env_find(const SerdEnv* const env,
-              const uint8_t* const name,
+              const char* const    name,
               const size_t         name_len)
 {
   for (size_t i = 0; i < env->n_prefixes; ++i) {
@@ -163,9 +162,9 @@ serd_env_set_prefix(SerdEnv* const        env,
 }
 
 SerdStatus
-serd_env_set_prefix_from_strings(SerdEnv* const       env,
-                                 const uint8_t* const name,
-                                 const uint8_t* const uri)
+serd_env_set_prefix_from_strings(SerdEnv* const    env,
+                                 const char* const name,
+                                 const char* const uri)
 {
   const SerdNode name_node = serd_node_from_string(SERD_LITERAL, name);
   const SerdNode uri_node  = serd_node_from_string(SERD_URI, uri);
@@ -186,9 +185,7 @@ serd_env_qualify(const SerdEnv* const  env,
   for (size_t i = 0; i < env->n_prefixes; ++i) {
     const SerdNode* const prefix_uri = &env->prefixes[i].uri;
     if (uri->n_bytes >= prefix_uri->n_bytes) {
-      if (!strncmp((const char*)uri->buf,
-                   (const char*)prefix_uri->buf,
-                   prefix_uri->n_bytes)) {
+      if (!strncmp(uri->buf, prefix_uri->buf, prefix_uri->n_bytes)) {
         *prefix     = env->prefixes[i].name;
         suffix->buf = uri->buf + prefix_uri->n_bytes;
         suffix->len = uri->n_bytes - prefix_uri->n_bytes;
@@ -209,8 +206,8 @@ serd_env_expand(const SerdEnv* const  env,
     return SERD_ERR_BAD_CURIE;
   }
 
-  const uint8_t* const colon =
-    (const uint8_t*)memchr(curie->buf, ':', curie->n_bytes + 1);
+  const char* const colon =
+    (const char*)memchr(curie->buf, ':', curie->n_bytes + 1);
   if (curie->type != SERD_CURIE || !colon) {
     return SERD_ERR_BAD_ARG;
   }
@@ -249,9 +246,9 @@ serd_env_expand_node(const SerdEnv* const env, const SerdNode* const node)
       return SERD_NODE_NULL;
     }
     const size_t len = prefix.len + suffix.len;
-    uint8_t*     buf = (uint8_t*)malloc(len + 1);
+    char*        buf = (char*)malloc(len + 1);
     SerdNode     ret = {buf, len, 0, SERD_URI};
-    snprintf((char*)buf, ret.n_bytes + 1, "%s%s", prefix.buf, suffix.buf);
+    snprintf(buf, ret.n_bytes + 1, "%s%s", prefix.buf, suffix.buf);
     return ret;
   }
   case SERD_BLANK: