From: Alexander Lakhin <exclusion@gmail.com>
To: pgsql-hackers <pgsql-hackers@postgresql.org>
To: Michael Paquier <michael@paquier.xyz>
Subject: Fix typos and inconsistencies for HEAD (take 9)
Date: Mon, 5 Aug 2019 00:33:34 +0300
Message-ID: <7ab243e0-116d-3e44-d120-76b3df7abefd@gmail.com> (raw)
Hello hackers,
Please consider fixing the next truss of typos and inconsistencies in
the tree:
9.1. NAMESPACE_SQLXML -> remove (not used since the introduction in
355e05ab)
9.2. NBXLOG_H -> NBTXLOG_H
9.3. NEWPAGE -> XLOG_FPI (orphaned since 54685338)
9.4. newXlogId, newXlogSeg -> newXlogSegNo (orphaned since dfda6eba)
9.5. nextblno -> nextblkno
9.6. noCatalogs -> arg
9.7. nodeWindowFunc.c -> nodeWindowAgg.c
9.8. NOINHERITS -> NOINHERIT
9.9. NO_RESPONSE -> PQPING_NO_RESPONSE
9.10. Normalizationdata.txt- > NormalizationTest.txt
9.11. NOT_AVAIL -> LOCKACQUIRE_NOT_AVAIL
9.12. not_point -> is_point (a contradiction with the check and a
comment, appeared in 4c1383ef)
9.13. nptrs, ptrs -> nipd, ipd
9.14. nbuffers, nrdatas -> max_block_id, ndatas
9.15. nrow -> nrows
9.16. nsubxcnt -> subxcnt
9.17. ntzones, zp -> zonecount, zpfirst
9.18. nvmblocks -> vm_nblocks
9.19. objAddress -> objAddr
9.20. ObjectWithArg -> ObjectWithArgs
9.21. objOid, relOid -> objectId, classId, subId
9.22. ObjType -> ObjectType
9.23. OffsetNumberMask -> remove (not used since PG95-1_01)
9.24. oid_in_function -> remove (orphaned since 578b2297)
9.25. oidzero, oidge -> remove (not used since the introduction in 8dc42a3a)
9.26. oidle -> remove (orphaned since 9f0ae0c8)
9.27. oideq -> remove (orphaned since 005a1217)
9.28. OldestMemberMXactID -> OldestMemberMXactId
9.29. oldestXidDb -> oldestXactDb
9.30. oldest-Xmin -> oldestXmin
9.31. old_key_tup -> old_key_tuple
9.32. on_dsm_callback -> on_dsm_detach callback
9.33. ONE_PAGE -> TBM_ONE_PAGE
9.34. opt_boolean -> opt_boolean_or_string (renamed in 5c84fe46)
9.35. OptimizableStmt -> PreparableStmt (orphaned since aa83bc04)
9.36. optType -> remove (not used since the introduction in 500b62b0)
9.37. organizationUnitName -> organizationalUnitName (see
https://github.com/openssl/openssl/issues/1843)
9.38. origTupDesc -> origTupdesc
9.39. outputvariables -> output variables
9.40. ovflpgs -> ovflpages
9.41. OWNER_TO -> OWNER TO
9.42. PackedPostingList -> GinPostingList (an inconsistency since 36a35c55)
9.43. PageClearPrunable -> remove (not used since 6f10eb21)
9.44. pageLSN -> curPageLSN
9.45. paramListInfo -> ParamListInfo
9.46. PARSER_FUNC_H -> PARSE_FUNC_H
9.47. parse_hba -> parse_hba_line (renamed in 98723810)
9.48. parse_json -> pg_parse_json
9.49. pathkeys_contain_in -> pathkeys_contained_in
9.50. pattern1 -> pattern
9.51. patters -> patterns
9.52. PerAggData -> WindowStatePerAggData
9.53. PerFuncData -> WindowStatePerFuncData
9.54. pgaddtup -> this function
9.55. pg_atomic_test_and_set_flag -> pg_atomic_test_set_flag
9.56. pg_binary_read_file -> pg_read_binary_file
9.57. pgcommonfiles -> remove (orphaned since a7301839)
9.58. pgcontrolvalue, xlrecvalue -> oldvalue, newvalue
9.59. pg_dlsym -> dlsym (orphaned since 842cb9fa)
9.60. pg_encconv_tbl -> pg_enc2name_tbl (orphaned since eb335a03)
9.61. PGgetline -> PQgetline
9.62. pghackers -> pgsql-hackers
9.63. _pg_keysequal -> remove (not used since d26e1eba)
9.64. pg_language_metadata -> pg_largeobject_metadata
9.65. pgp_armor -> pg_armor
9.66. PGresTuple -> PGresult tuple
9.67. pgstat_recv_resetshared -> pgstat_recv_resetsharedcounter
9.68. pgtypes_date_months_short -> months (abbreviations)
9.69. PgXact -> MyPgXact
9.70. PlaceholderVars -> PlaceHolderVars
9.71. placeToPage -> beginPlaceToPage
9.72. PLpgSQL_dynfors -> PLpgSQL_stmt_dynfors
9.73. plpgsql_init -> _PG_init (orphaned since b09bfcaa)
9.74. PLy_munge_source -> PLy_procedure_munge_source
9.75. Point-in-Time -> Point-In-Time
9.76. PO_FILES, ALL_PO_FILES -> remove (orphaned since 4e1c7207, 5dd41f35)
9.77. pop_size -> pool_size
9.78. posid -> ip_posid, blkid -> ip_blkid
9.79. pq_close, pq_comm_reset -> socket_close, socket_comm_reset
(renamed in 2bd9e412)
9.80. pqcomprim.c -> pqformat.c (orphaned since 95cc41b)
9.81. PQescapeByte()a -> PQescapeBytea()
9.82. PQSetenv -> pqSetenv
9.83. PreCommit_CheckForSerializableConflicts ->
PreCommit_CheckForSerializationFailure
9.84. PREFER -> remove (not used since the introduction in 7bcc6d98)
9.85. prepared_transactions -> prepared-transactions
9.86. pressurel -> pressure
9.87. prettyprint -> pretty-print
9.88. PRETTY_SCHEMA -> remove (not used since the introduction in 3d2aed66)
9.89. printAttName, terseOutput, width -> PrintAttNames, TerseOutput,
colWidth
9.90. priorWALFileName -> remove (not used since the introduction in
51be14e9)
9.91. PROC_H -> _PROC_H_
9.92. ProcLWLockTranche -> remove (orphaned since 3761fe3c and
non-informational)
9.93. PUTBYTE -> remove (not used since the introduction in 0c0dde61)
9.94. px_hmac_block_size, px_hmac_reset -> remove (not used since the
introduction in df24cb73)
9.95. PX_MAX_NAMELEN -> remove (not used since 3cc86612)
Also, please fix my typo (from the previous set), that was somehow
silted through a double check.
Best regards,
Alexander
Attachments:download all (11 files, .tar.gz)fix_for_fix_8.15.patch (text/x-patch, 633B) downloaddiff --git a/src/backend/libpq/auth.c b/src/backend/libpq/auth.cindex 5fb1f1b7d7..3713dd2884 100644--- a/src/backend/libpq/auth.c+++ b/src/backend/libpq/auth.c@@ -1797,7 +1797,7 @@ interpret_ident_response(const char *ident_response,
/*
* Talk to the ident server on "remote_addr" and find out who
* owns the tcp connection to "local_addr"
- * It the username successfully retrieved, check the usermap.+ * If the username successfully retrieved, check the usermap.
*
* XXX: Using WaitLatchOrSocket() and doing a CHECK_FOR_INTERRUPTS() if the
* latch was set would improve the responsiveness to timeouts/cancellations.
fixes_9.1-10.patch (text/x-patch, 5.9K) downloaddiff --git a/doc/src/sgml/ref/set_role.sgml b/doc/src/sgml/ref/set_role.sgmlindex 0ef6eb9a9c..9ab0d6af04 100644--- a/doc/src/sgml/ref/set_role.sgml+++ b/doc/src/sgml/ref/set_role.sgml@@ -70,7 +70,7 @@ RESET ROLE
effectively drops all the privileges assigned directly to the session user
and to the other roles it is a member of, leaving only the privileges
available to the named role. On the other hand, if the session user role
- has the <literal>NOINHERITS</literal> attribute, <command>SET ROLE</command> drops the+ has the <literal>NOINHERIT</literal> attribute, <command>SET ROLE</command> drops the
privileges assigned directly to the session user and instead acquires the
privileges available to the named role.
</para>
diff --git a/src/backend/access/hash/hash_xlog.c b/src/backend/access/hash/hash_xlog.cindex d7b7098110..86e7210acb 100644--- a/src/backend/access/hash/hash_xlog.c+++ b/src/backend/access/hash/hash_xlog.c@@ -706,7 +706,7 @@ hash_xlog_squeeze_page(XLogReaderState *record)
/*
* if the page on which are adding tuples is a page previous to freed
- * overflow page, then update its nextblno.+ * overflow page, then update its nextblkno.
*/
if (xldata->is_prev_bucket_same_wrt)
{
diff --git a/src/backend/access/nbtree/nbtsort.c b/src/backend/access/nbtree/nbtsort.cindex d0b9013caf..b30cf9e989 100644--- a/src/backend/access/nbtree/nbtsort.c+++ b/src/backend/access/nbtree/nbtsort.c@@ -657,7 +657,7 @@ _bt_blwritepage(BTWriteState *wstate, Page page, BlockNumber blkno)
/* XLOG stuff */
if (wstate->btws_use_wal)
{
- /* We use the heap NEWPAGE record type for this */+ /* We use the XLOG_FPI record type for this */
log_newpage(&wstate->index->rd_node, MAIN_FORKNUM, blkno, page, true);
}
diff --git a/src/backend/utils/adt/xml.c b/src/backend/utils/adt/xml.cindex 5e629d29ea..f5b477be2b 100644--- a/src/backend/utils/adt/xml.c+++ b/src/backend/utils/adt/xml.c@@ -227,7 +227,6 @@ const TableFuncRoutine XmlTableRoutine =
/* from SQL/XML:2008 section 4.9 */
#define NAMESPACE_XSD "http://www.w3.org/2001/XMLSchema"
#define NAMESPACE_XSI "http://www.w3.org/2001/XMLSchema-instance"
-#define NAMESPACE_SQLXML "http://standards.iso.org/iso/9075/2003/sqlxml"
#ifdef USE_LIBXML
diff --git a/src/bin/pg_resetwal/pg_resetwal.c b/src/bin/pg_resetwal/pg_resetwal.cindex 9d9c33d78c..74063ebf2f 100644--- a/src/bin/pg_resetwal/pg_resetwal.c+++ b/src/bin/pg_resetwal/pg_resetwal.c@@ -934,8 +934,8 @@ RewriteControlFile(void)
*
* On entry, ControlFile.checkPointCopy.redo and ControlFile.xlog_seg_size
* are assumed valid (note that we allow the old xlog seg size to differ
- * from what we're using). On exit, newXlogId and newXlogSeg are set to- * suitable values for the beginning of replacement WAL (in our seg size).+ * from what we're using). On exit, newXlogSegNo is set to suitable+ * value for the beginning of replacement WAL (in our seg size).
*/
static void
FindEndOfXLOG(void)
diff --git a/src/common/unicode/norm_test.c b/src/common/unicode/norm_test.cindex 421dd4e68e..fee58a184a 100644--- a/src/common/unicode/norm_test.c+++ b/src/common/unicode/norm_test.c@@ -66,7 +66,7 @@ main(int argc, char **argv)
if (pg_wcscmp(test->output, result) != 0)
{
- printf("FAILURE (Normalizationdata.txt line %d):\n", test->linenum);+ printf("FAILURE (NormalizationTest.txt line %d):\n", test->linenum);
printf("input:\t%s\n", print_wchar_str(test->input));
printf("expected:\t%s\n", print_wchar_str(test->output));
printf("got\t%s\n", print_wchar_str(result));
diff --git a/src/include/access/nbtxlog.h b/src/include/access/nbtxlog.hindex 9beccc86ea..afa614da25 100644--- a/src/include/access/nbtxlog.h+++ b/src/include/access/nbtxlog.h@@ -259,4 +259,4 @@ extern void btree_desc(StringInfo buf, XLogReaderState *record);
extern const char *btree_identify(uint8 info);
extern void btree_mask(char *pagedata, BlockNumber blkno);
-#endif /* NBXLOG_H */+#endif /* NBTXLOG_H */diff --git a/src/include/commands/tablecmds.h b/src/include/commands/tablecmds.hindex ac2bfaff1e..9c25a805f2 100644--- a/src/include/commands/tablecmds.h+++ b/src/include/commands/tablecmds.h@@ -87,7 +87,7 @@ extern void RangeVarCallbackOwnsTable(const RangeVar *relation,
Oid relId, Oid oldRelId, void *arg);
extern void RangeVarCallbackOwnsRelation(const RangeVar *relation,
- Oid relId, Oid oldRelId, void *noCatalogs);+ Oid relId, Oid oldRelId, void *arg);
extern bool PartConstraintImpliedByRelConstraint(Relation scanrel,
List *partConstraint);
diff --git a/src/include/executor/execExpr.h b/src/include/executor/execExpr.hindex 8e7f7c3d13..d21dbead0a 100644--- a/src/include/executor/execExpr.h+++ b/src/include/executor/execExpr.h@@ -576,7 +576,7 @@ typedef struct ExprEvalStep
/* for EEOP_WINDOW_FUNC */
struct
{
- /* out-of-line state, modified by nodeWindowFunc.c */+ /* out-of-line state, modified by nodeWindowAgg.c */
WindowFuncExprState *wfstate;
} window_func;
diff --git a/src/interfaces/libpq/fe-connect.c b/src/interfaces/libpq/fe-connect.cindex a329ebbf93..d262b57021 100644--- a/src/interfaces/libpq/fe-connect.c+++ b/src/interfaces/libpq/fe-connect.c@@ -3732,8 +3732,8 @@ internal_ping(PGconn *conn)
* PQPING_NO_RESPONSE. This result could be somewhat misleading for a
* pre-7.4 server, since it won't send back a SQLSTATE, but those are long
* out of support. Another corner case where the server could return a
- * failure without a SQLSTATE is fork failure, but NO_RESPONSE isn't- * totally unreasonable for that anyway. We expect that every other+ * failure without a SQLSTATE is fork failure, but PQPING_NO_RESPONSE+ * isn't totally unreasonable for that anyway. We expect that every other
* failure case in a modern server will produce a report with a SQLSTATE.
*
* NOTE: whenever we get around to making libpq generate SQLSTATEs for
fixes_9.11-20.patch (text/x-patch, 6.5K) downloaddiff --git a/contrib/earthdistance/earthdistance--1.1.sql b/contrib/earthdistance/earthdistance--1.1.sqlindex 9136a54a7b..543338aff3 100644--- a/contrib/earthdistance/earthdistance--1.1.sql+++ b/contrib/earthdistance/earthdistance--1.1.sql@@ -28,7 +28,7 @@ AS 'SELECT ''6378168''::float8';
-- centered about the origin with the radius of the earth.
CREATE DOMAIN earth AS cube
- CONSTRAINT not_point check(cube_is_point(value))+ CONSTRAINT is_point check(cube_is_point(value))
CONSTRAINT not_3d check(cube_dim(value) <= 3)
CONSTRAINT on_surface check(abs(cube_distance(value, '(0)'::cube) /
earth() - 1) < '10e-7'::float8);
diff --git a/contrib/postgres_fdw/postgres_fdw.c b/contrib/postgres_fdw/postgres_fdw.cindex 033aeb2556..06a205877d 100644--- a/contrib/postgres_fdw/postgres_fdw.c+++ b/contrib/postgres_fdw/postgres_fdw.c@@ -2794,7 +2794,7 @@ estimate_path_cost_size(PlannerInfo *root,
/*
* Back into an estimate of the number of retrieved rows. Just in
- * case this is nuts, clamp to at most nrow.+ * case this is nuts, clamp to at most nrows.
*/
retrieved_rows = clamp_row_est(rows / fpinfo->local_conds_sel);
retrieved_rows = Min(retrieved_rows, nrows);
diff --git a/src/backend/access/heap/visibilitymap.c b/src/backend/access/heap/visibilitymap.cindex 64dfe06b26..a08922b079 100644--- a/src/backend/access/heap/visibilitymap.c+++ b/src/backend/access/heap/visibilitymap.c@@ -124,7 +124,7 @@
/* prototypes for internal routines */
static Buffer vm_readbuf(Relation rel, BlockNumber blkno, bool extend);
-static void vm_extend(Relation rel, BlockNumber nvmblocks);+static void vm_extend(Relation rel, BlockNumber vm_nblocks);
/*
diff --git a/src/backend/access/transam/README b/src/backend/access/transam/READMEindex ad4083eb6b..b5a2cb2de8 100644--- a/src/backend/access/transam/README+++ b/src/backend/access/transam/README@@ -530,7 +530,7 @@ void XLogResetInsertion(void)
construction workspace. This is only needed if you have already called
XLogBeginInsert(), but decide to not insert the record after all.
-void XLogEnsureRecordSpace(int max_block_id, int nrdatas)+void XLogEnsureRecordSpace(int max_block_id, int ndatas)
Normally, the WAL record construction buffers have the following limits:
diff --git a/src/backend/parser/parse_oper.c b/src/backend/parser/parse_oper.cindex 0e1015962d..20abbb3b55 100644--- a/src/backend/parser/parse_oper.c+++ b/src/backend/parser/parse_oper.c@@ -134,7 +134,7 @@ LookupOperName(ParseState *pstate, List *opername, Oid oprleft, Oid oprright,
/*
* LookupOperWithArgs
* Like LookupOperName, but the argument types are specified by
- * a ObjectWithArg node.+ * a ObjectWithArgs node.
*/
Oid
LookupOperWithArgs(ObjectWithArgs *oper, bool noError)
diff --git a/src/backend/replication/logical/reorderbuffer.c b/src/backend/replication/logical/reorderbuffer.cindex e8ffa0492f..5fa3d7323e 100644--- a/src/backend/replication/logical/reorderbuffer.c+++ b/src/backend/replication/logical/reorderbuffer.c@@ -1381,7 +1381,7 @@ ReorderBufferCopySnap(ReorderBuffer *rb, Snapshot orig_snap,
snap->subxip[i++] = txn->xid;
/*
- * nsubxcnt isn't decreased when subtransactions abort, so count manually.+ * subxcnt isn't decreased when subtransactions abort, so count manually.
* Since it's an upper boundary it is safe to use it for the allocation
* above.
*/
diff --git a/src/backend/storage/lmgr/lock.c b/src/backend/storage/lmgr/lock.cindex 1b7053cb1c..98bb6921a3 100644--- a/src/backend/storage/lmgr/lock.c+++ b/src/backend/storage/lmgr/lock.c@@ -995,9 +995,9 @@ LockAcquireExtended(const LOCKTAG *locktag,
Assert(status == STATUS_FOUND);
/*
- * We can't acquire the lock immediately. If caller specified no- * blocking, remove useless table entries and return NOT_AVAIL without- * waiting.+ * We can't acquire the lock immediately. If caller specified+ * no blocking, remove useless table entries and return+ * LOCKACQUIRE_NOT_AVAIL without waiting.
*/
if (dontWait)
{
diff --git a/src/include/access/gin_private.h b/src/include/access/gin_private.hindex afb3e15721..cd83f5fe8f 100644--- a/src/include/access/gin_private.h+++ b/src/include/access/gin_private.h@@ -443,7 +443,7 @@ extern void ginInsertCleanup(GinState *ginstate, bool full_clean,
/* ginpostinglist.c */
-extern GinPostingList *ginCompressPostingList(const ItemPointer ptrs, int nptrs,+extern GinPostingList *ginCompressPostingList(const ItemPointer ipd, int nipd,
int maxsize, int *nwritten);
extern int ginPostingListDecodeAllSegmentsToTbm(GinPostingList *ptr, int totalsize, TIDBitmap *tbm);
diff --git a/src/include/access/xloginsert.h b/src/include/access/xloginsert.hindex df24089ea4..eeb0412b66 100644--- a/src/include/access/xloginsert.h+++ b/src/include/access/xloginsert.h@@ -42,7 +42,7 @@
extern void XLogBeginInsert(void);
extern void XLogSetRecordFlags(uint8 flags);
extern XLogRecPtr XLogInsert(RmgrId rmid, uint8 info);
-extern void XLogEnsureRecordSpace(int nbuffers, int ndatas);+extern void XLogEnsureRecordSpace(int max_block_id, int ndatas);
extern void XLogRegisterData(char *data, int len);
extern void XLogRegisterBuffer(uint8 block_id, Buffer buffer, uint8 flags);
extern void XLogRegisterBlock(uint8 block_id, RelFileNode *rnode,
diff --git a/src/include/commands/extension.h b/src/include/commands/extension.hindex c724430aec..02fc17d7c0 100644--- a/src/include/commands/extension.h+++ b/src/include/commands/extension.h@@ -43,7 +43,7 @@ extern ObjectAddress InsertExtensionTuple(const char *extName, Oid extOwner,
extern ObjectAddress ExecAlterExtensionStmt(ParseState *pstate, AlterExtensionStmt *stmt);
extern ObjectAddress ExecAlterExtensionContentsStmt(AlterExtensionContentsStmt *stmt,
- ObjectAddress *objAddress);+ ObjectAddress *objAddr);
extern Oid get_extension_oid(const char *extname, bool missing_ok);
extern char *get_extension_name(Oid ext_oid);
diff --git a/src/timezone/zic.c b/src/timezone/zic.cindex c27fb456d0..7a5a313c5f 100644--- a/src/timezone/zic.c+++ b/src/timezone/zic.c@@ -146,7 +146,7 @@ static char lowerit(char);
static void mkdirs(char const *, bool);
static void newabbr(const char *abbr);
static zic_t oadd(zic_t t1, zic_t t2);
-static void outzone(const struct zone *zp, ptrdiff_t ntzones);+static void outzone(const struct zone *zpfirst, ptrdiff_t zonecount);
static zic_t rpytime(const struct rule *rp, zic_t wantedy);
static void rulesub(struct rule *rp,
const char *loyearp, const char *hiyearp,
fixes_9.21-30.patch (text/x-patch, 4.4K) downloaddiff --git a/contrib/sepgsql/sepgsql.h b/contrib/sepgsql/sepgsql.hindex 7197e3cdf0..4787934650 100644--- a/contrib/sepgsql/sepgsql.h+++ b/contrib/sepgsql/sepgsql.h@@ -271,7 +271,7 @@ extern void sepgsql_avc_init(void);
*/
extern char *sepgsql_get_client_label(void);
extern void sepgsql_init_client_label(void);
-extern char *sepgsql_get_label(Oid relOid, Oid objOid, int32 subId);+extern char *sepgsql_get_label(Oid classId, Oid objectId, int32 subId);
extern void sepgsql_object_relabel(const ObjectAddress *object,
const char *seclabel);
diff --git a/src/backend/access/transam/clog.c b/src/backend/access/transam/clog.cindex 34c74d96f8..595c860aaa 100644--- a/src/backend/access/transam/clog.c+++ b/src/backend/access/transam/clog.c@@ -92,7 +92,7 @@ static int ZeroCLOGPage(int pageno, bool writeXlog);
static bool CLOGPagePrecedes(int page1, int page2);
static void WriteZeroPageXlogRec(int pageno);
static void WriteTruncateXlogRec(int pageno, TransactionId oldestXact,
- Oid oldestXidDb);+ Oid oldestXactDb);
static void TransactionIdSetPageStatus(TransactionId xid, int nsubxids,
TransactionId *subxids, XidStatus status,
XLogRecPtr lsn, int pageno,
diff --git a/src/backend/access/transam/multixact.c b/src/backend/access/transam/multixact.cindex 2e169dd3f9..7b2448e05b 100644--- a/src/backend/access/transam/multixact.c+++ b/src/backend/access/transam/multixact.c@@ -282,7 +282,7 @@ typedef struct MultiXactStateData
} MultiXactStateData;
/*
- * Last element of OldestMemberMXactID and OldestVisibleMXactId arrays.+ * Last element of OldestMemberMXactId and OldestVisibleMXactId arrays.
* Valid elements are (1..MaxOldestSlot); element 0 is never used.
*/
#define MaxOldestSlot (MaxBackends + max_prepared_xacts)
diff --git a/src/backend/catalog/objectaddress.c b/src/backend/catalog/objectaddress.cindex caf48cefa9..ce8a4e927d 100644--- a/src/backend/catalog/objectaddress.c+++ b/src/backend/catalog/objectaddress.c@@ -530,7 +530,7 @@ static const ObjectPropertyType ObjectProperty[] =
/*
* This struct maps the string object types as returned by
- * getObjectTypeDescription into ObjType enum values. Note that some enum+ * getObjectTypeDescription into ObjectType enum values. Note that some enum
* values can be obtained by different names, and that some string object types
* do not have corresponding values in the output enum. The user of this map
* must be careful to test for invalid values being returned.
diff --git a/src/backend/commands/copy.c b/src/backend/commands/copy.cindex 4f04d122c3..3aeef30b28 100644--- a/src/backend/commands/copy.c+++ b/src/backend/commands/copy.c@@ -176,7 +176,6 @@ typedef struct CopyStateData
* Working state for COPY FROM
*/
AttrNumber num_defaults;
- FmgrInfo oid_in_function;
FmgrInfo *in_functions; /* array of input functions for each attrs */
Oid *typioparams; /* array of element types for in_functions */
int *defmap; /* array of default att numbers */
diff --git a/src/backend/commands/vacuum.c b/src/backend/commands/vacuum.cindex e7b379dfda..046bb04732 100644--- a/src/backend/commands/vacuum.c+++ b/src/backend/commands/vacuum.c@@ -847,7 +847,7 @@ get_all_vacuum_rels(int options)
}
/*
- * vacuum_set_xid_limits() -- compute oldest-Xmin and freeze cutoff points+ * vacuum_set_xid_limits() -- compute oldestXmin and freeze cutoff points
*
* The output parameters are:
* - oldestXmin is the cutoff value used to distinguish whether tuples are
diff --git a/src/bin/pg_dump/pg_dump.h b/src/bin/pg_dump/pg_dump.hindex ccf2153fac..dfba58ac58 100644--- a/src/bin/pg_dump/pg_dump.h+++ b/src/bin/pg_dump/pg_dump.h@@ -18,10 +18,6 @@
#define oidcmp(x,y) ( ((x) < (y) ? -1 : ((x) > (y)) ? 1 : 0) )
-#define oideq(x,y) ( (x) == (y) )-#define oidle(x,y) ( (x) <= (y) )-#define oidge(x,y) ( (x) >= (y) )-#define oidzero(x) ( (x) == 0 )
/*
* The data structures used to store system catalog information. Every
diff --git a/src/include/storage/off.h b/src/include/storage/off.hindex 82655d1466..02e663f6ed 100644--- a/src/include/storage/off.h+++ b/src/include/storage/off.h@@ -26,7 +26,6 @@ typedef uint16 OffsetNumber;
#define InvalidOffsetNumber ((OffsetNumber) 0)
#define FirstOffsetNumber ((OffsetNumber) 1)
#define MaxOffsetNumber ((OffsetNumber) (BLCKSZ / sizeof(ItemIdData)))
-#define OffsetNumberMask (0xffff) /* valid uint16 bits */
/* ----------------
* support macros
fixes_9.31-40.patch (text/x-patch, 5.4K) downloaddiff --git a/doc/src/sgml/sslinfo.sgml b/doc/src/sgml/sslinfo.sgmlindex 0fde0fc10e..78ff83d642 100644--- a/doc/src/sgml/sslinfo.sgml+++ b/doc/src/sgml/sslinfo.sgml@@ -185,7 +185,7 @@ countryName (alias C)
localityName (alias L)
stateOrProvinceName (alias ST)
organizationName (alias O)
-organizationUnitName (alias OU)+organizationalUnitName (alias OU)
title
description
initials
diff --git a/src/backend/access/heap/heapam.c b/src/backend/access/heap/heapam.cindex 94309949fa..e33f019939 100644--- a/src/backend/access/heap/heapam.c+++ b/src/backend/access/heap/heapam.c@@ -74,7 +74,7 @@ static HeapTuple heap_prepare_insert(Relation relation, HeapTuple tup,
TransactionId xid, CommandId cid, int options);
static XLogRecPtr log_heap_update(Relation reln, Buffer oldbuf,
Buffer newbuf, HeapTuple oldtup,
- HeapTuple newtup, HeapTuple old_key_tup,+ HeapTuple newtup, HeapTuple old_key_tuple,
bool all_visible_cleared, bool new_all_visible_cleared);
static Bitmapset *HeapDetermineModifiedColumns(Relation relation,
Bitmapset *interesting_cols,
diff --git a/src/backend/commands/define.c b/src/backend/commands/define.cindex b6c618aa00..9c93e415f1 100644--- a/src/backend/commands/define.c+++ b/src/backend/commands/define.c@@ -139,7 +139,7 @@ defGetBoolean(DefElem *def)
/*
* The set of strings accepted here should match up with the
- * grammar's opt_boolean production.+ * grammar's opt_boolean_or_string production.
*/
if (pg_strcasecmp(sval, "true") == 0)
return true;
diff --git a/src/backend/commands/prepare.c b/src/backend/commands/prepare.cindex c12b613763..7e0a041fab 100644--- a/src/backend/commands/prepare.c+++ b/src/backend/commands/prepare.c@@ -143,7 +143,7 @@ PrepareQuery(PrepareStmt *stmt, const char *queryString,
}
/*
- * grammar only allows OptimizableStmt, so this check should be redundant+ * grammar only allows PreparableStmt, so this check should be redundant
*/
switch (query->commandType)
{
diff --git a/src/backend/nodes/tidbitmap.c b/src/backend/nodes/tidbitmap.cindex bf53459996..23905a9c35 100644--- a/src/backend/nodes/tidbitmap.c+++ b/src/backend/nodes/tidbitmap.c@@ -1021,7 +1021,7 @@ tbm_iterate(TBMIterator *iterator)
PagetableEntry *page;
int ntuples;
- /* In ONE_PAGE state, we don't allocate an spages[] array */+ /* In TBM_ONE_PAGE state, we don't allocate an spages[] array */
if (tbm->status == TBM_ONE_PAGE)
page = &tbm->entry1;
else
diff --git a/src/backend/storage/ipc/shm_mq.c b/src/backend/storage/ipc/shm_mq.cindex 4c245d1f85..551515960d 100644--- a/src/backend/storage/ipc/shm_mq.c+++ b/src/backend/storage/ipc/shm_mq.c@@ -1251,7 +1251,7 @@ shm_mq_inc_bytes_written(shm_mq *mq, Size n)
pg_atomic_read_u64(&mq->mq_bytes_written) + n);
}
-/* Shim for on_dsm_callback. */+/* Shim for on_dsm_detach callback. */
static void
shm_mq_detach_callback(dsm_segment *seg, Datum arg)
{
diff --git a/src/bin/pg_dump/pg_restore.c b/src/bin/pg_dump/pg_restore.cindex f9b1ae6809..40a6b3745c 100644--- a/src/bin/pg_dump/pg_restore.c+++ b/src/bin/pg_dump/pg_restore.c@@ -54,8 +54,6 @@
static void usage(const char *progname);
-typedef struct option optType;-
int
main(int argc, char **argv)
{
diff --git a/src/include/access/gin_private.h b/src/include/access/gin_private.hindex afb3e15721..9345052bff 100644--- a/src/include/access/gin_private.h+++ b/src/include/access/gin_private.h@@ -54,7 +54,7 @@ typedef struct GinState
bool oneCol; /* true if single-column index */
/*
- * origTupDesc is the nominal tuple descriptor of the index, ie, the i'th+ * origTupdesc is the nominal tuple descriptor of the index, ie, the i'th
* attribute shows the key type (not the input data type!) of the i'th
* index column. In a single-column index this describes the actual leaf
* index tuples. In a multi-column index, the actual leaf tuples contain
diff --git a/src/include/access/hash.h b/src/include/access/hash.hindex 107c3d01ae..f16eebeee2 100644--- a/src/include/access/hash.h+++ b/src/include/access/hash.h@@ -252,8 +252,8 @@ typedef struct HashMetaPageData
uint32 hashm_maxbucket; /* ID of maximum bucket in use */
uint32 hashm_highmask; /* mask to modulo into entire table */
uint32 hashm_lowmask; /* mask to modulo into lower half of table */
- uint32 hashm_ovflpoint; /* splitpoint from which ovflpgs being- * allocated */+ uint32 hashm_ovflpoint; /* splitpoint from which ovflpage+ * being allocated */
uint32 hashm_firstfree; /* lowest-number free ovflpage (bit#) */
uint32 hashm_nmaps; /* number of bitmap pages */
RegProcedure hashm_procid; /* hash function id from pg_proc */
diff --git a/src/interfaces/ecpg/ecpglib/execute.c b/src/interfaces/ecpg/ecpglib/execute.cindex 4e2016c7ae..de5c43a460 100644--- a/src/interfaces/ecpg/ecpglib/execute.c+++ b/src/interfaces/ecpg/ecpglib/execute.c@@ -2062,8 +2062,9 @@ ecpg_do_prologue(int lineno, const int compat, const int force_indicator,
/*------
* create a list of variables
*
- * The variables are listed with input variables preceding outputvariables- * The end of each group is marked by an end marker. per variable we list:+ * The variables are listed with input variables preceding output+ * variables. The end of each group is marked by an end marker.+ * Per variable we list:
*
* type - as defined in ecpgtype.h
* value - where to store the data
fixes_9.41-50.patch (text/x-patch, 5.3K) downloaddiff --git a/src/backend/access/gin/README b/src/backend/access/gin/READMEindex 838fdc0d63..76c12ae2f6 100644--- a/src/backend/access/gin/README+++ b/src/backend/access/gin/README@@ -270,7 +270,7 @@ is stored in the higher bits. That requires 43 bits in total, which
conveniently fits in at most 6 bytes.
A compressed posting list is passed around and stored on disk in a
-PackedPostingList struct. The first item in the list is stored uncompressed+GinPostingList struct. The first item in the list is stored uncompressed
as a regular ItemPointerData, followed by the length of the list in bytes,
followed by the packed items.
diff --git a/src/backend/access/gist/gistget.c b/src/backend/access/gist/gistget.cindex 4e0c500a22..1d3b022b34 100644--- a/src/backend/access/gist/gistget.c+++ b/src/backend/access/gist/gistget.c@@ -74,7 +74,7 @@ gistkillitems(IndexScanDesc scan)
/*
* Mark all killedItems as dead. We need no additional recheck, because,
- * if page was modified, pageLSN must have changed.+ * if page was modified, curPageLSN must have changed.
*/
for (i = 0; i < so->numKilled; i++)
{
diff --git a/src/backend/nodes/params.c b/src/backend/nodes/params.cindex f5d56138ee..cf4387e40f 100644--- a/src/backend/nodes/params.c+++ b/src/backend/nodes/params.c@@ -144,7 +144,7 @@ EstimateParamListSpace(ParamListInfo paramLI)
}
/*
- * Serialize a paramListInfo structure into caller-provided storage.+ * Serialize a ParamListInfo structure into caller-provided storage.
*
* We write the number of parameters first, as a 4-byte integer, and then
* write details for each parameter in turn. The details for each parameter
diff --git a/src/backend/optimizer/plan/planner.c b/src/backend/optimizer/plan/planner.cindex 8f51f59f8a..dacd757c24 100644--- a/src/backend/optimizer/plan/planner.c+++ b/src/backend/optimizer/plan/planner.c@@ -4242,7 +4242,7 @@ consider_groupingsets_paths(PlannerInfo *root,
* 2) If there are no empty sets and only unsortable sets, then the
* rollups list will be empty (and thus l_start == NULL), and
* group_pathkeys will be NIL; we must ensure that the vacuously-true
- * pathkeys_contain_in test doesn't cause us to crash.+ * pathkeys_contained_in test doesn't cause us to crash.
*/
if (l_start != NULL &&
pathkeys_contained_in(root->group_pathkeys, path->pathkeys))
diff --git a/src/bin/initdb/initdb.c b/src/bin/initdb/initdb.cindex 09eabfbe5a..9c303e9cc4 100644--- a/src/bin/initdb/initdb.c+++ b/src/bin/initdb/initdb.c@@ -1295,7 +1295,7 @@ setup_config(void)
err = WSAStartup(MAKEWORD(2, 2), &wsaData);
#endif
- /* for best results, this code should match parse_hba() */+ /* for best results, this code should match parse_hba_line() */
hints.ai_flags = AI_NUMERICHOST;
hints.ai_family = AF_UNSPEC;
hints.ai_socktype = 0;
diff --git a/src/bin/psql/describe.h b/src/bin/psql/describe.hindex 17736c3782..b50c7d209c 100644--- a/src/bin/psql/describe.h+++ b/src/bin/psql/describe.h@@ -31,7 +31,7 @@ extern bool describeOperators(const char *pattern, bool verbose, bool showSystem
extern bool describeRoles(const char *pattern, bool verbose, bool showSystem);
/* \drds */
-extern bool listDbRoleSettings(const char *pattern1, const char *pattern2);+extern bool listDbRoleSettings(const char *pattern, const char *pattern2);
/* \z (or \dp) */
extern bool permissionsList(const char *pattern);
diff --git a/src/bin/psql/tab-complete.c b/src/bin/psql/tab-complete.cindex 3f7001fb69..d40800709b 100644--- a/src/bin/psql/tab-complete.c+++ b/src/bin/psql/tab-complete.c@@ -1686,7 +1686,7 @@ psql_completion(const char *text, int start, int end)
/* ALTER LANGUAGE <name> */
else if (Matches("ALTER", "LANGUAGE", MatchAny))
- COMPLETE_WITH("OWNER_TO", "RENAME TO");+ COMPLETE_WITH("OWNER TO", "RENAME TO");
/* ALTER LARGE OBJECT <oid> */
else if (Matches("ALTER", "LARGE", "OBJECT", MatchAny))
diff --git a/src/include/parser/parse_func.h b/src/include/parser/parse_func.hindex d1ec2d97f2..5a3b287eaf 100644--- a/src/include/parser/parse_func.h+++ b/src/include/parser/parse_func.h@@ -11,8 +11,8 @@
*
*-------------------------------------------------------------------------
*/
-#ifndef PARSER_FUNC_H-#define PARSER_FUNC_H+#ifndef PARSE_FUNC_H+#define PARSE_FUNC_H
#include "catalog/namespace.h"
#include "parser/parse_node.h"
diff --git a/src/include/storage/bufpage.h b/src/include/storage/bufpage.hindex 4ef6d8ddd4..f18d703c7a 100644--- a/src/include/storage/bufpage.h+++ b/src/include/storage/bufpage.h@@ -402,8 +402,6 @@ do { \
TransactionIdPrecedes(xid, ((PageHeader) (page))->pd_prune_xid)) \
((PageHeader) (page))->pd_prune_xid = (xid); \
} while (0)
-#define PageClearPrunable(page) \- (((PageHeader) (page))->pd_prune_xid = InvalidTransactionId)
/* ----------------------------------------------------------------
diff --git a/src/include/utils/jsonapi.h b/src/include/utils/jsonapi.hindex 5f4d479a7b..1c56acca55 100644--- a/src/include/utils/jsonapi.h+++ b/src/include/utils/jsonapi.h@@ -94,7 +94,7 @@ typedef struct JsonSemAction
} JsonSemAction;
/*
- * parse_json will parse the string in the lex calling the+ * pg_parse_json will parse the string in the lex calling the
* action functions in sem at the appropriate points. It is
* up to them to keep what state they need in semstate. If they
* need access to the state of the lexer, then its pointer
fixes_9.51-60.patch (text/x-patch, 5.1K) downloaddiff --git a/contrib/test_decoding/test_decoding.c b/contrib/test_decoding/test_decoding.cindex a0ded67b9a..6c33c4bded 100644--- a/contrib/test_decoding/test_decoding.c+++ b/contrib/test_decoding/test_decoding.c@@ -24,7 +24,7 @@
PG_MODULE_MAGIC;
-/* These must be available to pg_dlsym() */+/* These must be available to dlsym() */
extern void _PG_init(void);
extern void _PG_output_plugin_init(OutputPluginCallbacks *cb);
diff --git a/src/backend/access/hash/hashinsert.c b/src/backend/access/hash/hashinsert.cindex 5321762d5e..89876d2ccd 100644--- a/src/backend/access/hash/hashinsert.c+++ b/src/backend/access/hash/hashinsert.c@@ -257,8 +257,8 @@ restart_insert:
* _hash_pgaddtup() -- add a tuple to a particular page in the index.
*
* This routine adds the tuple to the page as requested; it does not write out
- * the page. It is an error to call pgaddtup() without pin and write lock on- * the target buffer.+ * the page. It is an error to call this function without pin and write lock+ * on the target buffer.
*
* Returns the offset number at which the tuple was inserted. This function
* is responsible for preserving the condition that tuples in a hash index
diff --git a/src/backend/executor/nodeWindowAgg.c b/src/backend/executor/nodeWindowAgg.cindex def00cd7c5..ac441a19b8 100644--- a/src/backend/executor/nodeWindowAgg.c+++ b/src/backend/executor/nodeWindowAgg.c@@ -93,7 +93,7 @@ typedef struct WindowStatePerFuncData
bool resulttypeByVal;
bool plain_agg; /* is it just a plain aggregate function? */
- int aggno; /* if so, index of its PerAggData */+ int aggno; /* if so, index of its WindowStatePerAggData */
WindowObject winobj; /* object used in window function API */
} WindowStatePerFuncData;
@@ -142,7 +142,7 @@ typedef struct WindowStatePerAggData
resulttypeByVal,
transtypeByVal;
- int wfuncno; /* index of associated PerFuncData */+ int wfuncno; /* index of associated WindowStatePerFuncData */
/* Context holding transition value and possibly other subsidiary data */
MemoryContext aggcontext; /* may be private, or winstate->aggcontext */
diff --git a/src/backend/utils/adt/genfile.c b/src/backend/utils/adt/genfile.cindex f526106530..5d4f26a490 100644--- a/src/backend/utils/adt/genfile.c+++ b/src/backend/utils/adt/genfile.c@@ -325,7 +325,7 @@ pg_read_binary_file(PG_FUNCTION_ARGS)
/*
* Wrapper functions for the 1 and 3 argument variants of pg_read_file_v2()
- * and pg_binary_read_file().+ * and pg_read_binary_file().
*
* These are necessary to pass the sanity check in opr_sanity, which checks
* that all built-in functions that share the implementing C function take
diff --git a/src/backend/utils/mmgr/slab.c b/src/backend/utils/mmgr/slab.cindex e23669fb4f..bd8d200982 100644--- a/src/backend/utils/mmgr/slab.c+++ b/src/backend/utils/mmgr/slab.c@@ -40,7 +40,7 @@
*
* For each block, we maintain pointer to the first free chunk - this is quite
* cheap and allows us to skip all the preceding used chunks, eliminating
- * a significant number of lookups in many common usage patters. In the worst+ * a significant number of lookups in many common usage patterns. In the worst
* case this performs as if the pointer was not maintained.
*
* We cache the freelist index for the blocks with the fewest free chunks
diff --git a/src/include/access/commit_ts.h b/src/include/access/commit_ts.hindex e22bfbf3e6..1433aa3ff7 100644--- a/src/include/access/commit_ts.h+++ b/src/include/access/commit_ts.h@@ -35,7 +35,7 @@ extern Size CommitTsShmemSize(void);
extern void CommitTsShmemInit(void);
extern void BootStrapCommitTs(void);
extern void StartupCommitTs(void);
-extern void CommitTsParameterChange(bool xlrecvalue, bool pgcontrolvalue);+extern void CommitTsParameterChange(bool newvalue, bool oldvalue);
extern void CompleteCommitTsInitialization(void);
extern void ShutdownCommitTs(void);
extern void CheckPointCommitTs(void);
diff --git a/src/include/mb/pg_wchar.h b/src/include/mb/pg_wchar.hindex 0339b98041..3e3e6c470e 100644--- a/src/include/mb/pg_wchar.h+++ b/src/include/mb/pg_wchar.h@@ -289,7 +289,7 @@ typedef enum pg_enc
#define PG_ENCODING_BE_LAST PG_KOI8U
/*
- * Please use these tests before access to pg_encconv_tbl[]+ * Please use these tests before access to pg_enc2name_tbl[]
* or to other places...
*/
#define PG_VALID_BE_ENCODING(_enc) \
diff --git a/src/include/port/atomics.h b/src/include/port/atomics.hindex f93009eff5..fab66f8e98 100644--- a/src/include/port/atomics.h+++ b/src/include/port/atomics.h@@ -178,7 +178,7 @@ pg_atomic_init_flag(volatile pg_atomic_flag *ptr)
}
/*
- * pg_atomic_test_and_set_flag - TAS()+ * pg_atomic_test_set_flag - TAS()
*
* Returns true if the flag has successfully been set, false otherwise.
*
diff --git a/src/tools/msvc/Project.pm b/src/tools/msvc/Project.pmindex 0d3554659b..7d16a36868 100644--- a/src/tools/msvc/Project.pm+++ b/src/tools/msvc/Project.pm@@ -230,7 +230,7 @@ sub AddDir
if ($filter eq "LIBOBJS")
{
no warnings qw(once);
- if (grep(/$p/, @main::pgportfiles, @main::pgcommonfiles)+ if (grep(/$p/, @main::pgportfiles)
== 1)
{
$p =~ s/\.c/\.o/;
fixes_9.61-70.patch (text/x-patch, 6.1K) downloaddiff --git a/contrib/pgcrypto/pgp-pgsql.c b/contrib/pgcrypto/pgp-pgsql.cindex 0984e01a14..3feadf7b70 100644--- a/contrib/pgcrypto/pgp-pgsql.c+++ b/contrib/pgcrypto/pgp-pgsql.c@@ -761,7 +761,7 @@ pgp_pub_decrypt_text(PG_FUNCTION_ARGS)
*/
/*
- * Helper function for pgp_armor. Converts arrays of keys and values into+ * Helper function for pg_armor. Converts arrays of keys and values into
* plain C arrays, and checks that they don't contain invalid characters.
*/
static int
diff --git a/doc/src/sgml/libpq.sgml b/doc/src/sgml/libpq.sgmlindex ef2e97afd1..e7295abda2 100644--- a/doc/src/sgml/libpq.sgml+++ b/doc/src/sgml/libpq.sgml@@ -5509,7 +5509,7 @@ int PQendcopy(PGconn *conn);
It should either be issued when the last string has been sent
to the server using <xref linkend="libpq-PQputline"/> or when the
last string has been received from the server using
- <function>PGgetline</function>. It must be issued or the server+ <function>PQgetline</function>. It must be issued or the server
will get <quote>out of sync</quote> with the client. Upon return
from this function, the server is ready to receive the next SQL
command. The return value is 0 on successful completion,
diff --git a/src/backend/catalog/aclchk.c b/src/backend/catalog/aclchk.cindex 291412e305..88ce37c08d 100644--- a/src/backend/catalog/aclchk.c+++ b/src/backend/catalog/aclchk.c@@ -4114,7 +4114,7 @@ pg_largeobject_aclmask_snapshot(Oid lobj_oid, Oid roleid,
return mask;
/*
- * Get the largeobject's ACL from pg_language_metadata+ * Get the largeobject's ACL from pg_largeobject_metadata
*/
pg_lo_meta = table_open(LargeObjectMetadataRelationId,
AccessShareLock);
diff --git a/src/backend/catalog/information_schema.sql b/src/backend/catalog/information_schema.sqlindex 9c21ac7c62..c42e76ea89 100644--- a/src/backend/catalog/information_schema.sql+++ b/src/backend/catalog/information_schema.sql@@ -48,10 +48,6 @@ CREATE FUNCTION _pg_expandarray(IN anyarray, OUT x anyelement, OUT n int)
pg_catalog.array_upper($1,1),
1) as g(s)';
-CREATE FUNCTION _pg_keysequal(smallint[], smallint[]) RETURNS boolean- LANGUAGE sql IMMUTABLE PARALLEL SAFE -- intentionally not STRICT, to allow inlining- AS 'select $1 operator(pg_catalog.<@) $2 and $2 operator(pg_catalog.<@) $1';-
/* Given an index's OID and an underlying-table column number, return the
* column's position in the index (NULL if not there) */
CREATE FUNCTION _pg_index_position(oid, smallint) RETURNS int
diff --git a/src/backend/optimizer/plan/planner.c b/src/backend/optimizer/plan/planner.cindex 8f51f59f8a..2cde93e9ad 100644--- a/src/backend/optimizer/plan/planner.c+++ b/src/backend/optimizer/plan/planner.c@@ -5177,7 +5177,7 @@ make_group_input_target(PlannerInfo *root, PathTarget *final_target)
* a regular aggregation node would, plus any aggregates used in HAVING;
* except that the Aggref nodes should be marked as partial aggregates.
*
- * In addition, we'd better emit any Vars and PlaceholderVars that are+ * In addition, we'd better emit any Vars and PlaceHolderVars that are
* used outside of Aggrefs in the aggregation tlist and HAVING. (Presumably,
* these would be Vars that are grouped by or used in grouping expressions.)
*
diff --git a/src/backend/postmaster/pgstat.c b/src/backend/postmaster/pgstat.cindex b4f2b28b51..2bb14cdd02 100644--- a/src/backend/postmaster/pgstat.c+++ b/src/backend/postmaster/pgstat.c@@ -6122,7 +6122,7 @@ pgstat_recv_resetcounter(PgStat_MsgResetcounter *msg, int len)
}
/* ----------
- * pgstat_recv_resetshared() -+ * pgstat_recv_resetsharedcounter() -
*
* Reset some shared statistics of the cluster.
* ----------
diff --git a/src/backend/utils/time/snapmgr.c b/src/backend/utils/time/snapmgr.cindex 40fe6ed3d3..47b0517596 100644--- a/src/backend/utils/time/snapmgr.c+++ b/src/backend/utils/time/snapmgr.c@@ -1171,7 +1171,7 @@ AtEOXact_Snapshot(bool isCommit, bool resetXmin)
/*
* During normal commit processing, we call ProcArrayEndTransaction() to
- * reset the PgXact->xmin. That call happens prior to the call to+ * reset the MyPgXact->xmin. That call happens prior to the call to
* AtEOXact_Snapshot(), so we need not touch xmin here at all.
*/
if (resetXmin)
diff --git a/src/include/catalog/catversion.h b/src/include/catalog/catversion.hindex 507b082acb..80def7d401 100644--- a/src/include/catalog/catversion.h+++ b/src/include/catalog/catversion.h@@ -25,8 +25,8 @@
*
* The rule for developers is: if you commit a change that requires
* an initdb, you should update the catalog version number (as well as
- * notifying the pghackers mailing list, which has been the informal- * practice for a long time).+ * notifying the pgsql-hackers mailing list, which has been the+ * informal practice for a long time).
*
* The catalog version number is placed here since modifying files in
* include/catalog is the most common kind of initdb-forcing change.
diff --git a/src/interfaces/ecpg/pgtypeslib/datetime.c b/src/interfaces/ecpg/pgtypeslib/datetime.cindex 53fcd2157f..9ee460b9a1 100644--- a/src/interfaces/ecpg/pgtypeslib/datetime.c+++ b/src/interfaces/ecpg/pgtypeslib/datetime.c@@ -623,7 +623,7 @@ PGTYPESdate_defmt_asc(date * d, const char *fmt, const char *str)
/*
* evil[tm] hack: if we read the pgtypes_date_months and haven't
- * found a match, reset list to point to pgtypes_date_months_short+ * found a match, reset list to point to months (abbreviations)
* and reset the counter variable i
*/
if (list == pgtypes_date_months)
diff --git a/src/interfaces/libpq/libpq-int.h b/src/interfaces/libpq/libpq-int.hindex fcf2bc2cab..0fe3e53539 100644--- a/src/interfaces/libpq/libpq-int.h+++ b/src/interfaces/libpq/libpq-int.h@@ -169,7 +169,7 @@ struct pg_result
int ntups;
int numAttributes;
PGresAttDesc *attDescs;
- PGresAttValue **tuples; /* each PGresTuple is an array of+ PGresAttValue **tuples; /* each PGresult tuple is an array of
* PGresAttValue's */
int tupArrSize; /* allocated size of tuples array */
int numParameters;
fixes_9.71-80.patch (text/x-patch, 5.2K) downloaddiff --git a/doc/src/sgml/backup.sgml b/doc/src/sgml/backup.sgmlindex 47dacee17c..dcfcbae9d9 100644--- a/doc/src/sgml/backup.sgml+++ b/doc/src/sgml/backup.sgml@@ -456,7 +456,7 @@ tar -cf backup.tar /usr/local/pgsql/data
</sect1>
<sect1 id="continuous-archiving">
- <title>Continuous Archiving and Point-in-Time Recovery (PITR)</title>+ <title>Continuous Archiving and Point-In-Time Recovery (PITR)</title>
<indexterm zone="backup">
<primary>continuous archiving</primary>
diff --git a/src/backend/access/gin/ginbtree.c b/src/backend/access/gin/ginbtree.cindex 11a8ed7bbc..4c29261256 100644--- a/src/backend/access/gin/ginbtree.c+++ b/src/backend/access/gin/ginbtree.c@@ -650,7 +650,7 @@ ginPlaceToPage(GinBtree btree, GinBtreeStack *stack,
}
else
{
- elog(ERROR, "invalid return code from GIN placeToPage method: %d", rc);+ elog(ERROR, "invalid return code from GIN beginPlaceToPage method: %d", rc);
result = false; /* keep compiler quiet */
}
diff --git a/src/backend/libpq/pqcomm.c b/src/backend/libpq/pqcomm.cindex 384887e70d..d8d863dda2 100644--- a/src/backend/libpq/pqcomm.c+++ b/src/backend/libpq/pqcomm.c@@ -44,8 +44,8 @@
* StreamClose - Close a client/backend connection
* TouchSocketFiles - Protect socket files against /tmp cleaners
* pq_init - initialize libpq at backend startup
- * pq_comm_reset - reset libpq during error recovery- * pq_close - shutdown libpq at backend exit+ * socket_comm_reset - reset libpq during error recovery+ * socket_close - shutdown libpq at backend exit
*
* low-level I/O:
* pq_getbytes - get a known number of bytes from connection
diff --git a/src/backend/optimizer/geqo/geqo_selection.c b/src/backend/optimizer/geqo/geqo_selection.cindex f8b75e6091..0a6bacc2f2 100644--- a/src/backend/optimizer/geqo/geqo_selection.c+++ b/src/backend/optimizer/geqo/geqo_selection.c@@ -91,7 +91,7 @@ geqo_selection(PlannerInfo *root, Chromosome *momma, Chromosome *daddy,
static int
linear_rand(PlannerInfo *root, int pool_size, double bias)
{
- double index; /* index between 0 and pop_size */+ double index; /* index between 0 and pool_size */
double max = (double) pool_size;
/*
diff --git a/src/include/storage/itemptr.h b/src/include/storage/itemptr.hindex 7eb274adac..d73eed585e 100644--- a/src/include/storage/itemptr.h+++ b/src/include/storage/itemptr.h@@ -22,8 +22,8 @@
*
* This is a pointer to an item within a disk page of a known file
* (for example, a cross-link from an index to its parent table).
- * blkid tells us which block, posid tells us which entry in the linp- * (ItemIdData) array we want.+ * ip_blkid tells us which block, ip_posid tells us which entry in+ * the linp (ItemIdData) array we want.
*
* Note: because there is an item pointer in each tuple header and index
* tuple header on disk, it's very important not to waste space with
diff --git a/src/interfaces/libpq/fe-misc.c b/src/interfaces/libpq/fe-misc.cindex 2d44845ccb..18fec4a173 100644--- a/src/interfaces/libpq/fe-misc.c+++ b/src/interfaces/libpq/fe-misc.c@@ -7,7 +7,7 @@
* miscellaneous useful functions
*
* The communication routines here are analogous to the ones in
- * backend/libpq/pqcomm.c and backend/libpq/pqcomprim.c, but operate+ * backend/libpq/pqcomm.c and backend/libpq/pqformat.c, but operate
* in the considerably different environment of the frontend libpq.
* In particular, we work with a bare nonblock-mode socket, rather than
* a stdio stream, so that we can avoid unwanted blocking of the application.
diff --git a/src/nls-global.mk b/src/nls-global.mkindex 5bb77b40eb..f06c91341c 100644--- a/src/nls-global.mk+++ b/src/nls-global.mk@@ -35,8 +35,6 @@ else
LANGUAGES = $(AVAIL_LANGUAGES)
endif
-PO_FILES = $(addprefix po/, $(addsuffix .po, $(LANGUAGES)))-ALL_PO_FILES = $(addprefix po/, $(addsuffix .po, $(AVAIL_LANGUAGES)))
MO_FILES = $(addprefix po/, $(addsuffix .mo, $(LANGUAGES)))
ifdef XGETTEXT
diff --git a/src/pl/plpgsql/src/pl_comp.c b/src/pl/plpgsql/src/pl_comp.cindex 30c7e967df..30246c5329 100644--- a/src/pl/plpgsql/src/pl_comp.c+++ b/src/pl/plpgsql/src/pl_comp.c@@ -2452,7 +2452,7 @@ delete_function(PLpgSQL_function *func)
plpgsql_free_function_memory(func);
}
-/* exported so we can call it from plpgsql_init() */+/* exported so we can call it from _PG_init() */
void
plpgsql_HashTableInit(void)
{
diff --git a/src/pl/plpgsql/src/plpgsql.h b/src/pl/plpgsql/src/plpgsql.hindex a5a5ef9de7..8b32075df0 100644--- a/src/pl/plpgsql/src/plpgsql.h+++ b/src/pl/plpgsql/src/plpgsql.h@@ -685,7 +685,7 @@ typedef struct PLpgSQL_stmt_fori
/*
* PLpgSQL_stmt_forq represents a FOR statement running over a SQL query.
* It is the common supertype of PLpgSQL_stmt_fors, PLpgSQL_stmt_forc
- * and PLpgSQL_dynfors.+ * and PLpgSQL_stmt_dynfors.
*/
typedef struct PLpgSQL_stmt_forq
{
diff --git a/src/pl/plpython/plpy_procedure.c b/src/pl/plpython/plpy_procedure.cindex 50b07cad82..6e08c03fcd 100644--- a/src/pl/plpython/plpy_procedure.c+++ b/src/pl/plpython/plpy_procedure.c@@ -471,7 +471,7 @@ PLy_procedure_munge_source(const char *name, const char *src)
*mp = '\0';
if (mp > (mrc + mlen))
- elog(FATAL, "buffer overrun in PLy_munge_source");+ elog(FATAL, "buffer overrun in PLy_procedure_munge_source");
return mrc;
}
fixes_9.81-90.patch (text/x-patch, 5.6K) downloaddiff --git a/contrib/pg_standby/pg_standby.c b/contrib/pg_standby/pg_standby.cindex 23f706b21d..031b1b5cd5 100644--- a/contrib/pg_standby/pg_standby.c+++ b/contrib/pg_standby/pg_standby.c@@ -58,7 +58,6 @@ char *triggerPath; /* where to find the trigger file? */
char *xlogFilePath; /* where we are going to restore to */
char *nextWALFileName; /* the file we need to get from archive */
char *restartWALFileName; /* the file from which we can restart restore */
-char *priorWALFileName; /* the file we need to get from archive */
char WALFilePath[MAXPGPATH * 2]; /* the file path including archive */
char restoreCommand[MAXPGPATH]; /* run this to restore */
char exclusiveCleanupFileName[MAXFNAMELEN]; /* the file we need to get
diff --git a/src/backend/regex/regcomp.c b/src/backend/regex/regcomp.cindex 8765af71d4..91078dcd80 100644--- a/src/backend/regex/regcomp.c+++ b/src/backend/regex/regcomp.c@@ -288,7 +288,6 @@ struct vars
#define NWBDRY 'W' /* non-word-boundary constraint */
#define SBEGIN 'A' /* beginning of string (even if not BOL) */
#define SEND 'Z' /* end of string (even if not EOL) */
-#define PREFER 'P' /* length preference */
/* is an arc colored, and hence on a color chain? */
#define COLORED(a) \
diff --git a/src/backend/storage/lmgr/predicate.c b/src/backend/storage/lmgr/predicate.cindex 2d709420c3..85a629f4fc 100644--- a/src/backend/storage/lmgr/predicate.c+++ b/src/backend/storage/lmgr/predicate.c@@ -364,7 +364,7 @@ static SERIALIZABLEXACT *OldCommittedSxact;
* These configuration variables are used to set the predicate lock table size
* and to control promotion of predicate locks to coarser granularity in an
* attempt to degrade performance (mostly as false positive serialization
- * failure) gracefully in the face of memory pressurel+ * failure) gracefully in the face of memory pressure.
*/
int max_predicate_locks_per_xact; /* set by guc.c */
int max_predicate_locks_per_relation; /* set by guc.c */
@@ -4809,7 +4809,7 @@ OnConflict_CheckForSerializationFailure(const SERIALIZABLEXACT *reader,
}
/*
- * PreCommit_CheckForSerializableConflicts+ * PreCommit_CheckForSerializationFailure
* Check for dangerous structures in a serializable transaction
* at commit.
*
diff --git a/src/backend/utils/adt/ruleutils.c b/src/backend/utils/adt/ruleutils.cindex 0c58f1f109..3af8bce206 100644--- a/src/backend/utils/adt/ruleutils.c+++ b/src/backend/utils/adt/ruleutils.c@@ -98,7 +98,6 @@
/* macros to test if pretty action needed */
#define PRETTY_PAREN(context) ((context)->prettyFlags & PRETTYFLAG_PAREN)
#define PRETTY_INDENT(context) ((context)->prettyFlags & PRETTYFLAG_INDENT)
-#define PRETTY_SCHEMA(context) ((context)->prettyFlags & PRETTYFLAG_SCHEMA)
/* ----------
@@ -115,7 +114,7 @@ typedef struct
List *windowTList; /* targetlist for resolving WINDOW clause */
int prettyFlags; /* enabling of pretty-print functions */
int wrapColumn; /* max line length, or -1 for no limit */
- int indentLevel; /* current indent level for prettyprint */+ int indentLevel; /* current indent level for pretty-print */
bool varprefix; /* true to print prefixes on Vars */
ParseExprKind special_exprkind; /* set only for exprkinds needing special
* handling */
diff --git a/src/interfaces/libpq/fe-exec.c b/src/interfaces/libpq/fe-exec.cindex 48d4ebdd96..b3c59a0992 100644--- a/src/interfaces/libpq/fe-exec.c+++ b/src/interfaces/libpq/fe-exec.c@@ -3291,7 +3291,7 @@ PQflush(PGconn *conn)
* PQfreemem - safely frees memory allocated
*
* Needed mostly by Win32, unless multithreaded DLL (/MD in VC6)
- * Used for freeing memory from PQescapeByte()a/PQunescapeBytea()+ * Used for freeing memory from PQescapeBytea()/PQunescapeBytea()
*/
void
PQfreemem(void *ptr)
diff --git a/src/interfaces/libpq/libpq-fe.h b/src/interfaces/libpq/libpq-fe.hindex 26198fc1de..039824196c 100644--- a/src/interfaces/libpq/libpq-fe.h+++ b/src/interfaces/libpq/libpq-fe.h@@ -565,10 +565,10 @@ extern void PQdisplayTuples(const PGresult *res,
extern void PQprintTuples(const PGresult *res,
FILE *fout, /* output stream */
- int printAttName, /* print attribute names */- int terseOutput, /* delimiter bars */- int width); /* width of column, if 0, use variable- * width */+ int PrintAttNames, /* print attribute names */+ int TerseOutput, /* delimiter bars */+ int colWidth); /* width of column, if 0,+ * use variable width */
/* === in fe-lobj.c === */
diff --git a/src/interfaces/libpq/libpq-int.h b/src/interfaces/libpq/libpq-int.hindex fcf2bc2cab..a0b142b501 100644--- a/src/interfaces/libpq/libpq-int.h+++ b/src/interfaces/libpq/libpq-int.h@@ -232,7 +232,8 @@ typedef enum
PGQUERY_DESCRIBE /* Describe Statement or Portal */
} PGQueryClass;
-/* PGSetenvStatusType defines the state of the PQSetenv state machine */+/* PGSetenvStatusType defines the state of the pqSetenv state machine */+
/* (this is used only for 2.0-protocol connections) */
typedef enum
{
diff --git a/src/test/isolation/Makefile b/src/test/isolation/Makefileindex c3c8280ea2..dc22e5ca9d 100644--- a/src/test/isolation/Makefile+++ b/src/test/isolation/Makefile@@ -55,7 +55,7 @@ installcheck: all
check: all
$(pg_isolation_regress_check) --schedule=$(srcdir)/isolation_schedule
-# Versions of the check tests that include the prepared_transactions test+# Versions of the check tests that include the prepared-transactions test
# It only makes sense to run these if set up to use prepared transactions,
# via TEMP_CONFIG for the check case, or via the postgresql.conf for the
# installcheck case.
fixes_9.91-95.patch (text/x-patch, 2.2K) downloaddiff --git a/contrib/pgcrypto/px.h b/contrib/pgcrypto/px.hindex cef9c4b456..1f0a93b1d6 100644--- a/contrib/pgcrypto/px.h+++ b/contrib/pgcrypto/px.h@@ -50,9 +50,6 @@ void *px_realloc(void *p, size_t s);
void px_free(void *p);
#endif
-/* max len of 'type' parms */-#define PX_MAX_NAMELEN 128-
/* max salt returned */
#define PX_MAX_SALT_LEN 128
@@ -210,8 +207,6 @@ void px_debug(const char *fmt,...) pg_attribute_printf(1, 2);
#define px_md_free(md) (md)->free(md)
#define px_hmac_result_size(hmac) (hmac)->result_size(hmac)
-#define px_hmac_block_size(hmac) (hmac)->block_size(hmac)-#define px_hmac_reset(hmac) (hmac)->reset(hmac)
#define px_hmac_init(hmac, key, klen) (hmac)->init(hmac, key, klen)
#define px_hmac_update(hmac, data, dlen) (hmac)->update(hmac, data, dlen)
#define px_hmac_finish(hmac, buf) (hmac)->finish(hmac, buf)
diff --git a/contrib/pgcrypto/sha1.c b/contrib/pgcrypto/sha1.cindex fb6a57d917..64671ac64d 100644--- a/contrib/pgcrypto/sha1.c+++ b/contrib/pgcrypto/sha1.c@@ -59,16 +59,6 @@ static uint32 _K[] = {0x5a827999, 0x6ed9eba1, 0x8f1bbcdc, 0xca62c1d6};
#define BCOUNT (ctxt->c.b64[0] / 8)
#define W(n) (ctxt->m.b32[(n)])
-#define PUTBYTE(x) \-do { \- ctxt->m.b8[(COUNT % 64)] = (x); \- COUNT++; \- COUNT %= 64; \- ctxt->c.b64[0] += 8; \- if (COUNT % 64 == 0) \- sha1_step(ctxt); \-} while (0)-
#define PUTPAD(x) \
do { \
ctxt->m.b8[(COUNT % 64)] = (x); \
diff --git a/src/backend/storage/ipc/procarray.c b/src/backend/storage/ipc/procarray.cindex fadab62950..469aac38c4 100644--- a/src/backend/storage/ipc/procarray.c+++ b/src/backend/storage/ipc/procarray.c@@ -265,7 +265,6 @@ CreateSharedProcArray(void)
&found);
}
- /* Register and initialize fields of ProcLWLockTranche */
LWLockRegisterTranche(LWTRANCHE_PROC, "proc");
}
diff --git a/src/include/storage/proc.h b/src/include/storage/proc.hindex ac7ee72952..281e1db725 100644--- a/src/include/storage/proc.h+++ b/src/include/storage/proc.h@@ -332,4 +332,4 @@ extern PGPROC *AuxiliaryPidGetProc(int pid);
extern void BecomeLockGroupLeader(void);
extern bool BecomeLockGroupMember(PGPROC *leader, int pid);
-#endif /* PROC_H */+#endif /* _PROC_H_ */
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: exclusion@gmail.com, michael@paquier.xyz
Subject: Re: Fix typos and inconsistencies for HEAD (take 9)
In-Reply-To: <7ab243e0-116d-3e44-d120-76b3df7abefd@gmail.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox