Make serd_writer_set_root_uri() take a string view

This is generally more convenient, and the node was just being copied anyway.

2 files changed, 4 insertions, 6 deletions

diff --git a/src/serdi.c b/src/serdi.c
index ef5a7633..2de41ce7 100644
--- a/src/serdi.c
+++ b/src/serdi.c
@@ -478,9 +478,7 @@ main(int argc, char** argv)
   }
 
   if (root_uri) {
-    SerdNode* const root = serd_new_uri(SERD_STRING(root_uri));
-    serd_writer_set_root_uri(writer, root);
-    serd_node_free(root);
+    serd_writer_set_root_uri(writer, SERD_STRING(root_uri));
   }
 
   serd_writer_chop_blank_prefix(writer, chop_prefix);
diff --git a/src/writer.c b/src/writer.c
index e56f0b75..968d1a9f 100644
--- a/src/writer.c
+++ b/src/writer.c
@@ -1389,7 +1389,7 @@ serd_writer_set_base_uri(SerdWriter* writer, const SerdNode* uri)
 }
 
 SerdStatus
-serd_writer_set_root_uri(SerdWriter* writer, const SerdNode* uri)
+serd_writer_set_root_uri(SerdWriter* writer, const SerdStringView uri)
 {
   assert(writer);
 
@@ -1397,8 +1397,8 @@ serd_writer_set_root_uri(SerdWriter* writer, const SerdNode* uri)
   writer->root_node = NULL;
   writer->root_uri  = SERD_URI_NULL;
 
-  if (uri) {
-    writer->root_node = serd_node_copy(uri);
+  if (uri.len) {
+    writer->root_node = serd_new_uri(uri);
     writer->root_uri  = serd_node_uri_view(writer->root_node);
   }