*printf %y arg results in period separators instead of commas for lists and hashes

[davidnich]

ex:

2020-06-10 08:46:47.167245 qorus-0:65 T6: publishing event: ["MR-PUB-PROCESS-MEMORY-CHANGE". {id: "qdsp-omq", restarted: False, node: "qorus-0", status: 2, status_string: "RUNNING", urls: ["tcp://172.17.0.4:44479"], host: "qorus-0", pid: 51425, type: "qdsp", client_id: "omq", started: 2020-06-10 05:53:28.978890 Wed +00:00 (UTC), vsz: 2126946304, rss: 32739328, priv: 24446976, priv_str: "23.31 MiB", connstr: "pgsql:omqk8s/omqk8s@omqk8s%manatee{min=3,max=10}", pct: 0, node_priv: 2340648960, node_priv_str: "2.18 GiB", node_ram: 134965501952, node_ram_str: "125.70 GiB", node_ram_in_use: 103624409088, node_ram_in_use_str: "96.51 GiB", node_cpu_count: 32, node_load_pct: 1.71875}]

code:

QDBG_LOG("publishing event: %y", v);

ex:

2020-06-10 08:46:46.896981 qorus-0:65 T7: QorusMaster::publishRemoteNodeInfo() "qorus-1": {node_priv: 977789952. node_priv_str: "932.49 MiB". node_ram_in_use: 104507805696, node_ram_in_use_str: "97.33 GiB", node_cpu_count: 32, node_load_pct: 1.90625, machine_ram_total: 134965501952, machine_load_pct: 1.59375}

code:

QDBG_LOG("QorusMaster::publishRemoteNodeInfo() %y: %y", node, node_info);

C++ code for lists in QoreListNode.cpp:

    if (foff == FMT_YAML_SHORT) {
        str.concat('[');
        ConstListIterator li(this);
        while (li.next()) {
            QoreValue n = li.getValue();
            if (n.getAsString(str, foff, xsink))
                return -1;
            if (!li.last())
                str.concat(", ");
        }
        str.concat(']');
        return 0;
    }

C++ code for hashes in QoreHashNode.cpp:

if (foff == FMT_YAML_SHORT) {
        str.concat('[');
        ConstListIterator li(this);
        while (li.next()) {
            QoreValue n = li.getValue();
            if (n.getAsString(str, foff, xsink))
                return -1;
            if (!li.last())
                str.concat(", ");
        }
        str.concat(']');
        return 0;
    }

[davidnich]

the bug was in calls to internal function q_fix_decimal()