diff options
| author | Peter Eisentraut | 2024-11-28 07:19:22 +0000 |
|---|---|---|
| committer | Peter Eisentraut | 2024-11-28 07:27:20 +0000 |
| commit | 7f798aca1d5df290aafad41180baea0ae311b4ee (patch) | |
| tree | b4c8132ec85c21f6c72308b5857defd70958de69 /src/backend/commands/variable.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/commands/variable.c')
| -rw-r--r-- | src/backend/commands/variable.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/backend/commands/variable.c b/src/backend/commands/variable.c index 99462681925..0ecff94d0ed 100644 --- a/src/backend/commands/variable.c +++ b/src/backend/commands/variable.c @@ -232,7 +232,7 @@ check_datestyle(char **newval, void **extra, GucSource source) return false; myextra[0] = newDateStyle; myextra[1] = newDateOrder; - *extra = (void *) myextra; + *extra = myextra; return true; } @@ -901,7 +901,7 @@ check_session_authorization(char **newval, void **extra, GucSource source) return false; myextra->roleid = roleid; myextra->is_superuser = is_superuser; - *extra = (void *) myextra; + *extra = myextra; return true; } @@ -1015,7 +1015,7 @@ check_role(char **newval, void **extra, GucSource source) return false; myextra->roleid = roleid; myextra->is_superuser = is_superuser; - *extra = (void *) myextra; + *extra = myextra; return true; } |