aboutsummaryrefslogtreecommitdiff
path: root/src/backend/commands/statscmds.c
diff options
context:
space:
mode:
authorPeter Eisentraut <peter@eisentraut.org>2021-09-09 07:58:12 +0200
committerPeter Eisentraut <peter@eisentraut.org>2021-09-09 08:36:52 +0200
commitcbdf75bf8053f88bbae6b307f34ab057424a370f (patch)
treeeeaa5841506bbf6dfc086b281dfded59721f2e07 /src/backend/commands/statscmds.c
parent3b231596ccfc6bb6564c63a574e429765c0f775e (diff)
downloadpostgresql-cbdf75bf8053f88bbae6b307f34ab057424a370f.tar.gz
postgresql-cbdf75bf8053f88bbae6b307f34ab057424a370f.zip
Remove useless casts
Casting the argument of strVal() to (Value *) is useless, since strVal() already does that. Most code didn't do that anyway; this was apparently just a style that snuck into certain files. Reviewed-by: Dagfinn Ilmari Mannsåker <ilmari@ilmari.org> Reviewed-by: Kyotaro Horiguchi <horikyota.ntt@gmail.com> Discussion: https://www.postgresql.org/message-id/flat/5ba6bc5b-3f95-04f2-2419-f8ddb4c046fb@enterprisedb.com
Diffstat (limited to 'src/backend/commands/statscmds.c')
-rw-r--r--src/backend/commands/statscmds.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/commands/statscmds.c b/src/backend/commands/statscmds.c
index 59369f87362..78917844dee 100644
--- a/src/backend/commands/statscmds.c
+++ b/src/backend/commands/statscmds.c
@@ -335,7 +335,7 @@ CreateStatistics(CreateStatsStmt *stmt)
build_mcv = false;
foreach(cell, stmt->stat_types)
{
- char *type = strVal((Value *) lfirst(cell));
+ char *type = strVal(lfirst(cell));
if (strcmp(type, "ndistinct") == 0)
{