diff options
| author | Tom Lane | 2013-04-28 04:18:45 +0000 |
|---|---|---|
| committer | Tom Lane | 2013-04-28 04:18:45 +0000 |
| commit | f8db76e875099e5e49f5cd729a673e84c0b0471b (patch) | |
| tree | 162e43ab4a10fbeef78201e9adc87c433fe9d197 /contrib/sepgsql/hooks.c | |
| parent | 5525e6c40bbda351a19b48317eba0f79aa32e447 (diff) | |
Editorialize a bit on new ProcessUtility() API.
Choose a saner ordering of parameters (adding a new input param after
the output params seemed a bit random), update the function's header
comment to match reality (cmon folks, is this really that hard?),
get rid of useless and sloppily-defined distinction between
PROCESS_UTILITY_SUBCOMMAND and PROCESS_UTILITY_GENERATED.
Diffstat (limited to 'contrib/sepgsql/hooks.c')
| -rw-r--r-- | contrib/sepgsql/hooks.c | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/contrib/sepgsql/hooks.c b/contrib/sepgsql/hooks.c index 04c5120890..a1320e8638 100644 --- a/contrib/sepgsql/hooks.c +++ b/contrib/sepgsql/hooks.c @@ -299,10 +299,10 @@ sepgsql_exec_check_perms(List *rangeTabls, bool abort) static void sepgsql_utility_command(Node *parsetree, const char *queryString, + ProcessUtilityContext context, ParamListInfo params, DestReceiver *dest, - char *completionTag, - ProcessUtilityContext context) + char *completionTag) { sepgsql_context_info_t saved_context_info = sepgsql_context_info; ListCell *cell; @@ -362,11 +362,13 @@ sepgsql_utility_command(Node *parsetree, } if (next_ProcessUtility_hook) - (*next_ProcessUtility_hook) (parsetree, queryString, params, - dest, completionTag, context); + (*next_ProcessUtility_hook) (parsetree, queryString, + context, params, + dest, completionTag); else - standard_ProcessUtility(parsetree, queryString, params, - dest, completionTag, context); + standard_ProcessUtility(parsetree, queryString, + context, params, + dest, completionTag); } PG_CATCH(); { |