Validation for empty objects
Now (correctly) fails to parse {,}
This commit is contained in:
1 parent
e98624e5a9
commit
22f8aed31e
3 files changed
+19
-1
No files matched your search
@@ -632,6 +632,15 @@ static int jvp_object_delete(jv_complex* object, jvp_string* key) {
|
|||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static int jvp_object_length(jv_complex* object) {
|
||||||
|
int n = 0;
|
||||||
|
for (int i=0; i<jvp_object_size(object); i++) {
|
||||||
|
struct object_slot* slot = jvp_object_get_slot(object, i);
|
||||||
|
if (slot->string) n++;
|
||||||
|
}
|
||||||
|
return n;
|
||||||
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Objects (public interface)
|
* Objects (public interface)
|
||||||
*/
|
*/
|
||||||
@@ -670,6 +679,13 @@ jv jv_object_delete(jv object, jv key) {
|
|||||||
return object;
|
return object;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
int jv_object_length(jv object) {
|
||||||
|
assert(jv_get_kind(object) == JV_KIND_OBJECT);
|
||||||
|
int n = jvp_object_length(&object.val.complex);
|
||||||
|
jv_free(object);
|
||||||
|
return n;
|
||||||
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Object iteration (internal helpers)
|
* Object iteration (internal helpers)
|
||||||
*/
|
*/
|
||||||
|
|||||||
@@ -109,6 +109,7 @@ jv jv_object();
|
|||||||
jv jv_object_get(jv object, jv key);
|
jv jv_object_get(jv object, jv key);
|
||||||
jv jv_object_set(jv object, jv key, jv value);
|
jv jv_object_set(jv object, jv key, jv value);
|
||||||
jv jv_object_delete(jv object, jv key);
|
jv jv_object_delete(jv object, jv key);
|
||||||
|
int jv_object_length(jv object);
|
||||||
|
|
||||||
int jv_object_iter(jv);
|
int jv_object_iter(jv);
|
||||||
int jv_object_iter_next(jv, int);
|
int jv_object_iter_next(jv, int);
|
||||||
|
|||||||
+2
-1
@@ -100,7 +100,8 @@ pfunc token(char ch) {
|
|||||||
} else {
|
} else {
|
||||||
if (jv_get_kind(stack[stackpos-1]) != JV_KIND_OBJECT)
|
if (jv_get_kind(stack[stackpos-1]) != JV_KIND_OBJECT)
|
||||||
return "Unmatched '}'";
|
return "Unmatched '}'";
|
||||||
// FIXME: assert object empty
|
if (jv_object_length(jv_copy(stack[stackpos-1])) != 0)
|
||||||
|
return "Expected another key-value pair";
|
||||||
}
|
}
|
||||||
hasnext = 1;
|
hasnext = 1;
|
||||||
next = stack[--stackpos];
|
next = stack[--stackpos];
|
||||||
|
|||||||
Reference in new issue
Block a user