]> granicus.if.org Git - postgresql/commit
Fix thinko in JsObjectSize() macro.
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 29 May 2017 22:51:56 +0000 (18:51 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 29 May 2017 22:51:56 +0000 (18:51 -0400)
commite45c5be99d08d7bb6708d7bb1dd0f5d99798c6aa
tree8e49f4b90ada4e0d49d94c9ff9c34bcf2a349eb0
parentf3db7f164a29c5cbdc1d6d5d0d23854df58783c1
Fix thinko in JsObjectSize() macro.

The macro gave the wrong answers for a JsObject with is_json == 0:
it would return 1 if jsonb_cont == NULL, or if that wasn't NULL,
it would return 1 for any non-zero size.

We could fix that, but the only use of this macro at present is in the
JsObjectIsEmpty() macro, so it seems simpler and clearer to get rid of
JsObjectSize() and put corrected logic into JsObjectIsEmpty().

Thinko in commit cf35346e8, so no need for back-patch.

Nikita Glukhov

Discussion: https://postgr.es/m/fbd1d566-bba0-a3de-d6d0-d3b1d7c24ff2@postgrespro.ru
src/backend/utils/adt/jsonfuncs.c