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/utils/adt/tsrank.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/utils/adt/tsrank.c')
-rw-r--r-- | src/backend/utils/adt/tsrank.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/backend/utils/adt/tsrank.c b/src/backend/utils/adt/tsrank.c index d456a039510..586a9d6b8d3 100644 --- a/src/backend/utils/adt/tsrank.c +++ b/src/backend/utils/adt/tsrank.c @@ -185,7 +185,7 @@ SortAndUniqItems(TSQuery q, int *size) /* remove duplicates */ while (ptr - res < *size) { - if (compareQueryOperand((void *) ptr, (void *) prevptr, (void *) operand) != 0) + if (compareQueryOperand(ptr, prevptr, operand) != 0) { prevptr++; *prevptr = *ptr; @@ -671,7 +671,7 @@ Cover(DocRepresentation *doc, int len, QueryRepresentation *qr, CoverExt *ext) { fillQueryRepresentationData(qr, ptr); - if (TS_execute(GETQUERY(qr->query), (void *) qr, + if (TS_execute(GETQUERY(qr->query), qr, TS_EXEC_EMPTY, checkcondition_QueryOperand)) { if (WEP_GETPOS(ptr->pos) > ext->q) @@ -701,7 +701,7 @@ Cover(DocRepresentation *doc, int len, QueryRepresentation *qr, CoverExt *ext) */ fillQueryRepresentationData(qr, ptr); - if (TS_execute(GETQUERY(qr->query), (void *) qr, + if (TS_execute(GETQUERY(qr->query), qr, TS_EXEC_EMPTY, checkcondition_QueryOperand)) { if (WEP_GETPOS(ptr->pos) < ext->p) |