Re: pgsql: Add basic JSON_TABLE() functionality

From: Erik Rijkers <er(at)xs4all(dot)nl>
To: Amit Langote <amitlan(at)postgresql(dot)org>, pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Add basic JSON_TABLE() functionality
Date: 2024-04-04 12:30:31
Message-ID: 254b2fa2-2f6b-a30a-20ee-21f8a2c12a50@xs4all.nl
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Op 4/4/24 om 13:21 schreef Amit Langote:
> Add basic JSON_TABLE() functionality
>

Great that it's now committed. Congrats!

There is one 'uninitialized' muttering during compile (gcc 13.2.0):

In function ‘transformJsonFuncExpr’,
inlined from ‘transformExprRecurse’ at parse_expr.c:373:13:
parse_expr.c:4317:34: warning: ‘default_format’ may be used
uninitialized [-Wmaybe-uninitialized]
4317 | jsexpr->formatted_expr = transformJsonValueExpr(pstate,
func_name,
|
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
4318 |
func->context_item,
|
~~~~~~~~~~~~~~~~~~~
4319 |
default_format,
|
~~~~~~~~~~~~~~~
4320 |
JSONBOID,
|
~~~~~~~~~
4321 |
false);
|
~~~~~~
parse_expr.c: In function ‘transformExprRecurse’:
parse_expr.c:4257:24: note: ‘default_format’ was declared here
4257 | JsonFormatType default_format;
| ^~~~~~~~~~~~~~

Thanks!

Erik

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Andrew Dunstan 2024-04-04 13:06:06 Re: pgsql: Add basic JSON_TABLE() functionality
Previous Message Amit Langote 2024-04-04 11:57:23 pgsql: Fix typo introduced in 6185c9737