diff options
| author | Peter Eisentraut <peter@eisentraut.org> | 2021-09-09 07:58:12 +0200 |
|---|---|---|
| committer | Peter Eisentraut <peter@eisentraut.org> | 2021-09-09 08:36:52 +0200 |
| commit | cbdf75bf8053f88bbae6b307f34ab057424a370f (patch) | |
| tree | eeaa5841506bbf6dfc086b281dfded59721f2e07 /src/backend/commands/dropcmds.c | |
| parent | 3b231596ccfc6bb6564c63a574e429765c0f775e (diff) | |
| download | postgresql-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/dropcmds.c')
| -rw-r--r-- | src/backend/commands/dropcmds.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/src/backend/commands/dropcmds.c b/src/backend/commands/dropcmds.c index 97e5e9a765c..4e545adf95a 100644 --- a/src/backend/commands/dropcmds.c +++ b/src/backend/commands/dropcmds.c @@ -255,7 +255,7 @@ does_not_exist_skipping(ObjectType objtype, Node *object) { case OBJECT_ACCESS_METHOD: msg = gettext_noop("access method \"%s\" does not exist, skipping"); - name = strVal((Value *) object); + name = strVal(object); break; case OBJECT_TYPE: case OBJECT_DOMAIN: @@ -285,7 +285,7 @@ does_not_exist_skipping(ObjectType objtype, Node *object) break; case OBJECT_SCHEMA: msg = gettext_noop("schema \"%s\" does not exist, skipping"); - name = strVal((Value *) object); + name = strVal(object); break; case OBJECT_STATISTIC_EXT: if (!schema_does_not_exist_skipping(castNode(List, object), &msg, &name)) @@ -324,7 +324,7 @@ does_not_exist_skipping(ObjectType objtype, Node *object) break; case OBJECT_EXTENSION: msg = gettext_noop("extension \"%s\" does not exist, skipping"); - name = strVal((Value *) object); + name = strVal(object); break; case OBJECT_FUNCTION: { @@ -392,7 +392,7 @@ does_not_exist_skipping(ObjectType objtype, Node *object) } case OBJECT_LANGUAGE: msg = gettext_noop("language \"%s\" does not exist, skipping"); - name = strVal((Value *) object); + name = strVal(object); break; case OBJECT_CAST: { @@ -434,7 +434,7 @@ does_not_exist_skipping(ObjectType objtype, Node *object) break; case OBJECT_EVENT_TRIGGER: msg = gettext_noop("event trigger \"%s\" does not exist, skipping"); - name = strVal((Value *) object); + name = strVal(object); break; case OBJECT_RULE: if (!owningrel_does_not_exist_skipping(castNode(List, object), &msg, &name)) @@ -447,11 +447,11 @@ does_not_exist_skipping(ObjectType objtype, Node *object) break; case OBJECT_FDW: msg = gettext_noop("foreign-data wrapper \"%s\" does not exist, skipping"); - name = strVal((Value *) object); + name = strVal(object); break; case OBJECT_FOREIGN_SERVER: msg = gettext_noop("server \"%s\" does not exist, skipping"); - name = strVal((Value *) object); + name = strVal(object); break; case OBJECT_OPCLASS: { @@ -479,7 +479,7 @@ does_not_exist_skipping(ObjectType objtype, Node *object) break; case OBJECT_PUBLICATION: msg = gettext_noop("publication \"%s\" does not exist, skipping"); - name = strVal((Value *) object); + name = strVal(object); break; default: elog(ERROR, "unrecognized object type: %d", (int) objtype); |