make cue_compile_{string,bytes} return an error

Signed-off-by: Aram Hăvărneanu <[email protected]>

Author: 4ad

cue.h

@@ -99,8 +99,8 @@ extern "C" {
 cue_ctx	cue_newctx(void);
 char*	cue_error_string(cue_error);
 
-cue_value	cue_compile_string(cue_ctx, char*, cue_bopt*);
-cue_value	cue_compile_bytes(cue_ctx, void*, size_t, cue_bopt*);
+cue_error	cue_compile_string(cue_ctx, char*, cue_bopt*, cue_value*);
+cue_error	cue_compile_bytes(cue_ctx, void*, size_t, cue_bopt*, cue_value*);
 cue_value	cue_top(cue_ctx);
 cue_value	cue_bottom(cue_ctx);
 cue_value	cue_unify(cue_value, cue_value);

value.c

@@ -17,14 +17,14 @@
 #include "option.h"
 #include "_cgo_export.h"
 
-cue_value
-cue_compile_string(cue_ctx ctx, char *s, cue_bopt *opts) {
-	return cue_compile_string_raw(ctx, s, opts, cue_bopt_len(opts));
+cue_error
+cue_compile_string(cue_ctx ctx, char *s, cue_bopt *opts, cue_value *v) {
+	return cue_compile_string_raw(ctx, s, opts, cue_bopt_len(opts), v);
 }
 
-cue_value
-cue_compile_bytes(cue_ctx ctx, void *b, size_t len, cue_bopt *opts) {
-	return cue_compile_bytes_raw(ctx, b, len, opts, cue_bopt_len(opts));
+cue_error
+cue_compile_bytes(cue_ctx ctx, void *b, size_t len, cue_bopt *opts, cue_value *v) {
+	return cue_compile_bytes_raw(ctx, b, len, opts, cue_bopt_len(opts), v);
 }
 
 cue_error

value.go

@@ -28,17 +28,27 @@ import (
 )
 
 //export cue_compile_string_raw
-func cue_compile_string_raw(ctx C.cue_ctx, str *C.char, opts *C.struct_cue_bopt, len C.size_t) C.cue_value {
+func cue_compile_string_raw(ctx C.cue_ctx, str *C.char, opts *C.struct_cue_bopt, len C.size_t, v *C.cue_value) C.cue_error {
 	bopts := buildOptions(opts, len)
 	val := cueContext(ctx).CompileString(C.GoString(str), bopts...)
-	return cueValueHandle(val)
+
+	if err := val.Err(); err != nil {
+		return cueErrorHandle(err)
+	}
+	*v = cueValueHandle(val)
+	return 0
 }
 
 //export cue_compile_bytes_raw
-func cue_compile_bytes_raw(ctx C.cue_ctx, buf unsafe.Pointer, bufLen C.size_t, opts *C.struct_cue_bopt, optLen C.size_t) C.cue_value {
+func cue_compile_bytes_raw(ctx C.cue_ctx, buf unsafe.Pointer, bufLen C.size_t, opts *C.struct_cue_bopt, optLen C.size_t, v *C.cue_value) C.cue_error {
 	bopts := buildOptions(opts, optLen)
 	val := cueContext(ctx).CompileBytes(C.GoBytes(buf, C.int(bufLen)), bopts...)
-	return cueValueHandle(val)
+
+	if err := val.Err(); err != nil {
+		return cueErrorHandle(err)
+	}
+	*v = cueValueHandle(val)
+	return 0
 }
 
 //export cue_top