Small refactoring
authorPeter Eisentraut
Fri, 23 Mar 2018 21:18:22 +0000 (17:18 -0400)
committerPeter Eisentraut
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: