diff options
| author | Peter Eisentraut <peter@eisentraut.org> | 2024-11-28 08:19:22 +0100 |
|---|---|---|
| committer | Peter Eisentraut <peter@eisentraut.org> | 2024-11-28 08:27:20 +0100 |
| commit | 7f798aca1d5df290aafad41180baea0ae311b4ee (patch) | |
| tree | b4c8132ec85c21f6c72308b5857defd70958de69 /src/backend/parser/parse_param.c | |
| parent | 97525bc5c8ffb31475d23955d08e9ec9c1408f33 (diff) | |
| download | postgresql-7f798aca1d5df290aafad41180baea0ae311b4ee.tar.gz postgresql-7f798aca1d5df290aafad41180baea0ae311b4ee.zip | |
Remove useless casts to (void *)
Many of them just seem to have been copied around for no real reason.
Their presence causes (small) risks of hiding actual type mismatches
or silently discarding qualifiers
Discussion: https://www.postgresql.org/message-id/flat/461ea37c-8b58-43b4-9736-52884e862820@eisentraut.org
Diffstat (limited to 'src/backend/parser/parse_param.c')
| -rw-r--r-- | src/backend/parser/parse_param.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/backend/parser/parse_param.c b/src/backend/parser/parse_param.c index b617591ef68..1ec6c920fbc 100644 --- a/src/backend/parser/parse_param.c +++ b/src/backend/parser/parse_param.c @@ -72,7 +72,7 @@ setup_parse_fixed_parameters(ParseState *pstate, parstate->paramTypes = paramTypes; parstate->numParams = numParams; - pstate->p_ref_hook_state = (void *) parstate; + pstate->p_ref_hook_state = parstate; pstate->p_paramref_hook = fixed_paramref_hook; /* no need to use p_coerce_param_hook */ } @@ -88,7 +88,7 @@ setup_parse_variable_parameters(ParseState *pstate, parstate->paramTypes = paramTypes; parstate->numParams = numParams; - pstate->p_ref_hook_state = (void *) parstate; + pstate->p_ref_hook_state = parstate; pstate->p_paramref_hook = variable_paramref_hook; pstate->p_coerce_param_hook = variable_coerce_param_hook; } @@ -274,7 +274,7 @@ check_variable_parameters(ParseState *pstate, Query *query) if (*parstate->numParams > 0) (void) query_tree_walker(query, check_parameter_resolution_walker, - (void *) pstate, 0); + pstate, 0); } /* @@ -318,10 +318,10 @@ check_parameter_resolution_walker(Node *node, ParseState *pstate) /* Recurse into RTE subquery or not-yet-planned sublink subquery */ return query_tree_walker((Query *) node, check_parameter_resolution_walker, - (void *) pstate, 0); + pstate, 0); } return expression_tree_walker(node, check_parameter_resolution_walker, - (void *) pstate); + pstate); } /* |