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_collate.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_collate.c')
| -rw-r--r-- | src/backend/parser/parse_collate.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/src/backend/parser/parse_collate.c b/src/backend/parser/parse_collate.c index 44529bb49e6..2874731c352 100644 --- a/src/backend/parser/parse_collate.c +++ b/src/backend/parser/parse_collate.c @@ -108,7 +108,7 @@ assign_query_collations(ParseState *pstate, Query *query) */ (void) query_tree_walker(query, assign_query_collations_walker, - (void *) pstate, + pstate, QTW_IGNORE_RANGE_TABLE | QTW_IGNORE_CTE_SUBQUERIES); } @@ -295,7 +295,7 @@ assign_collations_walker(Node *node, assign_collations_context *context) (void) expression_tree_walker(node, assign_collations_walker, - (void *) &loccontext); + &loccontext); collation = expr->collOid; Assert(OidIsValid(collation)); @@ -317,7 +317,7 @@ assign_collations_walker(Node *node, assign_collations_context *context) /* ... but first, recurse */ (void) expression_tree_walker(node, assign_collations_walker, - (void *) &loccontext); + &loccontext); if (OidIsValid(expr->resultcollid)) { @@ -403,7 +403,7 @@ assign_collations_walker(Node *node, assign_collations_context *context) /* ... but first, recurse */ (void) expression_tree_walker(node, assign_collations_walker, - (void *) &loccontext); + &loccontext); if (OidIsValid(typcollation)) { @@ -444,7 +444,7 @@ assign_collations_walker(Node *node, assign_collations_context *context) case T_TargetEntry: (void) expression_tree_walker(node, assign_collations_walker, - (void *) &loccontext); + &loccontext); /* * TargetEntry can have only one child, and should bubble that @@ -488,7 +488,7 @@ assign_collations_walker(Node *node, assign_collations_context *context) case T_MergeAction: (void) expression_tree_walker(node, assign_collations_walker, - (void *) &loccontext); + &loccontext); /* * When we're invoked on a query's jointree, we don't need to do @@ -528,7 +528,7 @@ assign_collations_walker(Node *node, assign_collations_context *context) case T_List: (void) expression_tree_walker(node, assign_collations_walker, - (void *) &loccontext); + &loccontext); /* * When processing a list, collation state just bubbles up from @@ -699,7 +699,7 @@ assign_collations_walker(Node *node, assign_collations_context *context) */ (void) expression_tree_walker(node, assign_collations_walker, - (void *) &loccontext); + &loccontext); break; } |