From 64024d0fa6a6dc048b2b846738846da597025f56 Mon Sep 17 00:00:00 2001
From: David Robillard <d@drobilla.net>
Date: Thu, 25 Feb 2021 11:16:13 -0500
Subject: Leave statement caret at the start of literals

This allows a precise location to be reported for errors within literals, by
adding the offset of the error in the literal to the caret.  This will be used
to report nice errors for things like regular expressions and supported XSD
datatypes.
---
 src/read_turtle.c |  5 ++++-
 src/reader.c      | 17 +++++++++++++----
 src/reader.h      |  7 +++++++
 3 files changed, 24 insertions(+), 5 deletions(-)

(limited to 'src')

diff --git a/src/read_turtle.c b/src/read_turtle.c
index 027f0918..c3970a1e 100644
--- a/src/read_turtle.c
+++ b/src/read_turtle.c
@@ -13,6 +13,7 @@
 #include "try.h"
 #include "turtle.h"
 
+#include "serd/caret.h"
 #include "serd/node.h"
 #include "serd/reader.h"
 #include "serd/sink.h"
@@ -610,6 +611,7 @@ read_object(SerdReader* const  reader,
             bool* const        ate_dot)
 {
   const size_t orig_stack_size = reader->stack.size;
+  SerdCaret    orig_caret      = reader->source->caret;
 
   assert(ctx->subject);
 
@@ -660,6 +662,7 @@ read_object(SerdReader* const  reader,
     break;
   case '\"':
   case '\'':
+    ++orig_caret.col;
     st = read_literal(reader, &o, ate_dot);
     break;
   default:
@@ -668,7 +671,7 @@ read_object(SerdReader* const  reader,
   }
 
   if (!st && simple && o) {
-    st = emit_statement(reader, *ctx, o);
+    st = emit_statement_at(reader, *ctx, o, &orig_caret);
   }
 
   serd_stack_pop_to(&reader->stack, orig_stack_size);
diff --git a/src/reader.c b/src/reader.c
index f34f7974..7e0864cc 100644
--- a/src/reader.c
+++ b/src/reader.c
@@ -162,9 +162,10 @@ tokcmp(const SerdNode* const node, const char* const tok, const size_t n)
 }
 
 SerdStatus
-emit_statement(SerdReader* const reader,
-               const ReadContext ctx,
-               SerdNode* const   o)
+emit_statement_at(SerdReader* const reader,
+                  const ReadContext ctx,
+                  SerdNode* const   o,
+                  SerdCaret* const  caret)
 {
   if (reader->stack.size + (2 * sizeof(SerdNode)) > reader->stack.buf_size) {
     return SERD_BAD_STACK;
@@ -175,7 +176,7 @@ emit_statement(SerdReader* const reader,
   serd_node_zero_pad(o);
 
   const SerdStatement statement = {{ctx.subject, ctx.predicate, o, ctx.graph},
-                                   &reader->source->caret};
+                                   caret};
 
   const SerdStatus st =
     serd_sink_write_statement(reader->sink, *ctx.flags, &statement);
@@ -184,6 +185,14 @@ emit_statement(SerdReader* const reader,
   return st;
 }
 
+SerdStatus
+emit_statement(SerdReader* const reader,
+               const ReadContext ctx,
+               SerdNode* const   o)
+{
+  return emit_statement_at(reader, ctx, o, &reader->source->caret);
+}
+
 SerdStatus
 serd_reader_read_document(SerdReader* const reader)
 {
diff --git a/src/reader.h b/src/reader.h
index 601453b0..132d038e 100644
--- a/src/reader.h
+++ b/src/reader.h
@@ -10,6 +10,7 @@
 #include "try.h"
 
 #include "serd/attributes.h"
+#include "serd/caret.h"
 #include "serd/node.h"
 #include "serd/reader.h"
 #include "serd/sink.h"
@@ -84,6 +85,12 @@ blank_id(SerdReader* reader);
 void
 set_blank_id(SerdReader* reader, SerdNode* node, size_t buf_size);
 
+SerdStatus
+emit_statement_at(SerdReader* reader,
+                  ReadContext ctx,
+                  SerdNode*   o,
+                  SerdCaret*  caret);
+
 SerdStatus
 emit_statement(SerdReader* reader, ReadContext ctx, SerdNode* o);
 
-- 
cgit v1.2.1