Small refactoring
authorPeter Eisentraut <peter_e@gmx.net>
Fri, 23 Mar 2018 21:18:22 +0000 (17:18 -0400)
committerPeter Eisentraut <peter_e@gmx.net>
Fri, 23 Mar 2018 21:18:22 +0000 (17:18 -0400)
Put the "atomic" argument of ExecuteDoStmt() and ExecuteCallStmt() into
a variable instead of repeating the formula.

src/backend/tcop/utility.c

index 8481fcca367f34f19e6cf6e74249e4148950516c..e144583bd1f1285420ba864f4ae82823ddc7282a 100644 (file)
@@ -382,6 +382,7 @@ standard_ProcessUtility(PlannedStmt *pstmt,
 {
        Node       *parsetree = pstmt->utilityStmt;
        bool            isTopLevel = (context == PROCESS_UTILITY_TOPLEVEL);
+       bool            isAtomicContext = (context != PROCESS_UTILITY_TOPLEVEL || IsTransactionBlock());
        ParseState *pstate;
 
        check_xact_readonly(parsetree);
@@ -514,8 +515,7 @@ standard_ProcessUtility(PlannedStmt *pstmt,
                        break;
 
                case T_DoStmt:
-                       ExecuteDoStmt((DoStmt *) parsetree,
-                                                 (context != PROCESS_UTILITY_TOPLEVEL || IsTransactionBlock()));
+                       ExecuteDoStmt((DoStmt *) parsetree, isAtomicContext);
                        break;
 
                case T_CreateTableSpaceStmt:
@@ -644,9 +644,7 @@ standard_ProcessUtility(PlannedStmt *pstmt,
                        break;
 
                case T_CallStmt:
-                       ExecuteCallStmt(castNode(CallStmt, parsetree), params,
-                                                       (context != PROCESS_UTILITY_TOPLEVEL || IsTransactionBlock()),
-                                                       dest);
+                       ExecuteCallStmt(castNode(CallStmt, parsetree), params, isAtomicContext, dest);
                        break;
 
                case T_ClusterStmt: