When an extension calls sym_lookup on a deferred variable, it should always succeed.

1 files changed, 8 insertions, 10 deletions

diff --git a/extension/testext.c b/extension/testext.c
index 7c61bb0d..42ec0915 100644
--- a/extension/testext.c
+++ b/extension/testext.c
@@ -303,11 +303,11 @@ var_test(int nargs, awk_value_t *result)
 		goto out;
 	}
 
-	/* look up PROCINFO - should fail */
+	/* look up PROCINFO - should succeed */
 	if (sym_lookup("PROCINFO", AWK_ARRAY, & value))
-		printf("var_test: sym_lookup of PROCINFO failed - got a value!\n");
+		printf("var_test: sym_lookup of PROCINFO passed - got a value!\n");
 	else
-		printf("var_test: sym_lookup of PROCINFO passed - did not get a value\n");
+		printf("var_test: sym_lookup of PROCINFO failed - did not get a value\n");
 
 	/* look up a reserved variable - should pass */
 	if (sym_lookup("ARGC", AWK_NUMBER, & value))
@@ -399,8 +399,11 @@ test_deferred(int nargs, awk_value_t *result)
 		printf("test_deferred: nargs not right (%d should be 0)\n", nargs);
 		goto out;
 	}
-	arr.val_type = AWK_ARRAY;
-	arr.array_cookie = create_array();
+
+	if (! sym_lookup("PROCINFO", AWK_ARRAY, & arr)) {
+		printf("test_deferred: %d: sym_lookup failed\n", __LINE__);
+		goto out;
+	}
 
 	for (i = 0; i < sizeof(seed)/sizeof(seed[0]); i++) {
 		make_const_string(seed[i].name, strlen(seed[i].name), & index);
@@ -411,11 +414,6 @@ test_deferred(int nargs, awk_value_t *result)
 		}
 	}
 
-	if (! sym_update("PROCINFO", & arr)) {
-		printf("test_deferred: %d: sym_update failed\n", __LINE__);
-		goto out;
-	}
-
 	/* test that it still contains the values we loaded */
 	for (i = 0; i < sizeof(seed)/sizeof(seed[0]); i++) {
 		make_const_string(seed[i].name, strlen(seed[i].name), & index);