about summary refs log tree commit diff
diff options
context:
space:
mode:
authorMikael Magnusson <mikachu@gmail.com>2015-12-13 17:35:12 +0100
committerMikael Magnusson <mikachu@gmail.com>2016-05-08 21:57:06 +0200
commit546a6b42b3a2b1172fbd099a0fee2864af16461c (patch)
tree5d3c312593262f892fc2ef0bf387c8cf9456367d
parent7db77ff1bb173a3db5501b195da9effc3b4fbe53 (diff)
downloadzsh-546a6b42b3a2b1172fbd099a0fee2864af16461c.tar.gz
zsh-546a6b42b3a2b1172fbd099a0fee2864af16461c.tar.xz
zsh-546a6b42b3a2b1172fbd099a0fee2864af16461c.zip
Assert on problems
-rw-r--r--Src/builtin.c3
-rw-r--r--Src/params.c14
2 files changed, 15 insertions, 2 deletions
diff --git a/Src/builtin.c b/Src/builtin.c
index 27355feff..534f49b57 100644
--- a/Src/builtin.c
+++ b/Src/builtin.c
@@ -30,6 +30,8 @@
 /* this is defined so we get the prototype for open_memstream */
 #define _GNU_SOURCE 1
 
+#include <assert.h>
+
 #include "zsh.mdh"
 #include "builtin.pro"
 
@@ -3404,6 +3406,7 @@ bin_unset(char *name, char **argv, Options ops, int func)
 		    } else {
 			/* start is after the element for reverse index */
 			int start = vbuf.start - !!(vbuf.flags & VALFLAG_INV);
+			assert(vbuf.pm->length == arrlen(vbuf.pm->u.arr));
 			if (start < vbuf.pm->length) {
 			    char *arr[2];
 			    arr[0] = "";
diff --git a/Src/params.c b/Src/params.c
index cda9252ec..cdaa59a92 100644
--- a/Src/params.c
+++ b/Src/params.c
@@ -27,6 +27,8 @@
  *
  */
 
+#include <assert.h>
+
 #include "zsh.mdh"
 #include "params.pro"
 
@@ -1453,6 +1455,7 @@ getarg(char **str, int *inv, Value v, int a2, zlong *w,
 	    if (!ta || !*ta)
 		return !down;
 	    len = v->pm->length;
+	    assert(len == arrlen(ta));
 	    if (beg < 0)
 		beg += len;
 	    if (down) {
@@ -1476,6 +1479,7 @@ getarg(char **str, int *inv, Value v, int a2, zlong *w,
 	} else if (word) {
 	    ta = sepsplit(d = s = getstrvalue(v), sep, 1, 1);
 	    len = v->pm->length;
+	    assert(len == arrlen(ta));
 	    if (beg < 0)
 		beg += len;
 	    if (down) {
@@ -2025,6 +2029,7 @@ getstrvalue(Value v)
 	if (v->isarr)
 	    s = sepjoin(ss, NULL, 1);
 	else {
+	    assert(v->pm->length == arrlen(ss));
 	    if (v->start < 0)
 		v->start += v->pm->length;
 	    s = (v->start >= v->pm->length || v->start < 0) ?
@@ -2222,6 +2227,7 @@ getarrvalue(Value v)
     s = getvaluearr(v);
     if (v->start == 0 && v->end == -1)
 	return s;
+    assert(v->pm->length == arrlen(s));
     if (v->start < 0)
 	v->start += v->pm->length;
     if (v->end < 0)
@@ -2559,6 +2565,7 @@ setarrvalue(Value v, char **val)
 	int post_assignment_length;
 	int i;
 
+	assert(v->pm->length == arrlen(old));
 	q = old;
 
 	if ((v->flags & VALFLAG_INV) && unset(KSHARRAYS)) {
@@ -2668,9 +2675,10 @@ getaparam(char *s, int *len)
     if (!idigit(*s) && (v = getvalue(&vbuf, &s, 0)) &&
 	PM_TYPE(v->pm->node.flags) == PM_ARRAY)
     {
-	if (len)
+	if (len) {
 	    *len = v->pm->length;
-	    //*len = arrlen(v->pm->gsu.a->getfn(v->pm));
+	    assert (*len == arrlen(v->pm->gsu.a->getfn(v->pm)));
+	}
 	//fprintf(stderr, "%i %i\n", v->pm->length, arrlen(v->pm->gsu.a->getfn(v->pm)));
 	return v->pm->gsu.a->getfn(v->pm);
     }
@@ -2938,6 +2946,7 @@ assignaparam(char *s, char **val, int flags)
     if (flags & ASSPM_AUGMENT) {
     	if (v->start == 0 && v->end == -1) {
 	    if (PM_TYPE(v->pm->node.flags) & PM_ARRAY) {
+		assert(v->pm->length == arrlen(v->pm->gsu.a->getfn(v->pm)));
 	    	v->start = 
 		    //arrlen(v->pm->gsu.a->getfn(v->pm));
 		    v->pm->length;
@@ -2948,6 +2957,7 @@ assignaparam(char *s, char **val, int flags)
 	    if (v->end > 0)
 		v->start = v->end--;
 	    else if (PM_TYPE(v->pm->node.flags) & PM_ARRAY) {
+		assert(v->pm->length == arrlen(v->pm->gsu.a->getfn(v->pm)));
 		v->end 
 		    //= arrlen(v->pm->gsu.a->getfn(v->pm)) + v->end;
 		    += v->pm->length;