Small code simplifications

strVal() can be used in a couple of places instead of coding the same
thing by hand.

diff --git a/src/backend/foreign/foreign.c b/src/backend/foreign/foreign.c
index 61e48ca3f87d79b3f9ae8b26e88622f1a91b2202..3e79c852c1715be5a19a7a5440313f33321308fd 100644 (file)
--- a/src/backend/foreign/foreign.c
+++ b/src/backend/foreign/foreign.c
@@ -544,7 +544,7 @@ deflist_to_tuplestore(ReturnSetInfo *rsinfo, List *options)
        nulls[0] = false;
        if (def->arg)
        {
-           values[1] = CStringGetTextDatum(((Value *) (def->arg))->val.str);
+           values[1] = CStringGetTextDatum(strVal(def->arg));
            nulls[1] = false;
        }
        else

diff --git a/src/backend/parser/gram.y b/src/backend/parser/gram.y
index 61f023604153d4224578bb034cee567bd253d49e..469de52bc29b671ac7f2d103a47f64fb5648b36b 100644 (file)
--- a/src/backend/parser/gram.y
+++ b/src/backend/parser/gram.y
@@ -13890,7 +13890,7 @@ func_expr_common_subexpr:
                     * Convert SystemTypeName() to SystemFuncName() even though
                     * at the moment they result in the same thing.
                     */
-                   $$ = (Node *) makeFuncCall(SystemFuncName(((Value *) llast($5->names))->val.str),
+                   $$ = (Node *) makeFuncCall(SystemFuncName(strVal(llast($5->names))),
                                               list_make1($3),
                                               COERCE_EXPLICIT_CALL,
                                               @1);