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/optimizer/util/tlist.c | |
| parent | 97525bc5c8ffb31475d23955d08e9ec9c1408f33 (diff) | |
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/optimizer/util/tlist.c')
| -rw-r--r-- | src/backend/optimizer/util/tlist.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/backend/optimizer/util/tlist.c b/src/backend/optimizer/util/tlist.c index 7ef7f34d8b5..482350285a0 100644 --- a/src/backend/optimizer/util/tlist.c +++ b/src/backend/optimizer/util/tlist.c @@ -1139,8 +1139,7 @@ split_pathtarget_walker(Node *node, split_pathtarget_context *context) context->current_depth = 0; context->current_sgref = 0; /* subexpressions are not sortgroup items */ - (void) expression_tree_walker(node, split_pathtarget_walker, - (void *) context); + (void) expression_tree_walker(node, split_pathtarget_walker, context); /* Depth is one more than any SRF below it */ srf_depth = context->current_depth + 1; @@ -1181,8 +1180,7 @@ split_pathtarget_walker(Node *node, split_pathtarget_context *context) * examine its inputs. */ context->current_sgref = 0; /* subexpressions are not sortgroup items */ - return expression_tree_walker(node, split_pathtarget_walker, - (void *) context); + return expression_tree_walker(node, split_pathtarget_walker, context); } /* |