]> granicus.if.org Git - postgresql/commitdiff
Fix inconsistencies in the code
authorMichael Paquier <michael@paquier.xyz>
Mon, 8 Jul 2019 04:15:09 +0000 (13:15 +0900)
committerMichael Paquier <michael@paquier.xyz>
Mon, 8 Jul 2019 04:15:09 +0000 (13:15 +0900)
This addresses a couple of issues in the code:
- Typos and inconsistencies in comments and function declarations.
- Removal of unreferenced function declarations.
- Removal of unnecessary compile flags.
- A cleanup error in regressplans.sh.

Author: Alexander Lakhin
Discussion: https://postgr.es/m/0c991fdf-2670-1997-c027-772a420c4604@gmail.com

37 files changed:
contrib/cube/cube.c
doc/src/sgml/catalogs.sgml
doc/src/sgml/gist.sgml
src/backend/access/transam/multixact.c
src/backend/catalog/aclchk.c
src/backend/catalog/namespace.c
src/backend/executor/README
src/backend/executor/execScan.c
src/backend/executor/execTuples.c
src/backend/executor/nodeProjectSet.c
src/backend/executor/nodeRecursiveunion.c
src/backend/lib/dshash.c
src/backend/postmaster/autovacuum.c
src/backend/replication/basebackup.c
src/backend/storage/buffer/localbuf.c
src/backend/storage/ipc/procarray.c
src/backend/storage/lmgr/predicate.c
src/backend/storage/smgr/md.c
src/backend/utils/adt/tsrank.c
src/backend/utils/misc/tzparser.c
src/bin/pg_dump/pg_backup_db.c
src/include/access/heapam.h
src/include/access/timeline.h
src/include/executor/executor.h
src/include/executor/tablefunc.h
src/include/nodes/execnodes.h
src/include/partitioning/partprune.h
src/include/port/atomics/generic-acc.h
src/interfaces/ecpg/compatlib/exports.txt
src/interfaces/ecpg/ecpglib/ecpglib_extern.h
src/interfaces/ecpg/ecpglib/execute.c
src/interfaces/ecpg/include/ecpglib.h
src/interfaces/ecpg/preproc/Makefile
src/interfaces/ecpg/preproc/preproc_extern.h
src/pl/plpgsql/src/pl_exec.c
src/test/regress/regressplans.sh
src/tools/msvc/Mkvcbuild.pm

index 2be0055c3fbbe4305bbf6821692bd854797d85ca..4f49c5981ab623a53f1fe6adda4f262669197bfc 100644 (file)
@@ -590,7 +590,7 @@ g_cube_picksplit(PG_FUNCTION_ARGS)
                        v->spl_nright++;
                }
        }
-       *left = *right = FirstOffsetNumber; /* sentinel value, see dosplit() */
+       *left = *right = FirstOffsetNumber; /* sentinel value */
 
        v->spl_ldatum = PointerGetDatum(datum_l);
        v->spl_rdatum = PointerGetDatum(datum_r);
index 09690b6c76bdb388245157cc6548c91e57c61cb9..3428a7c0fa78e588fcf36d03983075ebd450346a 100644 (file)
@@ -11036,7 +11036,7 @@ SELECT * FROM pg_locks pl LEFT JOIN pg_prepared_xacts ppx
       <entry><structfield>kinds</structfield></entry>
       <entry><type>text[]</type></entry>
       <entry></entry>
-      <entry>Types of exdended statistics enabled for this record</entry>
+      <entry>Types of extended statistics enabled for this record</entry>
      </row>
 
      <row>
index 44a3b2c03c5b4bf06f4467c599f529ca55e9496b..763b8cf7fda40b6908f59386094f1dca2463e110 100644 (file)
@@ -910,7 +910,7 @@ Datum
 my_fetch(PG_FUNCTION_ARGS)
 {
     GISTENTRY  *entry = (GISTENTRY *) PG_GETARG_POINTER(0);
-    input_data_type *in = DatumGetP(entry->key);
+    input_data_type *in = DatumGetPointer(entry->key);
     fetched_data_type *fetched_data;
     GISTENTRY  *retval;
 
index fad859a812e54a49eed41859740265a979c50887..2e169dd3f92e1933bf9c8185ebfd1a40ae5da7e3 100644 (file)
@@ -367,8 +367,10 @@ static bool SetOffsetVacuumLimit(bool is_startup);
 static bool find_multixact_start(MultiXactId multi, MultiXactOffset *result);
 static void WriteMZeroPageXlogRec(int pageno, uint8 info);
 static void WriteMTruncateXlogRec(Oid oldestMultiDB,
-                                                                 MultiXactId startOff, MultiXactId endOff,
-                                                                 MultiXactOffset startMemb, MultiXactOffset endMemb);
+                                                                 MultiXactId startTruncOff,
+                                                                 MultiXactId endTruncOff,
+                                                                 MultiXactOffset startTruncMemb,
+                                                                 MultiXactOffset endTruncMemb);
 
 
 /*
@@ -2784,7 +2786,7 @@ ReadMultiXactCounts(uint32 *multixacts, MultiXactOffset *members)
 
 /*
  * Multixact members can be removed once the multixacts that refer to them
- * are older than every datminxmid.  autovacuum_multixact_freeze_max_age and
+ * are older than every datminmxid.  autovacuum_multixact_freeze_max_age and
  * vacuum_multixact_freeze_table_age work together to make sure we never have
  * too many multixacts; we hope that, at least under normal circumstances,
  * this will also be sufficient to keep us from using too many offsets.
index 2797af35c3d48259e737246bb3a439731f7b2473..291412e305bb5c5a50a7d044a76e8ec25a123384 100644 (file)
@@ -1641,7 +1641,7 @@ expand_all_col_privileges(Oid table_oid, Form_pg_class classForm,
 
 /*
  *     This processes attributes, but expects to be called from
- *     ExecGrant_Relation, not directly from ExecGrantStmt.
+ *     ExecGrant_Relation, not directly from ExecuteGrantStmt.
  */
 static void
 ExecGrant_Attribute(InternalGrant *istmt, Oid relOid, const char *relname,
index 97b911679cf83a1bd1d646790dfe560055f71dab..05c6ca81b98a12411b248fd3233f0734c8f14a09 100644 (file)
@@ -504,9 +504,9 @@ RangeVarGetCreationNamespace(const RangeVar *newRelation)
  * permission on the target namespace, this function will instead signal
  * an ERROR.
  *
- * If non-NULL, *existing_oid is set to the OID of any existing relation with
- * the same name which already exists in that namespace, or to InvalidOid if
- * no such relation exists.
+ * If non-NULL, *existing_relation_id is set to the OID of any existing relation
+ * with the same name which already exists in that namespace, or to InvalidOid
+ * if no such relation exists.
  *
  * If lockmode != NoLock, the specified lock mode is acquired on the existing
  * relation, if any, provided that the current user owns the target relation.
index 05f197bc75b692c2571ce9fb971ad19525e9b85c..18b2ac1865954de15d76462df4ca08072264dc9c 100644 (file)
@@ -186,9 +186,9 @@ Expression Evaluation
 
 To allow for different methods of expression evaluation, and for
 better branch/jump target prediction, expressions are evaluated by
-calling ExprState->evalfunc (via ExprEvalExpr() and friends).
+calling ExprState->evalfunc (via ExecEvalExpr() and friends).
 
-ExprReadyExpr() can choose the method of interpretation by setting
+ExecReadyExpr() can choose the method of interpretation by setting
 evalfunc to an appropriate function.  The default execution function,
 ExecInterpExpr, is implemented in execExprInterp.c; see its header
 comment for details.  Special-case evalfuncs are used for certain
index 67c4be5108f21abbd7f884367b699cf54f682b99..c0e4a5376c33741022134281200d86e9d0e64039 100644 (file)
@@ -98,8 +98,7 @@ ExecScanFetch(ScanState *node,
  *             ExecScan
  *
  *             Scans the relation using the 'access method' indicated and
- *             returns the next qualifying tuple in the direction specified
- *             in the global variable ExecDirection.
+ *             returns the next qualifying tuple.
  *             The access method returns the next tuple and ExecScan() is
  *             responsible for checking the tuple returned against the qual-clause.
  *
index a5cb7bba0f84fc142587c56d93fd4625e70baedd..5ee2a464bb448c213e40b2c66acd9067cde69152 100644 (file)
@@ -1148,7 +1148,7 @@ ExecAllocTableSlot(List **tupleTable, TupleDesc desc,
  *             This releases any resources (buffer pins, tupdesc refcounts)
  *             held by the tuple table, and optionally releases the memory
  *             occupied by the tuple table data structure.
- *             It is expected that this routine be called by EndPlan().
+ *             It is expected that this routine be called by ExecEndPlan().
  * --------------------------------
  */
 void
index 277d2783711164236f0c6cb71dcab16cc0285123..facdb4232e67eb328e10e3e5a9fc137a53737146 100644 (file)
@@ -196,8 +196,8 @@ ExecProjectSRF(ProjectSetState *node, bool continuing)
        Assert(hassrf);
 
        /*
-        * If all the SRFs returned EndResult, we consider that as no row being
-        * produced.
+        * If all the SRFs returned ExprEndResult, we consider that as no row
+        * being produced.
         */
        if (hasresult)
        {
index 9c5eed7def3971c537d57be575802438ee192cda..81deb61c21522e877736aa4a5dce5a0b612505db 100644 (file)
@@ -160,7 +160,7 @@ ExecRecursiveUnion(PlanState *pstate)
 }
 
 /* ----------------------------------------------------------------
- *             ExecInitRecursiveUnionScan
+ *             ExecInitRecursiveUnion
  * ----------------------------------------------------------------
  */
 RecursiveUnionState *
@@ -263,7 +263,7 @@ ExecInitRecursiveUnion(RecursiveUnion *node, EState *estate, int eflags)
 }
 
 /* ----------------------------------------------------------------
- *             ExecEndRecursiveUnionScan
+ *             ExecEndRecursiveUnion
  *
  *             frees any storage allocated through C routines.
  * ----------------------------------------------------------------
index 24dd3722521203f8f22c4d36d831e3ad3ea2e1e8..350f8c0a665918ef38abc874a3991ef7875f9587 100644 (file)
@@ -409,7 +409,7 @@ dshash_find(dshash_table *hash_table, const void *key, bool exclusive)
        }
        else
        {
-               /* The caller will free the lock by calling dshash_release. */
+               /* The caller will free the lock by calling dshash_release_lock. */
                hash_table->find_locked = true;
                hash_table->find_exclusively_locked = exclusive;
                return ENTRY_FROM_ITEM(item);
index fd85b9c8f4493797cac83e84a4f62bebd61974a2..073f313337d4964df1273706f039e6ae4ec83897 100644 (file)
@@ -1069,7 +1069,7 @@ rebuild_database_list(Oid newdb)
                current_time = GetCurrentTimestamp();
 
                /*
-                * move the elements from the array into the dllist, setting the
+                * move the elements from the array into the dlist, setting the
                 * next_worker while walking the array
                 */
                for (i = 0; i < nelems; i++)
index 7a1b38466b8eb299eb490a18b6898b3bc9228c98..57f17e14181acf4322d7e48059963a7ed574c2cc 100644 (file)
@@ -1074,7 +1074,7 @@ sendDir(const char *path, int basepathlen, bool sizeonly, List *tablespaces,
                 * error in that case. The error handler further up will call
                 * do_pg_abort_backup() for us. Also check that if the backup was
                 * started while still in recovery, the server wasn't promoted.
-                * dp_pg_stop_backup() will check that too, but it's better to stop
+                * do_pg_stop_backup() will check that too, but it's better to stop
                 * the backup early than continue to the end and fail there.
                 */
                CHECK_FOR_INTERRUPTS();
index 391b6d6e16f4989655ce0e37c8f839f9ef1eab6a..f5f6a29222b7bf5582ee815e9ca916be33bf30c2 100644 (file)
@@ -361,7 +361,7 @@ DropRelFileNodeLocalBuffers(RelFileNode rnode, ForkNumber forkNum,
  *             This function removes from the buffer pool all pages of all forks
  *             of the specified relation.
  *
- *             See DropRelFileNodeAllBuffers in bufmgr.c for more notes.
+ *             See DropRelFileNodesAllBuffers in bufmgr.c for more notes.
  */
 void
 DropRelFileNodeAllLocalBuffers(RelFileNode rnode)
index 18a0f62ba67ca931babc60a3fe9a0db626e58045..ea02973dc73dec524f1b7b1829c1b644f341173e 100644 (file)
@@ -1428,10 +1428,11 @@ GetOldestXmin(Relation rel, int flags)
                result = replication_slot_xmin;
 
        /*
-        * After locks have been released and defer_cleanup_age has been applied,
-        * check whether we need to back up further to make logical decoding
-        * possible. We need to do so if we're computing the global limit (rel =
-        * NULL) or if the passed relation is a catalog relation of some kind.
+        * After locks have been released and vacuum_defer_cleanup_age has been
+        * applied, check whether we need to back up further to make logical
+        * decoding possible. We need to do so if we're computing the global limit
+        * (rel = NULL) or if the passed relation is a catalog relation of some
+        * kind.
         */
        if (!(flags & PROCARRAY_SLOTS_XMIN) &&
                (rel == NULL ||
index 2fedbc4c15ffc0b2d306c2e8a9f0b691f7803091..565c3ac4397592de9c73be864c0bb866f0215e41 100644 (file)
@@ -4360,7 +4360,7 @@ CheckTargetForConflictsIn(PREDICATELOCKTARGETTAG *targettag)
        /*
         * If we found one of our own SIREAD locks to remove, remove it now.
         *
-        * At this point our transaction already has an ExclusiveRowLock on the
+        * At this point our transaction already has a RowExclusiveLock on the
         * relation, so we are OK to drop the predicate lock on the tuple, if
         * found, without fearing that another write against the tuple will occur
         * before the MVCC information makes it to the buffer.
index 64acc3fa433bfa8d019e30394454f2f2f9ef5dde..58c94e9257a506a8aafa806e32012d7b22c4b39d 100644 (file)
@@ -1163,9 +1163,6 @@ _mdfd_getseg(SMgrRelation reln, ForkNumber forknum, BlockNumber blkno,
                         * replaying WAL data that has a write into a high-numbered
                         * segment of a relation that was later deleted. We want to go
                         * ahead and create the segments so we can finish out the replay.
-                        * However if the caller has specified
-                        * EXTENSION_REALLY_RETURN_NULL, then extension is not desired
-                        * even in recovery; we won't reach this point in that case.
                         *
                         * We have to maintain the invariant that segments before the last
                         * active segment are of size RELSEG_SIZE; therefore, if
index 55a96498798adfab2b606a8b7da43254b6dfd54d..e28859d8e1c05476cd233189b1ceddb022553b7f 100644 (file)
@@ -738,7 +738,7 @@ get_docrep(TSVector txt, QueryRepresentation *qr, int *doclen)
        doc = (DocRepresentation *) palloc(sizeof(DocRepresentation) * len);
 
        /*
-        * Iterate through query to make DocRepresentaion for words and it's
+        * Iterate through query to make DocRepresentation for words and it's
         * entries satisfied by query
         */
        for (i = 0; i < qr->query->size; i++)
index cdb9db475d0dcd1aade3220f2ab709aac47303ff..167db84c4deb58b71c8218ca8b23b698755b5ce3 100644 (file)
@@ -53,8 +53,7 @@ validateTzEntry(tzEntry *tzentry)
        unsigned char *p;
 
        /*
-        * Check restrictions imposed by datetkntbl storage format (see
-        * datetime.c)
+        * Check restrictions imposed by datetktbl storage format (see datetime.c)
         */
        if (strlen(tzentry->abbrev) > TOKMAXLEN)
        {
index 9d33c86a3baf16dde13baa1f6eb9684419270216..401e0c8883ee08a4778ffc13097f339f55c46c2b 100644 (file)
@@ -378,7 +378,7 @@ notice_processor(void *arg, const char *message)
        pg_log_generic(PG_LOG_INFO, "%s", message);
 }
 
-/* Like exit_fatal(), but with a complaint about a particular query. */
+/* Like fatal(), but with a complaint about a particular query. */
 static void
 die_on_query_failure(ArchiveHandle *AH, const char *query)
 {
index dffb57bf11a40343b36d73d90801361dbeed5a98..858bcb6bc96ea8bdc89d236b926351dd37d101ab 100644 (file)
@@ -112,7 +112,7 @@ extern TableScanDesc heap_beginscan(Relation relation, Snapshot snapshot,
                                                                        ParallelTableScanDesc parallel_scan,
                                                                        uint32 flags);
 extern void heap_setscanlimits(TableScanDesc scan, BlockNumber startBlk,
-                                                          BlockNumber endBlk);
+                                                          BlockNumber numBlks);
 extern void heapgetpage(TableScanDesc scan, BlockNumber page);
 extern void heap_rescan(TableScanDesc scan, ScanKey key, bool set_params,
                                                bool allow_strat, bool allow_sync, bool allow_pagemode);
index a6dc2edb89390d71b2ac0b5aef96b9ee2441b7b7..e83a73a3f1e893f88fa8db50c4987b7a515f4c33 100644 (file)
@@ -36,7 +36,7 @@ extern void writeTimeLineHistory(TimeLineID newTLI, TimeLineID parentTLI,
                                                                 XLogRecPtr switchpoint, char *reason);
 extern void writeTimeLineHistoryFile(TimeLineID tli, char *content, int size);
 extern void restoreTimeLineHistoryFiles(TimeLineID begin, TimeLineID end);
-extern bool tliInHistory(TimeLineID tli, List *expectedTLIs);
+extern bool tliInHistory(TimeLineID tli, List *expectedTLEs);
 extern TimeLineID tliOfPointInHistory(XLogRecPtr ptr, List *history);
 extern XLogRecPtr tliSwitchPoint(TimeLineID tli, List *history,
                                                                 TimeLineID *nextTLI);
index d056fd615130fcad497b71759728912992b3516b..1fb28b4596be0f655d303c8b238a3afeb216536f 100644 (file)
@@ -314,7 +314,7 @@ ExecEvalExprSwitchContext(ExprState *state,
  * ExecProject
  *
  * Projects a tuple based on projection info and stores it in the slot passed
- * to ExecBuildProjectInfo().
+ * to ExecBuildProjectionInfo().
  *
  * Note: the result is always a virtual tuple; therefore it may reference
  * the contents of the exprContext's scan tuples and/or temporary results
index 926bd570145c7be75e73d5287a3e25a61f78bf46..838c2c2ca2976466711fefb2cb961dd5e9b4c7da 100644 (file)
@@ -20,11 +20,11 @@ struct TableFuncScanState;
  * TableFuncRoutine holds function pointers used for generating content of
  * table-producer functions, such as XMLTABLE.
  *
- * InitBuilder initialize table builder private objects.  The output tuple
+ * InitOpaque initializes table builder private objects.  The output tuple
  * descriptor, input functions for the columns, and typioparams are passed
  * from executor state.
  *
- * SetDoc is called to define the input document.  The table builder may
+ * SetDocument is called to define the input document.  The table builder may
  * apply additional transformations not exposed outside the table builder
  * context.
  *
@@ -45,7 +45,7 @@ struct TableFuncScanState;
  * builder context such that each subsequent GetValue call returns the values
  * for the indicated column for the row being processed.
  *
- * DestroyBuilder shall release all resources associated with a table builder
+ * DestroyOpaque shall release all resources associated with a table builder
  * context.  It may be called either because all rows have been consumed, or
  * because an error occurred while processing the table expression.
  */
index 99b9fa414f138678b74a639cd249169b5dc2d122..98bdcbcef5a5a35b07a2b0b5dbb89ea88227422b 100644 (file)
@@ -593,7 +593,7 @@ typedef struct EState
         * and with which options.  es_jit is created on-demand when JITing is
         * performed.
         *
-        * es_jit_combined_instr is the combined, on demand allocated,
+        * es_jit_worker_instr is the combined, on demand allocated,
         * instrumentation from all workers. The leader's instrumentation is kept
         * separate, and is combined on demand by ExplainPrintJITSummary().
         */
index 81318c785a003cef9a0d7df1f6ecca922e8c62b4..b3e926865aef47ac82802fa4d77dab5f35f9e7a1 100644 (file)
@@ -60,10 +60,10 @@ typedef struct PartitionPruneContext
 } PartitionPruneContext;
 
 /*
- * PruneCxtStateIdx() computes the correct index into the stepcmpfuncs[],
- * exprstates[] and exprhasexecparam[] arrays for step step_id and
- * partition key column keyno.  (Note: there is code that assumes the
- * entries for a given step are sequential, so this is not chosen freely.)
+ * PruneCxtStateIdx() computes the correct index into the stepcmpfuncs[]
+ * and exprstates[] arrays for step step_id and partition key column keyno.
+ * (Note: there is code that assumes the entries for a given step are
+ * sequential, so this is not chosen freely.)
  */
 #define PruneCxtStateIdx(partnatts, step_id, keyno) \
        ((partnatts) * (step_id) + (keyno))
index eec5063cbc4e77d4f5b3071c36f9ebd17ec3e128..a64c841f4f17d9b7992da836f9c49dc703ecefb7 100644 (file)
@@ -68,7 +68,7 @@ pg_atomic_compare_exchange_u32_impl(volatile pg_atomic_uint32 *ptr,
        _Asm_mf();
        /*
         * Notes:
-        * DOWN_MEM_FENCE | _UP_MEM_FENCE prevents reordering by the compiler
+        * _DOWN_MEM_FENCE | _UP_MEM_FENCE prevents reordering by the compiler
         */
        current =  _Asm_cmpxchg(_SZ_W, /* word */
                                                        _SEM_REL,
index e0cfd7a22d066ca70261ae0fcf588311ec9fb776..86e9ca164047ae6504ccf85c985e0ca9e7ab3d5c 100644 (file)
@@ -1,5 +1,5 @@
 # src/interfaces/ecpg/compatlib/exports.txt
-# Functions to be exported by ecpg_compatlib DLL
+# Functions to be exported by libecpg_compat DLL
 ECPG_informix_get_var   1
 ECPG_informix_set_var   2
 decadd                  3
index 6cb7ab1a192dc60402bb95b19078cd3e8369d8f6..de8241bfc88466befafd4b8f8f43e96363700e52 100644 (file)
@@ -195,11 +195,8 @@ char          *ecpg_strdup(const char *, int);
 const char *ecpg_type_name(enum ECPGttype);
 int                    ecpg_dynamic_type(Oid);
 int                    sqlda_dynamic_type(Oid, enum COMPAT_MODE);
-void           ecpg_free_auto_mem(void);
 void           ecpg_clear_auto_mem(void);
 
-struct descriptor *ecpggetdescp(int, char *);
-
 struct descriptor *ecpg_find_desc(int line, const char *name);
 
 struct prepared_statement *ecpg_find_prepared_statement(const char *,
index 3c0294e98aa8f72d148e33bb5ca724000fe3c8f0..4394b420a15e84b04bbadccd4327cdcd0c96b0cd 100644 (file)
@@ -1899,7 +1899,7 @@ ecpg_process_output(struct statement *stmt, bool clear_result)
 
                        /*
                         * execution should never reach this code because it is already
-                        * handled in ECPGcheck_PQresult()
+                        * handled in ecpg_check_PQresult()
                         */
                        ecpg_log("ecpg_process_output on line %d: unknown execution status type\n",
                                         stmt->lineno);
index f1031d2595601dd0d64abda8aba3be33ff83429a..e12d703b4e98141d6770d006dcc6a4bf986791ba 100644 (file)
@@ -63,8 +63,6 @@ char     *ECPGprepared_statement(const char *, const char *, int);
 PGconn    *ECPGget_PGconn(const char *);
 PGTransactionStatusType ECPGtransactionStatus(const char *);
 
-char      *ECPGerrmsg(void);
-
  /* print an error message */
 void           sqlprint(void);
 
index 20e3b4787468a49a9f5e2d76def2cf5b188a6cb9..2ec1023626341f109c5b5b14440d1ab35a1d6048 100644 (file)
@@ -19,7 +19,6 @@ override CPPFLAGS := -I../include -I$(top_srcdir)/src/interfaces/ecpg/include \
        -I. -I$(srcdir) \
        -I$(top_srcdir)/src/interfaces/ecpg/ecpglib \
        -I$(libpq_srcdir) \
-       -DECPG_COMPILE \
        $(CPPFLAGS)
 
 override CFLAGS += $(PTHREAD_CFLAGS)
index 552ffc627bc354dc0511fd2c2b6f6b272e66bacb..7de5cd50878de5630c5a364d1cf91a720f3d33e1 100644 (file)
@@ -30,7 +30,6 @@ extern int    braces_open,
                        struct_level,
                        ecpg_internal_var;
 extern char *current_function;
-extern char *descriptor_index;
 extern char *descriptor_name;
 extern char *connection;
 extern char *input_filename;
index 08961e2af9cc4d10a19938dcb9783565a493036b..a718bccc24a2acbe9efecbc280fa72932e9b5212 100644 (file)
@@ -3294,10 +3294,10 @@ exec_stmt_return_next(PLpgSQL_execstate *estate,
         * reference; in particular, this path is always taken in functions with
         * one or more OUT parameters.
         *
-        * Unlike exec_statement_return, there's no special win here for R/W
-        * expanded values, since they'll have to get flattened to go into the
-        * tuplestore.  Indeed, we'd better make them R/O to avoid any risk of the
-        * casting step changing them in-place.
+        * Unlike exec_stmt_return, there's no special win here for R/W expanded
+        * values, since they'll have to get flattened to go into the tuplestore.
+        * Indeed, we'd better make them R/O to avoid any risk of the casting step
+        * changing them in-place.
         */
        if (stmt->retvarno >= 0)
        {
index 678ab0a3f227d9270ee966fa07d6cc6b56e3a484..31e7876daab56f821bfdb6664482db08d3e47720 100755 (executable)
@@ -72,7 +72,7 @@ mv -f regression.out planregress/out.in
 mv -f regression.diffs planregress/diffs.in
 PGOPTIONS="$PGOPTIONS      -fi -fn     -fh" $MAKE runtest
 mv -f regression.out planregress/out.inh
-mv -f regression.diffsregression.planregress/inh
+mv -f regression.diffs planregress/diffs.inh
 PGOPTIONS="$PGOPTIONS      -fi -fn -fm    " $MAKE runtest
 mv -f regression.out planregress/out.inm
 mv -f regression.diffs planregress/diffs.inm
index 6f928234eec072e0a7510657048b32bc2275d5c0..d1d0aed07ed01e8e246af7f725e853a292569c80 100644 (file)
@@ -309,7 +309,6 @@ sub mkvcbuild
        $ecpg->AddIncludeDir('src/interfaces/libpq');
        $ecpg->AddPrefixInclude('src/interfaces/ecpg/preproc');
        $ecpg->AddFiles('src/interfaces/ecpg/preproc', 'pgc.l', 'preproc.y');
-       $ecpg->AddDefine('ECPG_COMPILE');
        $ecpg->AddReference($libpgcommon, $libpgport);
 
        my $pgregress_ecpg =