Small stylistic improvement in recent FORCE QUOTE * code - use a bool instead of...

diff --git a/src/backend/commands/copy.c b/src/backend/commands/copy.c
index 9ee8ea2c8254e079e74f2eb3dc5a82b2b956a7f8..f74eb3593789ce788a721b3ee5947da303d16f85 100644 (file)
--- a/src/backend/commands/copy.c
+++ b/src/backend/commands/copy.c
@@ -729,9 +729,7 @@ DoCopy(const CopyStmt *stmt, const char *queryString)
        TupleDesc       tupDesc;
        int                     num_phys_attrs;
        uint64          processed;
-
-       /* a dummy list that represents 'all-columns' */
-       List            all_columns = { T_List };
+       bool            force_quote_all = false;
        
        /* Allocate workspace and zero all fields */
        cstate = (CopyStateData *) palloc0(sizeof(CopyStateData));
@@ -813,7 +811,7 @@ DoCopy(const CopyStmt *stmt, const char *queryString)
                                                 errmsg("conflicting or redundant options")));
 
                        if (IsA(defel->arg, A_Star))
-                               force_quote = &all_columns;
+                               force_quote_all = true;
                        else
                                force_quote = (List *) defel->arg;
                }
@@ -1099,7 +1097,7 @@ DoCopy(const CopyStmt *stmt, const char *queryString)
 
        /* Convert FORCE QUOTE name list to per-column flags, check validity */
        cstate->force_quote_flags = (bool *) palloc0(num_phys_attrs * sizeof(bool));
-       if (force_quote == &all_columns)
+       if (force_quote_all)
        {
                int             i;