mirror/stedolan-jq
1
0
Fork 0
mirror of https://github.com/stedolan/jq.git synced 2024-05-11 05:55:39 +00:00
Code Issues Releases Activity

jq_compile_args(): allow object args to be object

Browse Source
This commit is contained in:
Nicolas Williams
2017-02-19 18:08:03 -06:00
parent 7b1bd99de6
commit e8abc0a2f6
1 changed files with 26 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
src/execute.c
View File

@ -1068,7 +1068,7 @@ static struct bytecode *optimize(struct bytecode *bc) {
int jq_compile_args(jq_state *jq, const char* str, jv args) {
jv_nomem_handler(jq->nomem_handler, jq->nomem_handler_data);
assert(jv_get_kind(args) == JV_KIND_ARRAY);
assert(jv_get_kind(args) == JV_KIND_ARRAY || jv_get_kind(args) == JV_KIND_OBJECT);
struct locfile* locations;
locations = locfile_init(jq, "<top-level>", str, strlen(str));
block program;
@ -1079,12 +1079,32 @@ int jq_compile_args(jq_state *jq, const char* str, jv args) {
}
int nerrors = load_program(jq, locations, &program);
if (nerrors == 0) {
/*
* jq_compile_args() is a public function. It should always have
* been the case that args was an object, but originally it was an
* array.
*
* We have to do an O(N) binding operation for each argument given.
*
* The best way to address this might be to have a bind function
* that takes an object instead of a const block and a varname.
*
* However, with `--args` this can be avoided, as one can have a
* single such binding then: $ARGS.
*/
if (jv_get_kind(args) == JV_KIND_ARRAY) {
jv_array_foreach(args, i, arg) {
jv name = jv_object_get(jv_copy(arg), jv_string("name"));
jv value = jv_object_get(arg, jv_string("value"));
program = gen_var_binding(gen_const(value), jv_string_value(name), program);
jv_free(name);
}
} else if (jv_get_kind(args) == JV_KIND_OBJECT) {
jv_object_foreach(args, name, value) {
program = gen_var_binding(gen_const(value), jv_string_value(name), program);
jv_free(name);
}
}
nerrors = builtins_bind(jq, &program);
if (nerrors == 0) {