diff --git a/contrib/auto_explain/auto_explain.c b/contrib/auto_explain/auto_explain.c index af68479..7ecb9b4 100644 --- a/contrib/auto_explain/auto_explain.c +++ b/contrib/auto_explain/auto_explain.c @@ -181,7 +181,7 @@ explain_ExecutorStart(QueryDesc *queryDesc, int eflags) { if (auto_explain_enabled()) { - /* Enable per-node instrumentation iff log_analyze is required. */ + /* Enable per-node instrumentation if and only if log_analyze is required. */ if (auto_explain_log_analyze && (eflags & EXEC_FLAG_EXPLAIN_ONLY) == 0) { if (auto_explain_log_timing) diff --git a/contrib/pgbench/pgbench.c b/contrib/pgbench/pgbench.c index 16b7ab5..b0837c3 100644 --- a/contrib/pgbench/pgbench.c +++ b/contrib/pgbench/pgbench.c @@ -909,7 +909,7 @@ agg_vals_init(AggVals *aggs, instr_time start) aggs->start_time = INSTR_TIME_GET_DOUBLE(start); } -/* return false iff client should be disconnected */ +/* return false if and only if client should be disconnected */ static bool doCustom(TState *thread, CState *st, instr_time *conn_time, FILE *logfile, AggVals *agg) { diff --git a/doc/src/sgml/stylesheet.dsl b/doc/src/sgml/stylesheet.dsl index 29e885d..7682d79 100644 --- a/doc/src/sgml/stylesheet.dsl +++ b/doc/src/sgml/stylesheet.dsl @@ -86,7 +86,7 @@ ; ((member (gi ancestor) (book-element-list)) ; #t) ;; Add indirect references to the section or component a block - ;; is in iff chapters aren't autolabelled. (Otherwise "Figure 1-3" + ;; is in if and only if chapters aren't autolabelled. (Otherwise "Figure 1-3" ;; is sufficient) ((and (member (gi target) (block-element-list)) (not %chapter-autolabel%)) diff --git a/src/backend/access/common/heaptuple.c b/src/backend/access/common/heaptuple.c index 3d8d2eb..39b04d8 100644 --- a/src/backend/access/common/heaptuple.c +++ b/src/backend/access/common/heaptuple.c @@ -261,7 +261,7 @@ heap_fill_tuple(TupleDesc tupleDesc, */ /* ---------------- - * heap_attisnull - returns TRUE iff tuple attribute is not present + * heap_attisnull - returns TRUE if and only if tuple attribute is not present * ---------------- */ bool diff --git a/src/backend/access/gin/ginget.c b/src/backend/access/gin/ginget.c index 4bdbd45..a8e0625 100644 --- a/src/backend/access/gin/ginget.c +++ b/src/backend/access/gin/ginget.c @@ -705,7 +705,7 @@ entryGetItem(GinState *ginstate, GinScanEntry entry) * The current item is the smallest curItem among the inputs. key->curItem * is set to that value. key->curItemMatches is set to indicate whether that * TID passes the consistentFn test. If so, key->recheckCurItem is set true - * iff recheck is needed for this item pointer (including the case where the + * if and only if recheck is needed for this item pointer (including the case where the * item pointer is a lossy page pointer). * * If all entry streams are exhausted, sets key->isFinished to TRUE. diff --git a/src/backend/access/heap/heapam.c b/src/backend/access/heap/heapam.c index a771ccb..9f86af4 100644 --- a/src/backend/access/heap/heapam.c +++ b/src/backend/access/heap/heapam.c @@ -3020,7 +3020,7 @@ simple_heap_delete(Relation relation, ItemPointer tid) * * On success, the header fields of *newtup are updated to match the new * stored tuple; in particular, newtup->t_self is set to the TID where the - * new tuple was inserted, and its HEAP_ONLY_TUPLE flag is set iff a HOT + * new tuple was inserted, and its HEAP_ONLY_TUPLE flag is set if and only if a HOT * update was done. However, any TOAST changes in the new tuple's * data are not reflected into *newtup. * @@ -4943,7 +4943,7 @@ l5: else { /* - * Can get here iff the locking/updating transaction was running when + * Can get here if and only if the locking/updating transaction was running when * the infomask was extracted from the tuple, but finished before * TransactionIdIsInProgress got to run. Deal with it as if there was * no locker at all in the first place. diff --git a/src/backend/access/nbtree/nbtsort.c b/src/backend/access/nbtree/nbtsort.c index 9ddc275..4565148 100644 --- a/src/backend/access/nbtree/nbtsort.c +++ b/src/backend/access/nbtree/nbtsort.c @@ -215,7 +215,7 @@ _bt_leafbuild(BTSpool *btspool, BTSpool *btspool2) wstate.index = btspool->index; /* - * We need to log index creation in WAL iff WAL archiving/streaming is + * We need to log index creation in WAL if and only if WAL archiving/streaming is * enabled UNLESS the index isn't WAL-logged anyway. */ wstate.btws_use_wal = XLogIsNeeded() && RelationNeedsWAL(wstate.index); diff --git a/src/backend/access/transam/README b/src/backend/access/transam/README index 3a32471..7caaee6 100644 --- a/src/backend/access/transam/README +++ b/src/backend/access/transam/README @@ -438,7 +438,7 @@ critical section.) 4. Mark the shared buffer(s) as dirty with MarkBufferDirty(). (This must happen before the WAL record is inserted; see notes in SyncOneBuffer().) Note that marking a buffer dirty with MarkBufferDirty() should only -happen iff you write a WAL record; see Writing Hints below. +happen if and only if you write a WAL record; see Writing Hints below. 5. If the relation requires WAL-logging, build a WAL log record and pass it to XLogInsert(); then update the page's LSN using the returned XLOG @@ -590,7 +590,7 @@ Writing Hints ------------- In some cases, we write additional information to data blocks without -writing a preceding WAL record. This should only happen iff the data can +writing a preceding WAL record. This should only happen if and only if the data can be reconstructed later following a crash and the action is simply a way of optimising for performance. When a hint is written we use MarkBufferDirtyHint() to mark the block dirty. diff --git a/src/backend/access/transam/transam.c b/src/backend/access/transam/transam.c index 8965319..46aeb58 100644 --- a/src/backend/access/transam/transam.c +++ b/src/backend/access/transam/transam.c @@ -116,7 +116,7 @@ TransactionLogFetch(TransactionId transactionId) /* * TransactionIdDidCommit - * True iff transaction associated with the identifier did commit. + * True if and only if transaction associated with the identifier did commit. * * Note: * Assumes transaction identifier is valid. @@ -172,7 +172,7 @@ TransactionIdDidCommit(TransactionId transactionId) /* * TransactionIdDidAbort - * True iff transaction associated with the identifier did abort. + * True if and only if transaction associated with the identifier did abort. * * Note: * Assumes transaction identifier is valid. @@ -221,7 +221,7 @@ TransactionIdDidAbort(TransactionId transactionId) /* * TransactionIdIsKnownCompleted - * True iff transaction associated with the identifier is currently + * True if and only if transaction associated with the identifier is currently * known to have either committed or aborted. * * This does NOT look into pg_clog but merely probes our local cache diff --git a/src/backend/access/transam/twophase.c b/src/backend/access/transam/twophase.c index 66dbf58..9b62ccb 100644 --- a/src/backend/access/transam/twophase.c +++ b/src/backend/access/transam/twophase.c @@ -509,7 +509,7 @@ RemoveGXact(GlobalTransaction gxact) /* * TransactionIdIsPrepared - * True iff transaction associated with the identifier is prepared + * True if and only if transaction associated with the identifier is prepared * for two-phase commit * * Note: only gxacts marked "valid" are considered; but notice we do not diff --git a/src/backend/catalog/catalog.c b/src/backend/catalog/catalog.c index e3002eb..08eddca 100644 --- a/src/backend/catalog/catalog.c +++ b/src/backend/catalog/catalog.c @@ -97,7 +97,7 @@ GetDatabasePath(Oid dbNode, Oid spcNode) /* * IsSystemRelation - * True iff the relation is either a system catalog or toast table. + * True if and only if the relation is either a system catalog or toast table. * By a system catalog, we mean one that created in the pg_catalog schema * during initdb. User-created relations in pg_catalog don't count as * system catalogs. @@ -127,7 +127,7 @@ IsSystemClass(Oid relid, Form_pg_class reltuple) /* * IsCatalogRelation - * True iff the relation is a system catalog, or the toast table for + * True if and only if the relation is a system catalog, or the toast table for * a system catalog. By a system catalog, we mean one that created * in the pg_catalog schema during initdb. As with IsSystemRelation(), * user-created relations in pg_catalog don't count as system catalogs. @@ -144,7 +144,7 @@ IsCatalogRelation(Relation relation) /* * IsCatalogClass - * True iff the relation is a system catalog relation. + * True if and only if the relation is a system catalog relation. * * Check IsCatalogRelation() for details. */ @@ -178,7 +178,7 @@ IsCatalogClass(Oid relid, Form_pg_class reltuple) /* * IsToastRelation - * True iff relation is a TOAST support relation (or index). + * True if and only if relation is a TOAST support relation (or index). */ bool IsToastRelation(Relation relation) @@ -202,7 +202,7 @@ IsToastClass(Form_pg_class reltuple) /* * IsSystemNamespace - * True iff namespace is pg_catalog. + * True if and only if namespace is pg_catalog. * * NOTE: the reason this isn't a macro is to avoid having to include * catalog/pg_namespace.h in a lot of places. @@ -215,7 +215,7 @@ IsSystemNamespace(Oid namespaceId) /* * IsToastNamespace - * True iff namespace is pg_toast or my temporary-toast-table namespace. + * True if and only if namespace is pg_toast or my temporary-toast-table namespace. * * Note: this will return false for temporary-toast-table namespaces belonging * to other backends. Those are treated the same as other backends' regular @@ -231,7 +231,7 @@ IsToastNamespace(Oid namespaceId) /* * IsReservedName - * True iff name starts with the pg_ prefix. + * True if and only if name starts with the pg_ prefix. * * For some classes of objects, the prefix pg_ is reserved for * system objects only. As of 8.0, this is only true for diff --git a/src/backend/commands/analyze.c b/src/backend/commands/analyze.c index e7fcb55..9530ba5 100644 --- a/src/backend/commands/analyze.c +++ b/src/backend/commands/analyze.c @@ -355,7 +355,7 @@ do_analyze_rel(Relation onerel, VacuumStmt *vacstmt, save_sec_context | SECURITY_RESTRICTED_OPERATION); save_nestlevel = NewGUCNestLevel(); - /* measure elapsed time iff autovacuum logging requires it */ + /* measure elapsed time if and only if autovacuum logging requires it */ if (IsAutoVacuumWorkerProcess() && Log_autovacuum_min_duration >= 0) { pg_rusage_init(&ru0); diff --git a/src/backend/commands/cluster.c b/src/backend/commands/cluster.c index 14a5e5a..98de792 100644 --- a/src/backend/commands/cluster.c +++ b/src/backend/commands/cluster.c @@ -803,7 +803,7 @@ copy_heap_data(Oid OIDNewHeap, Oid OIDOldHeap, Oid OIDOldIndex, bool verbose, LockRelationOid(OldHeap->rd_rel->reltoastrelid, AccessExclusiveLock); /* - * We need to log the copied data in WAL iff WAL archiving/streaming is + * We need to log the copied data in WAL if and only if WAL archiving/streaming is * enabled AND it's a WAL-logged rel. */ use_wal = XLogIsNeeded() && RelationNeedsWAL(NewHeap); diff --git a/src/backend/commands/indexcmds.c b/src/backend/commands/indexcmds.c index 4259c47..ef15160 100644 --- a/src/backend/commands/indexcmds.c +++ b/src/backend/commands/indexcmds.c @@ -1052,7 +1052,7 @@ ComputeIndexAttrs(IndexInfo *indexInfo, attcollation = get_collation_oid(attribute->collation, false); /* - * Check we have a collation iff it's a collatable type. The only + * Check we have a collation if and only if it's a collatable type. The only * expected failures here are (1) COLLATE applied to a noncollatable * type, or (2) index expression had an unresolved collation. But we * might as well code this to be a complete consistency check. diff --git a/src/backend/commands/tablecmds.c b/src/backend/commands/tablecmds.c index 08b037e..0c59480 100644 --- a/src/backend/commands/tablecmds.c +++ b/src/backend/commands/tablecmds.c @@ -9136,7 +9136,7 @@ copy_relation_data(SMgrRelation src, SMgrRelation dst, page = (Page) buf; /* - * We need to log the copied data in WAL iff WAL archiving/streaming is + * We need to log the copied data in WAL if and only if WAL archiving/streaming is * enabled AND it's a permanent relation. */ use_wal = XLogIsNeeded() && relpersistence == RELPERSISTENCE_PERMANENT; diff --git a/src/backend/commands/vacuumlazy.c b/src/backend/commands/vacuumlazy.c index 75e5f15..d8faa79 100644 --- a/src/backend/commands/vacuumlazy.c +++ b/src/backend/commands/vacuumlazy.c @@ -190,7 +190,7 @@ lazy_vacuum_rel(Relation onerel, VacuumStmt *vacstmt, TransactionId new_frozen_xid; MultiXactId new_min_multi; - /* measure elapsed time iff autovacuum logging requires it */ + /* measure elapsed time if and only if autovacuum logging requires it */ if (IsAutoVacuumWorkerProcess() && Log_autovacuum_min_duration >= 0) { pg_rusage_init(&ru0); diff --git a/src/backend/executor/execMain.c b/src/backend/executor/execMain.c index 6b5f198..493a8c9 100644 --- a/src/backend/executor/execMain.c +++ b/src/backend/executor/execMain.c @@ -1250,7 +1250,7 @@ ExecGetTriggerResultRel(EState *estate, Oid relid) * ExecContextForcesOids * * This is pretty grotty: when doing INSERT, UPDATE, or CREATE TABLE AS, - * we need to ensure that result tuples have space for an OID iff they are + * we need to ensure that result tuples have space for an OID if and only if they are * going to be stored into a relation that has OIDs. In other contexts * we are free to choose whether to leave space for OIDs in result tuples * (we generally don't want to, but we do if a physical-tlist optimization diff --git a/src/backend/executor/execQual.c b/src/backend/executor/execQual.c index 0eba025..d8af8ba 100644 --- a/src/backend/executor/execQual.c +++ b/src/backend/executor/execQual.c @@ -5103,7 +5103,7 @@ ExecPrepareExpr(Expr *node, EState *estate) * ExecQual * * Evaluates a conjunctive boolean expression (qual list) and - * returns true iff none of the subexpressions are false. + * returns true if and only if none of the subexpressions are false. * (We also return true if the list is empty.) * * If some of the subexpressions yield NULL but none yield FALSE, diff --git a/src/backend/executor/functions.c b/src/backend/executor/functions.c index 9c17970..d7d93ec 100644 --- a/src/backend/executor/functions.c +++ b/src/backend/executor/functions.c @@ -1500,7 +1500,7 @@ ShutdownSQLFunction(Datum arg) * (Obviously the caller must pass a parsetree that is okay to modify when * using this flag.) Note that this flag does not affect whether the tlist is * considered to be a legal match to the result type, only how we react to - * allowed not-exact-match cases. *modifyTargetList will be set true iff + * allowed not-exact-match cases. *modifyTargetList will be set true if and only if * we had to make any "dangerous" changes that could modify the semantics of * the statement. If it is set true, the caller should not use the modified * statement, but for simplicity we apply the changes anyway. diff --git a/src/backend/executor/nodeAppend.c b/src/backend/executor/nodeAppend.c index 6185c1d..6fbce8a 100644 --- a/src/backend/executor/nodeAppend.c +++ b/src/backend/executor/nodeAppend.c @@ -68,7 +68,7 @@ static bool exec_append_initialize_next(AppendState *appendstate); * * Sets up the append state node for the "next" scan. * - * Returns t iff there is a "next" scan to process. + * Returns t if and only if there is a "next" scan to process. * ---------------------------------------------------------------- */ static bool diff --git a/src/backend/libpq/auth.c b/src/backend/libpq/auth.c index 9d0c389..75e593f 100644 --- a/src/backend/libpq/auth.c +++ b/src/backend/libpq/auth.c @@ -1388,7 +1388,7 @@ interpret_ident_response(const char *ident_response, * * IP addresses and port numbers are in network byte order. * - * But iff we're unable to get the information from ident, return false. + * But if and only if we're unable to get the information from ident, return false. */ static int ident_inet(hbaPort *port) diff --git a/src/backend/nodes/list.c b/src/backend/nodes/list.c index aebc5b6..af7cbe5 100644 --- a/src/backend/nodes/list.c +++ b/src/backend/nodes/list.c @@ -439,7 +439,7 @@ list_nth_oid(const List *list, int n) } /* - * Return true iff 'datum' is a member of the list. Equality is + * Return true if and only if 'datum' is a member of the list. Equality is * determined via equal(), so callers should ensure that they pass a * Node as 'datum'. */ @@ -461,7 +461,7 @@ list_member(const List *list, const void *datum) } /* - * Return true iff 'datum' is a member of the list. Equality is + * Return true if and only if 'datum' is a member of the list. Equality is * determined by using simple pointer comparison. */ bool @@ -482,7 +482,7 @@ list_member_ptr(const List *list, const void *datum) } /* - * Return true iff the integer 'datum' is a member of the list. + * Return true if and only if the integer 'datum' is a member of the list. */ bool list_member_int(const List *list, int datum) @@ -502,7 +502,7 @@ list_member_int(const List *list, int datum) } /* - * Return true iff the OID 'datum' is a member of the list. + * Return true if and only if the OID 'datum' is a member of the list. */ bool list_member_oid(const List *list, Oid datum) @@ -523,7 +523,7 @@ list_member_oid(const List *list, Oid datum) /* * Delete 'cell' from 'list'; 'prev' is the previous element to 'cell' - * in 'list', if any (i.e. prev == NULL iff list->head == cell) + * in 'list', if any (i.e. prev == NULL if and only if list->head == cell) * * The cell is pfree'd, as is the List header if this was the last member. */ diff --git a/src/backend/nodes/nodeFuncs.c b/src/backend/nodes/nodeFuncs.c index 123f2a6..03c4fdf 100644 --- a/src/backend/nodes/nodeFuncs.c +++ b/src/backend/nodes/nodeFuncs.c @@ -1545,7 +1545,7 @@ leftmostLoc(int loc1, int loc2) * "true" to abort the walk and immediately return "true" to the top-level * caller. This can be used to short-circuit the traversal if the walker * has found what it came for. "false" is returned to the top-level caller - * iff no invocation of the walker returned "true". + * if and only if no invocation of the walker returned "true". * * The node types handled by expression_tree_walker include all those * normally found in target lists and qualifier clauses during the planning diff --git a/src/backend/optimizer/path/indxpath.c b/src/backend/optimizer/path/indxpath.c index a912174..52f4236 100644 --- a/src/backend/optimizer/path/indxpath.c +++ b/src/backend/optimizer/path/indxpath.c @@ -1085,7 +1085,7 @@ build_paths_for_OR(PlannerInfo *root, RelOptInfo *rel, * test whether the added clauses are sufficient to imply the * predicate. If so, we can use the index in the current context. * - * We set useful_predicate to true iff the predicate was proven using + * We set useful_predicate to true if and only if the predicate was proven using * the current set of clauses. This is needed to prevent matching a * predOK index to an arm of an OR, which would be a legal but * pointlessly inefficient plan. (A better plan will be generated by diff --git a/src/backend/optimizer/path/pathkeys.c b/src/backend/optimizer/path/pathkeys.c index 9179c61..05af96c 100644 --- a/src/backend/optimizer/path/pathkeys.c +++ b/src/backend/optimizer/path/pathkeys.c @@ -1432,7 +1432,7 @@ right_merge_direction(PlannerInfo *root, PathKey *pathkey) { /* * Found a matching query sort column. Prefer this pathkey's - * direction iff it matches. Note that we ignore pk_nulls_first, + * direction if and only if it matches. Note that we ignore pk_nulls_first, * which means that a sort might be needed anyway ... but we still * want to prefer only one of the two possible directions, and we * might as well use this one. diff --git a/src/backend/optimizer/util/clauses.c b/src/backend/optimizer/util/clauses.c index 201529b..5929641 100644 --- a/src/backend/optimizer/util/clauses.c +++ b/src/backend/optimizer/util/clauses.c @@ -218,7 +218,7 @@ get_rightop(const Expr *clause) /* * not_clause * - * Returns t iff this is a 'not' clause: (NOT expr). + * Returns t if and only if this is a 'not' clause: (NOT expr). */ bool not_clause(Node *clause) @@ -262,7 +262,7 @@ get_notclausearg(Expr *notclause) /* * or_clause * - * Returns t iff the clause is an 'or' clause: (OR { expr }). + * Returns t if and only if the clause is an 'or' clause: (OR { expr }). */ bool or_clause(Node *clause) @@ -296,7 +296,7 @@ make_orclause(List *orclauses) /* * and_clause * - * Returns t iff its argument is an 'and' clause: (AND { expr }). + * Returns t if and only if its argument is an 'and' clause: (AND { expr }). */ bool and_clause(Node *clause) diff --git a/src/backend/optimizer/util/predtest.c b/src/backend/optimizer/util/predtest.c index eadd2d5..b6a995a 100644 --- a/src/backend/optimizer/util/predtest.c +++ b/src/backend/optimizer/util/predtest.c @@ -214,16 +214,16 @@ predicate_refuted_by(List *predicate_list, List *restrictinfo_list) * predicate clauses. * * The logic followed here is ("=>" means "implies"): - * atom A => atom B iff: predicate_implied_by_simple_clause says so - * atom A => AND-expr B iff: A => each of B's components - * atom A => OR-expr B iff: A => any of B's components - * AND-expr A => atom B iff: any of A's components => B - * AND-expr A => AND-expr B iff: A => each of B's components - * AND-expr A => OR-expr B iff: A => any of B's components, + * atom A => atom B if and only if: predicate_implied_by_simple_clause says so + * atom A => AND-expr B if and only if: A => each of B's components + * atom A => OR-expr B if and only if: A => any of B's components + * AND-expr A => atom B if and only if: any of A's components => B + * AND-expr A => AND-expr B if and only if: A => each of B's components + * AND-expr A => OR-expr B if and only if: A => any of B's components, * *or* any of A's components => B - * OR-expr A => atom B iff: each of A's components => B - * OR-expr A => AND-expr B iff: A => each of B's components - * OR-expr A => OR-expr B iff: each of A's components => any of B's + * OR-expr A => atom B if and only if: each of A's components => B + * OR-expr A => AND-expr B if and only if: A => each of B's components + * OR-expr A => OR-expr B if and only if: each of A's components => any of B's * * An "atom" is anything other than an AND or OR node. Notice that we don't * have any special logic to handle NOT nodes; these should have been pushed @@ -449,16 +449,16 @@ predicate_implied_by_recurse(Node *clause, Node *predicate) * predicate clauses. * * The logic followed here is ("R=>" means "refutes"): - * atom A R=> atom B iff: predicate_refuted_by_simple_clause says so - * atom A R=> AND-expr B iff: A R=> any of B's components - * atom A R=> OR-expr B iff: A R=> each of B's components - * AND-expr A R=> atom B iff: any of A's components R=> B - * AND-expr A R=> AND-expr B iff: A R=> any of B's components, + * atom A R=> atom B if and only if: predicate_refuted_by_simple_clause says so + * atom A R=> AND-expr B if and only if: A R=> any of B's components + * atom A R=> OR-expr B if and only if: A R=> each of B's components + * AND-expr A R=> atom B if and only if: any of A's components R=> B + * AND-expr A R=> AND-expr B if and only if: A R=> any of B's components, * *or* any of A's components R=> B - * AND-expr A R=> OR-expr B iff: A R=> each of B's components - * OR-expr A R=> atom B iff: each of A's components R=> B - * OR-expr A R=> AND-expr B iff: each of A's components R=> any of B's - * OR-expr A R=> OR-expr B iff: A R=> each of B's components + * AND-expr A R=> OR-expr B if and only if: A R=> each of B's components + * OR-expr A R=> atom B if and only if: each of A's components R=> B + * OR-expr A R=> AND-expr B if and only if: each of A's components R=> any of B's + * OR-expr A R=> OR-expr B if and only if: A R=> each of B's components * * In addition, if the predicate is a NOT-clause then we can use * A R=> NOT B if: A => B diff --git a/src/backend/optimizer/util/relnode.c b/src/backend/optimizer/util/relnode.c index 8ae8f55..c77477b 100644 --- a/src/backend/optimizer/util/relnode.c +++ b/src/backend/optimizer/util/relnode.c @@ -99,7 +99,7 @@ build_simple_rel(PlannerInfo *root, int relid, RelOptKind reloptkind) rel->relids = bms_make_singleton(relid); rel->rows = 0; rel->width = 0; - /* cheap startup cost is interesting iff not all tuples to be retrieved */ + /* cheap startup cost is interesting if and only if not all tuples to be retrieved */ rel->consider_startup = (root->tuple_fraction > 0); rel->reltargetlist = NIL; rel->pathlist = NIL; @@ -358,7 +358,7 @@ build_join_rel(PlannerInfo *root, joinrel->relids = bms_copy(joinrelids); joinrel->rows = 0; joinrel->width = 0; - /* cheap startup cost is interesting iff not all tuples to be retrieved */ + /* cheap startup cost is interesting if and only if not all tuples to be retrieved */ joinrel->consider_startup = (root->tuple_fraction > 0); joinrel->reltargetlist = NIL; joinrel->pathlist = NIL; diff --git a/src/backend/optimizer/util/restrictinfo.c b/src/backend/optimizer/util/restrictinfo.c index 62de590..617b1dd 100644 --- a/src/backend/optimizer/util/restrictinfo.c +++ b/src/backend/optimizer/util/restrictinfo.c @@ -318,7 +318,7 @@ make_sub_restrictinfos(Expr *clause, /* * restriction_is_or_clause * - * Returns t iff the restrictinfo node contains an 'or' clause. + * Returns t if and only if the restrictinfo node contains an 'or' clause. */ bool restriction_is_or_clause(RestrictInfo *restrictinfo) diff --git a/src/backend/parser/parse_clause.c b/src/backend/parser/parse_clause.c index aa704bb..0a8be82 100644 --- a/src/backend/parser/parse_clause.c +++ b/src/backend/parser/parse_clause.c @@ -240,7 +240,7 @@ interpretInhOption(InhOption inhOpt) } /* - * Given a relation-options list (of DefElems), return true iff the specified + * Given a relation-options list (of DefElems), return true if and only if the specified * table/result set should be created with OIDs. This needs to be done after * parsing the query string because the return value can depend upon the * default_with_oids GUC var. diff --git a/src/backend/parser/parse_coerce.c b/src/backend/parser/parse_coerce.c index 854d723..956f128 100644 --- a/src/backend/parser/parse_coerce.c +++ b/src/backend/parser/parse_coerce.c @@ -1406,7 +1406,7 @@ check_generic_type_consistency(Oid *actual_arg_types, { if (array_typeid == ANYARRAYOID) { - /* Special case for ANYARRAY input: okay iff no ANYELEMENT */ + /* Special case for ANYARRAY input: okay if and only if no ANYELEMENT */ if (have_anyelement) return false; return true; @@ -1648,7 +1648,7 @@ enforce_generic_type_consistency(Oid *actual_arg_types, { if (array_typeid == ANYARRAYOID && !have_anyelement) { - /* Special case for ANYARRAY input: okay iff no ANYELEMENT */ + /* Special case for ANYARRAY input: okay if and only if no ANYELEMENT */ array_typelem = ANYELEMENTOID; } else @@ -1685,7 +1685,7 @@ enforce_generic_type_consistency(Oid *actual_arg_types, { if (range_typeid == ANYRANGEOID && !have_anyelement) { - /* Special case for ANYRANGE input: okay iff no ANYELEMENT */ + /* Special case for ANYRANGE input: okay if and only if no ANYELEMENT */ range_typelem = ANYELEMENTOID; } else diff --git a/src/backend/parser/parse_relation.c b/src/backend/parser/parse_relation.c index 8760952..f33a5a3 100644 --- a/src/backend/parser/parse_relation.c +++ b/src/backend/parser/parse_relation.c @@ -2894,7 +2894,7 @@ errorMissingColumn(ParseState *pstate, /* - * Examine a fully-parsed query, and return TRUE iff any relation underlying + * Examine a fully-parsed query, and return TRUE if and only if any relation underlying * the query is a temporary relation (table, view, or materialized view). */ bool diff --git a/src/backend/storage/lmgr/lmgr.c b/src/backend/storage/lmgr/lmgr.c index 4c61a6f..c49e872 100644 --- a/src/backend/storage/lmgr/lmgr.c +++ b/src/backend/storage/lmgr/lmgr.c @@ -96,7 +96,7 @@ LockRelationOid(Oid relid, LOCKMODE lockmode) * ConditionalLockRelationOid * * As above, but only lock if we can get the lock without blocking. - * Returns TRUE iff the lock was acquired. + * Returns TRUE if and only if the lock was acquired. * * NOTE: we do not currently need conditional versions of all the * LockXXX routines in this file, but they could easily be added if needed. @@ -345,7 +345,7 @@ LockPage(Relation relation, BlockNumber blkno, LOCKMODE lockmode) * ConditionalLockPage * * As above, but only lock if we can get the lock without blocking. - * Returns TRUE iff the lock was acquired. + * Returns TRUE if and only if the lock was acquired. */ bool ConditionalLockPage(Relation relation, BlockNumber blkno, LOCKMODE lockmode) @@ -401,7 +401,7 @@ LockTuple(Relation relation, ItemPointer tid, LOCKMODE lockmode) * ConditionalLockTuple * * As above, but only lock if we can get the lock without blocking. - * Returns TRUE iff the lock was acquired. + * Returns TRUE if and only if the lock was acquired. */ bool ConditionalLockTuple(Relation relation, ItemPointer tid, LOCKMODE lockmode) diff --git a/src/backend/tcop/postgres.c b/src/backend/tcop/postgres.c index b7612b9..5c90fcf 100644 --- a/src/backend/tcop/postgres.c +++ b/src/backend/tcop/postgres.c @@ -903,7 +903,7 @@ exec_simple_query(const char *query_string) MemoryContextSwitchTo(oldcontext); /* - * We'll tell PortalRun it's a top-level command iff there's exactly one + * We'll tell PortalRun it's a top-level command if and only if there's exactly one * raw parsetree. If more than one, it's effectively a transaction block * and we want PreventTransactionChain to reject unsafe commands. (Note: * we're assuming that query rewrite cannot add commands that are diff --git a/src/backend/tcop/pquery.c b/src/backend/tcop/pquery.c index f85bd03..805dd4f 100644 --- a/src/backend/tcop/pquery.c +++ b/src/backend/tcop/pquery.c @@ -807,7 +807,7 @@ PortalRun(Portal portal, long count, bool isTopLevel, portal->status = PORTAL_READY; /* - * Since it's a forward fetch, say DONE iff atEnd is now true. + * Since it's a forward fetch, say DONE if and only if atEnd is now true. */ result = portal->atEnd; break; diff --git a/src/backend/utils/adt/acl.c b/src/backend/utils/adt/acl.c index 10729d5..363678f 100644 --- a/src/backend/utils/adt/acl.c +++ b/src/backend/utils/adt/acl.c @@ -650,7 +650,7 @@ aclitemout(PG_FUNCTION_ARGS) /* * aclitem_match - * Two AclItems are considered to match iff they have the same + * Two AclItems are considered to match if and only if they have the same * grantee and grantor; the privileges are ignored. */ static bool diff --git a/src/backend/utils/adt/array_selfuncs.c b/src/backend/utils/adt/array_selfuncs.c index e35edf8..3206c33 100644 --- a/src/backend/utils/adt/array_selfuncs.c +++ b/src/backend/utils/adt/array_selfuncs.c @@ -1021,7 +1021,7 @@ calc_hist(const float4 *hist, int nhist, int n) * * Imagine matrix M of size (n + 1) x (m + 1). Element M[i,j] denotes the * probability that exactly j of first i events occur. Obviously M[0,0] = 1. - * For any constant j, each increment of i increases the probability iff the + * For any constant j, each increment of i increases the probability if and only if the * event occurs. So, by the law of total probability: * M[i,j] = M[i - 1, j] * (1 - p[i]) + M[i - 1, j - 1] * p[i] * for i > 0, j > 0. diff --git a/src/backend/utils/adt/array_userfuncs.c b/src/backend/utils/adt/array_userfuncs.c index c62e3fb..412d471 100644 --- a/src/backend/utils/adt/array_userfuncs.c +++ b/src/backend/utils/adt/array_userfuncs.c @@ -522,7 +522,7 @@ array_agg_finalfn(PG_FUNCTION_ARGS) * possible for users to create NULL constants of type internal. */ if (PG_ARGISNULL(0)) - PG_RETURN_NULL(); /* returns null iff no input values */ + PG_RETURN_NULL(); /* returns null if and only if no input values */ /* cannot be called directly because of internal-type argument */ Assert(AggCheckCallContext(fcinfo, NULL)); diff --git a/src/backend/utils/adt/arrayfuncs.c b/src/backend/utils/adt/arrayfuncs.c index 311d0c2..adafd4a 100644 --- a/src/backend/utils/adt/arrayfuncs.c +++ b/src/backend/utils/adt/arrayfuncs.c @@ -651,7 +651,7 @@ ArrayCount(const char *str, int *dim, char typdelim) * Outputs: * values[]: filled with converted data values. * nulls[]: filled with is-null markers. - * *hasnulls: set TRUE iff there are any null elements. + * *hasnulls: set TRUE if and only if there are any null elements. * *nbytes: set to total size of data area needed (including alignment * padding but not including array header overhead). * @@ -1377,7 +1377,7 @@ array_recv(PG_FUNCTION_ARGS) * Outputs: * values[]: filled with converted data values. * nulls[]: filled with is-null markers. - * *hasnulls: set TRUE iff there are any null elements. + * *hasnulls: set TRUE if and only if there are any null elements. * *nbytes: set to total size of data area needed (including alignment * padding but not including array header overhead). * diff --git a/src/backend/utils/adt/int.c b/src/backend/utils/adt/int.c index 669355e..bcf7c17 100644 --- a/src/backend/utils/adt/int.c +++ b/src/backend/utils/adt/int.c @@ -378,12 +378,12 @@ bool_int4(PG_FUNCTION_ARGS) */ /* - * inteq - returns 1 iff arg1 == arg2 - * intne - returns 1 iff arg1 != arg2 - * intlt - returns 1 iff arg1 < arg2 - * intle - returns 1 iff arg1 <= arg2 - * intgt - returns 1 iff arg1 > arg2 - * intge - returns 1 iff arg1 >= arg2 + * inteq - returns 1 if and only if arg1 == arg2 + * intne - returns 1 if and only if arg1 != arg2 + * intlt - returns 1 if and only if arg1 < arg2 + * intle - returns 1 if and only if arg1 <= arg2 + * intgt - returns 1 if and only if arg1 > arg2 + * intge - returns 1 if and only if arg1 >= arg2 */ Datum diff --git a/src/backend/utils/adt/like_match.c b/src/backend/utils/adt/like_match.c index 1e5e00a..9cb14da 100644 --- a/src/backend/utils/adt/like_match.c +++ b/src/backend/utils/adt/like_match.c @@ -220,7 +220,7 @@ MatchText(char *t, int tlen, char *p, int plen, return LIKE_FALSE; /* end of pattern, but not of text */ /* - * End of text, but perhaps not of pattern. Match iff the remaining + * End of text, but perhaps not of pattern. Match if and only if the remaining * pattern can match a zero-length string, ie, it's zero or more %'s. */ while (plen > 0 && *p == '%') diff --git a/src/backend/utils/adt/mac.c b/src/backend/utils/adt/mac.c index aa9993f..732a4ac 100644 --- a/src/backend/utils/adt/mac.c +++ b/src/backend/utils/adt/mac.c @@ -40,7 +40,7 @@ macaddr_in(PG_FUNCTION_ARGS) char junk[2]; int count; - /* %1s matches iff there is trailing non-whitespace garbage */ + /* %1s matches if and only if there is trailing non-whitespace garbage */ count = sscanf(str, "%x:%x:%x:%x:%x:%x%1s", &a, &b, &c, &d, &e, &f, junk); diff --git a/src/backend/utils/adt/nabstime.c b/src/backend/utils/adt/nabstime.c index 74d24aa..61ee24d 100644 --- a/src/backend/utils/adt/nabstime.c +++ b/src/backend/utils/adt/nabstime.c @@ -959,7 +959,7 @@ timemi(PG_FUNCTION_ARGS) /* - * intinterval - returns true iff absolute date is in the tinterval + * intinterval - returns true if and only if absolute date is in the tinterval */ Datum intinterval(PG_FUNCTION_ARGS) @@ -1107,7 +1107,7 @@ btreltimecmp(PG_FUNCTION_ARGS) /* - * tintervalsame - returns true iff tinterval i1 is same as tinterval i2 + * tintervalsame - returns true if and only if tinterval i1 is same as tinterval i2 * Check begin and end time. */ Datum @@ -1256,17 +1256,17 @@ bttintervalcmp(PG_FUNCTION_ARGS) /* - * tintervalleneq - returns true iff length of tinterval i is equal to + * tintervalleneq - returns true if and only if length of tinterval i is equal to * reltime t - * tintervallenne - returns true iff length of tinterval i is not equal + * tintervallenne - returns true if and only if length of tinterval i is not equal * to reltime t - * tintervallenlt - returns true iff length of tinterval i is less than + * tintervallenlt - returns true if and only if length of tinterval i is less than * reltime t - * tintervallengt - returns true iff length of tinterval i is greater + * tintervallengt - returns true if and only if length of tinterval i is greater * than reltime t - * tintervallenle - returns true iff length of tinterval i is less or + * tintervallenle - returns true if and only if length of tinterval i is less or * equal than reltime t - * tintervallenge - returns true iff length of tinterval i is greater or + * tintervallenge - returns true if and only if length of tinterval i is greater or * equal than reltime t */ Datum @@ -1354,7 +1354,7 @@ tintervallenge(PG_FUNCTION_ARGS) } /* - * tintervalct - returns true iff tinterval i1 contains tinterval i2 + * tintervalct - returns true if and only if tinterval i1 contains tinterval i2 */ Datum tintervalct(PG_FUNCTION_ARGS) @@ -1375,7 +1375,7 @@ tintervalct(PG_FUNCTION_ARGS) } /* - * tintervalov - returns true iff tinterval i1 (partially) overlaps i2 + * tintervalov - returns true if and only if tinterval i1 (partially) overlaps i2 */ Datum tintervalov(PG_FUNCTION_ARGS) diff --git a/src/backend/utils/adt/name.c b/src/backend/utils/adt/name.c index a851502..f9d0a9c 100644 --- a/src/backend/utils/adt/name.c +++ b/src/backend/utils/adt/name.c @@ -117,17 +117,17 @@ namesend(PG_FUNCTION_ARGS) *****************************************************************************/ /* - * nameeq - returns 1 iff arguments are equal - * namene - returns 1 iff arguments are not equal + * nameeq - returns 1 if and only if arguments are equal + * namene - returns 1 if and only if arguments are not equal * * BUGS: * Assumes that "xy\0\0a" should be equal to "xy\0b". * If not, can do the comparison backwards for efficiency. * - * namelt - returns 1 iff a < b - * namele - returns 1 iff a <= b - * namegt - returns 1 iff a > b - * namege - returns 1 iff a >= b + * namelt - returns 1 if and only if a < b + * namele - returns 1 if and only if a <= b + * namegt - returns 1 if and only if a > b + * namege - returns 1 if and only if a >= b * */ Datum diff --git a/src/backend/utils/adt/pg_locale.c b/src/backend/utils/adt/pg_locale.c index 0bf350a..a3d7860 100644 --- a/src/backend/utils/adt/pg_locale.c +++ b/src/backend/utils/adt/pg_locale.c @@ -1166,7 +1166,7 @@ pg_newlocale_from_collation(Oid collid) * * This has the same API as the standard wcstombs_l() function; in particular, * tolen is the maximum number of bytes to store at *to, and *from must be - * zero-terminated. The output will be zero-terminated iff there is room. + * zero-terminated. The output will be zero-terminated if and only if there is room. */ size_t wchar2char(char *to, const wchar_t *from, size_t tolen, pg_locale_t locale) @@ -1235,7 +1235,7 @@ wchar2char(char *to, const wchar_t *from, size_t tolen, pg_locale_t locale) * null-terminated; instead, the number of input bytes is specified as * fromlen. Also, we ereport() rather than returning -1 for invalid * input encoding. tolen is the maximum number of wchar_t's to store at *to. - * The output will be zero-terminated iff there is room. + * The output will be zero-terminated if and only if there is room. */ size_t char2wchar(wchar_t *to, size_t tolen, const char *from, size_t fromlen, diff --git a/src/backend/utils/adt/tsvector_op.c b/src/backend/utils/adt/tsvector_op.c index 8b71de2..69b3665 100644 --- a/src/backend/utils/adt/tsvector_op.c +++ b/src/backend/utils/adt/tsvector_op.c @@ -550,7 +550,7 @@ tsvector_concat(PG_FUNCTION_ARGS) /* * Compare two strings by tsvector rules. * - * if isPrefix = true then it returns zero value iff b has prefix a + * if isPrefix = true then it returns zero value if and only if b has prefix a */ int32 tsCompareString(char *a, int lena, char *b, int lenb, bool prefix) diff --git a/src/backend/utils/cache/catcache.c b/src/backend/utils/cache/catcache.c index d17b6b0..e19f6da 100644 --- a/src/backend/utils/cache/catcache.c +++ b/src/backend/utils/cache/catcache.c @@ -1508,7 +1508,7 @@ SearchCatCacheList(CatCache *cache, nkeys, cur_skey); - /* The list will be ordered iff we are doing an index scan */ + /* The list will be ordered if and only if we are doing an index scan */ ordered = (scandesc->irel != NULL); while (HeapTupleIsValid(ntp = systable_getnext(scandesc))) diff --git a/src/backend/utils/cache/lsyscache.c b/src/backend/utils/cache/lsyscache.c index a4ce716..30d30f5 100644 --- a/src/backend/utils/cache/lsyscache.c +++ b/src/backend/utils/cache/lsyscache.c @@ -51,7 +51,7 @@ get_attavgwidth_hook_type get_attavgwidth_hook = NULL; /* * op_in_opfamily * - * Return t iff operator 'opno' is in operator family 'opfamily'. + * Return t if and only if operator 'opno' is in operator family 'opfamily'. * * This function only considers search operators, not ordering operators. */ diff --git a/src/backend/utils/mb/mbutils.c b/src/backend/utils/mb/mbutils.c index 08440e9..4aa68bd 100644 --- a/src/backend/utils/mb/mbutils.c +++ b/src/backend/utils/mb/mbutils.c @@ -1022,7 +1022,7 @@ GetMessageEncoding(void) /* * Result is palloc'ed null-terminated utf16 string. The character length - * is also passed to utf16len if not null. Returns NULL iff failed. + * is also passed to utf16len if not null. Returns NULL if and only if failed. */ WCHAR * pgwin32_message_to_UTF16(const char *str, int len, int *utf16len) diff --git a/src/backend/utils/mmgr/aset.c b/src/backend/utils/mmgr/aset.c index 099200c..2e6696c 100644 --- a/src/backend/utils/mmgr/aset.c +++ b/src/backend/utils/mmgr/aset.c @@ -227,13 +227,13 @@ typedef struct AllocChunkData /* * AllocPointerIsValid - * True iff pointer is valid allocation pointer. + * True if and only if pointer is valid allocation pointer. */ #define AllocPointerIsValid(pointer) PointerIsValid(pointer) /* * AllocSetIsValid - * True iff set is valid allocation set. + * True if and only if set is valid allocation set. */ #define AllocSetIsValid(set) PointerIsValid(set) diff --git a/src/backend/utils/time/tqual.c b/src/backend/utils/time/tqual.c index f626755..6b48e21 100644 --- a/src/backend/utils/time/tqual.c +++ b/src/backend/utils/time/tqual.c @@ -137,7 +137,7 @@ HeapTupleSetHintBits(HeapTupleHeader tuple, Buffer buffer, /* * HeapTupleSatisfiesSelf - * True iff heap tuple is valid "for itself". + * True if and only if heap tuple is valid "for itself". * * Here, we consider the effects of: * all committed transactions (as of the current instant) @@ -336,7 +336,7 @@ HeapTupleSatisfiesAny(HeapTuple htup, Snapshot snapshot, Buffer buffer) /* * HeapTupleSatisfiesToast - * True iff heap tuple is valid as a TOAST row. + * True if and only if heap tuple is valid as a TOAST row. * * This is a simplified version that only checks for VACUUM moving conditions. * It's appropriate for TOAST usage because TOAST really doesn't want to do @@ -687,7 +687,7 @@ HeapTupleSatisfiesUpdate(HeapTuple htup, CommandId curcid, /* * HeapTupleSatisfiesDirty - * True iff heap tuple is valid including effects of open transactions. + * True if and only if heap tuple is valid including effects of open transactions. * * Here, we consider the effects of: * all committed and in-progress transactions (as of the current instant) @@ -886,7 +886,7 @@ HeapTupleSatisfiesDirty(HeapTuple htup, Snapshot snapshot, /* * HeapTupleSatisfiesMVCC - * True iff heap tuple is valid for the given MVCC snapshot. + * True if and only if heap tuple is valid for the given MVCC snapshot. * * Here, we consider the effects of: * all transactions committed as of the time of the given snapshot diff --git a/src/bin/pg_dump/parallel.c b/src/bin/pg_dump/parallel.c index 6f2634b..a82c24d 100644 --- a/src/bin/pg_dump/parallel.c +++ b/src/bin/pg_dump/parallel.c @@ -753,7 +753,7 @@ GetIdleWorker(ParallelState *pstate) } /* - * Return true iff every worker process is in the WRKR_TERMINATED state. + * Return true if and only if every worker process is in the WRKR_TERMINATED state. */ static bool HasEveryWorkerTerminated(ParallelState *pstate) @@ -767,7 +767,7 @@ HasEveryWorkerTerminated(ParallelState *pstate) } /* - * Return true iff every worker is in the WRKR_IDLE state. + * Return true if and only if every worker is in the WRKR_IDLE state. */ bool IsEveryWorkerIdle(ParallelState *pstate) diff --git a/src/bin/pg_dump/pg_dump.c b/src/bin/pg_dump/pg_dump.c index ebbb5b7..9107aa6 100644 --- a/src/bin/pg_dump/pg_dump.c +++ b/src/bin/pg_dump/pg_dump.c @@ -5249,7 +5249,7 @@ getIndexes(Archive *fout, TableInfo tblinfo[], int numTables) * If we found a constraint matching the index, create an * entry for it. * - * In a pre-7.3 database, we take this path iff the index was + * In a pre-7.3 database, we take this path if and only if the index was * marked indisprimary. */ constrinfo[j].dobj.objType = DO_CONSTRAINT; @@ -9350,7 +9350,7 @@ dumpProcLang(Archive *fout, ProcLangInfo *plang) * dump it. * * However, for a language that belongs to an extension, we must not use - * the shouldDumpProcLangs heuristic, but just dump the language iff we're + * the shouldDumpProcLangs heuristic, but just dump the language if and only if we're * told to (via dobj.dump). Generally the support functions will belong * to the same extension and so have the same dump flags ... if they * don't, this might not work terribly nicely. @@ -10094,7 +10094,7 @@ dumpCast(Archive *fout, CastInfo *cast) * Builtin meaning, the namespace name does not start with "pg_". * * However, for a cast that belongs to an extension, we must not use this - * heuristic, but just dump the cast iff we're told to (via dobj.dump). + * heuristic, but just dump the cast if and only if we're told to (via dobj.dump). */ if (!cast->dobj.ext_member) { diff --git a/src/include/access/attnum.h b/src/include/access/attnum.h index 50320d2..caa92cd 100644 --- a/src/include/access/attnum.h +++ b/src/include/access/attnum.h @@ -29,14 +29,14 @@ typedef int16 AttrNumber; */ /* * AttributeNumberIsValid - * True iff the attribute number is valid. + * True if and only if the attribute number is valid. */ #define AttributeNumberIsValid(attributeNumber) \ ((bool) ((attributeNumber) != InvalidAttrNumber)) /* * AttrNumberIsForUserDefinedAttr - * True iff the attribute number corresponds to an user defined attribute. + * True if and only if the attribute number corresponds to an user defined attribute. */ #define AttrNumberIsForUserDefinedAttr(attributeNumber) \ ((bool) ((attributeNumber) > 0)) diff --git a/src/include/access/genam.h b/src/include/access/genam.h index a51f4c4..ccd57a5 100644 --- a/src/include/access/genam.h +++ b/src/include/access/genam.h @@ -118,7 +118,7 @@ typedef enum IndexUniqueCheck /* * IndexScanIsValid - * True iff the index scan is valid. + * True if and only if the index scan is valid. */ #define IndexScanIsValid(scan) PointerIsValid(scan) diff --git a/src/include/access/heapam.h b/src/include/access/heapam.h index bfdadc3..19e9afb 100644 --- a/src/include/access/heapam.h +++ b/src/include/access/heapam.h @@ -99,7 +99,7 @@ typedef struct HeapScanDescData *HeapScanDesc; /* * HeapScanIsValid - * True iff the heap scan is valid. + * True if and only if the heap scan is valid. */ #define HeapScanIsValid(scan) PointerIsValid(scan) diff --git a/src/include/access/htup_details.h b/src/include/access/htup_details.h index a3eba98..58feece 100644 --- a/src/include/access/htup_details.h +++ b/src/include/access/htup_details.h @@ -84,7 +84,7 @@ * A word about t_ctid: whenever a new tuple is stored on disk, its t_ctid * is initialized with its own TID (location). If the tuple is ever updated, * its t_ctid is changed to point to the replacement version of the tuple. - * Thus, a tuple is the latest version of its row iff XMAX is invalid or + * Thus, a tuple is the latest version of its row if and only if XMAX is invalid or * t_ctid points to itself (in which case, if XMAX is valid, the tuple is * either locked or deleted). One can follow the chain of t_ctid links * to find the newest version of the row. Beware however that VACUUM might diff --git a/src/include/access/sdir.h b/src/include/access/sdir.h index ffc3142..ebd67c8 100644 --- a/src/include/access/sdir.h +++ b/src/include/access/sdir.h @@ -28,7 +28,7 @@ typedef enum ScanDirection /* * ScanDirectionIsValid - * True iff scan direction is valid. + * True if and only if scan direction is valid. */ #define ScanDirectionIsValid(direction) \ ((bool) (BackwardScanDirection <= (direction) && \ @@ -36,21 +36,21 @@ typedef enum ScanDirection /* * ScanDirectionIsBackward - * True iff scan direction is backward. + * True if and only if scan direction is backward. */ #define ScanDirectionIsBackward(direction) \ ((bool) ((direction) == BackwardScanDirection)) /* * ScanDirectionIsNoMovement - * True iff scan direction indicates no movement. + * True if and only if scan direction indicates no movement. */ #define ScanDirectionIsNoMovement(direction) \ ((bool) ((direction) == NoMovementScanDirection)) /* * ScanDirectionIsForward - * True iff scan direction is forward. + * True if and only if scan direction is forward. */ #define ScanDirectionIsForward(direction) \ ((bool) ((direction) == ForwardScanDirection)) diff --git a/src/include/c.h b/src/include/c.h index 30b8f51..456e171 100644 --- a/src/include/c.h +++ b/src/include/c.h @@ -464,19 +464,19 @@ typedef NameData *Name; */ /* * BoolIsValid - * True iff bool is valid. + * True if and only if bool is valid. */ #define BoolIsValid(boolean) ((boolean) == false || (boolean) == true) /* * PointerIsValid - * True iff pointer is valid. + * True if and only if pointer is valid. */ #define PointerIsValid(pointer) ((const void*)(pointer) != NULL) /* * PointerIsAligned - * True iff pointer is properly aligned to point to the given type. + * True if and only if pointer is properly aligned to point to the given type. */ #define PointerIsAligned(pointer, type) \ (((uintptr_t)(pointer) % (sizeof (type))) == 0) diff --git a/src/include/lib/binaryheap.h b/src/include/lib/binaryheap.h index 8f83c23..569278b 100644 --- a/src/include/lib/binaryheap.h +++ b/src/include/lib/binaryheap.h @@ -12,8 +12,8 @@ #define BINARYHEAP_H /* - * For a max-heap, the comparator must return <0 iff a < b, 0 iff a == b, - * and >0 iff a > b. For a min-heap, the conditions are reversed. + * For a max-heap, the comparator must return <0 if and only if a < b, 0 if and only if a == b, + * and >0 if and only if a > b. For a min-heap, the conditions are reversed. */ typedef int (*binaryheap_comparator) (Datum a, Datum b, void *arg); diff --git a/src/include/nodes/memnodes.h b/src/include/nodes/memnodes.h index f79ebd4..b398439 100644 --- a/src/include/nodes/memnodes.h +++ b/src/include/nodes/memnodes.h @@ -67,7 +67,7 @@ typedef struct MemoryContextData /* * MemoryContextIsValid - * True iff memory context is valid. + * True if and only if memory context is valid. * * Add new context types to the set accepted by this macro. */ diff --git a/src/include/nodes/parsenodes.h b/src/include/nodes/parsenodes.h index ad58b39..51c9ed2 100644 --- a/src/include/nodes/parsenodes.h +++ b/src/include/nodes/parsenodes.h @@ -1574,7 +1574,7 @@ typedef struct CreateStmt * * If skip_validation is true then we skip checking that the existing rows * in the table satisfy the constraint, and just install the catalog entries - * for the constraint. A new FK constraint is marked as valid iff + * for the constraint. A new FK constraint is marked as valid if and only if * initially_valid is true. (Usually skip_validation and initially_valid * are inverses, but we can set both true if the table is known empty.) * diff --git a/src/include/pgstat.h b/src/include/pgstat.h index 0b458e5..e432247 100644 --- a/src/include/pgstat.h +++ b/src/include/pgstat.h @@ -674,7 +674,7 @@ typedef struct PgBackendStatus */ int st_changecount; - /* The entry is valid iff st_procpid > 0, unused if st_procpid == 0 */ + /* The entry is valid if and only if st_procpid > 0, unused if st_procpid == 0 */ int st_procpid; /* Times when current backend, transaction, and activity started */ diff --git a/src/include/storage/block.h b/src/include/storage/block.h index bc503cf..7c05363 100644 --- a/src/include/storage/block.h +++ b/src/include/storage/block.h @@ -65,14 +65,14 @@ typedef BlockIdData *BlockId; /* block identifier */ /* * BlockNumberIsValid - * True iff blockNumber is valid. + * True if and only if blockNumber is valid. */ #define BlockNumberIsValid(blockNumber) \ ((bool) ((BlockNumber) (blockNumber) != InvalidBlockNumber)) /* * BlockIdIsValid - * True iff the block identifier is valid. + * True if and only if the block identifier is valid. */ #define BlockIdIsValid(blockId) \ ((bool) PointerIsValid(blockId)) diff --git a/src/include/storage/buf.h b/src/include/storage/buf.h index f1f8801..e676f03 100644 --- a/src/include/storage/buf.h +++ b/src/include/storage/buf.h @@ -26,13 +26,13 @@ typedef int Buffer; /* * BufferIsInvalid - * True iff the buffer is invalid. + * True if and only if the buffer is invalid. */ #define BufferIsInvalid(buffer) ((buffer) == InvalidBuffer) /* * BufferIsLocal - * True iff the buffer is local (not visible to other backends). + * True if and only if the buffer is local (not visible to other backends). */ #define BufferIsLocal(buffer) ((buffer) < 0) diff --git a/src/include/storage/bufmgr.h b/src/include/storage/bufmgr.h index 89447d0..924faa6 100644 --- a/src/include/storage/bufmgr.h +++ b/src/include/storage/bufmgr.h @@ -78,7 +78,7 @@ extern PGDLLIMPORT int32 *LocalRefCount; /* * BufferIsValid - * True iff the given buffer number is valid (either as a shared + * True if and only if the given buffer number is valid (either as a shared * or local buffer). * * Note: For a long time this was defined the same as BufferIsPinned, @@ -103,7 +103,7 @@ extern PGDLLIMPORT int32 *LocalRefCount; /* * BufferIsPinned - * True iff the buffer is pinned (also checks for valid buffer number). + * True if and only if the buffer is pinned (also checks for valid buffer number). * * NOTE: what we check here is that *this* backend holds a pin on * the buffer. We do not care whether some other backend does. diff --git a/src/include/storage/bufpage.h b/src/include/storage/bufpage.h index c222c32..d4d2569 100644 --- a/src/include/storage/bufpage.h +++ b/src/include/storage/bufpage.h @@ -203,7 +203,7 @@ typedef PageHeaderData *PageHeader; /* * PageIsValid - * True iff page is valid. + * True if and only if page is valid. */ #define PageIsValid(page) PointerIsValid(page) @@ -214,14 +214,14 @@ typedef PageHeaderData *PageHeader; /* * PageIsEmpty - * returns true iff no itemid has been allocated on the page + * returns true if and only if no itemid has been allocated on the page */ #define PageIsEmpty(page) \ (((PageHeader) (page))->pd_lower <= SizeOfPageHeaderData) /* * PageIsNew - * returns true iff page has not been initialized (by PageInit) + * returns true if and only if page has not been initialized (by PageInit) */ #define PageIsNew(page) (((PageHeader) (page))->pd_upper == 0) @@ -250,7 +250,7 @@ typedef PageHeaderData *PageHeader; /* * PageSizeIsValid - * True iff the page size is valid. + * True if and only if the page size is valid. */ #define PageSizeIsValid(pageSize) ((pageSize) == BLCKSZ) diff --git a/src/include/storage/itemid.h b/src/include/storage/itemid.h index a91cf97..bb4f91f 100644 --- a/src/include/storage/itemid.h +++ b/src/include/storage/itemid.h @@ -79,42 +79,42 @@ typedef uint16 ItemLength; /* * ItemIdIsValid - * True iff item identifier is valid. + * True if and only if item identifier is valid. * This is a pretty weak test, probably useful only in Asserts. */ #define ItemIdIsValid(itemId) PointerIsValid(itemId) /* * ItemIdIsUsed - * True iff item identifier is in use. + * True if and only if item identifier is in use. */ #define ItemIdIsUsed(itemId) \ ((itemId)->lp_flags != LP_UNUSED) /* * ItemIdIsNormal - * True iff item identifier is in state NORMAL. + * True if and only if item identifier is in state NORMAL. */ #define ItemIdIsNormal(itemId) \ ((itemId)->lp_flags == LP_NORMAL) /* * ItemIdIsRedirected - * True iff item identifier is in state REDIRECT. + * True if and only if item identifier is in state REDIRECT. */ #define ItemIdIsRedirected(itemId) \ ((itemId)->lp_flags == LP_REDIRECT) /* * ItemIdIsDead - * True iff item identifier is in state DEAD. + * True if and only if item identifier is in state DEAD. */ #define ItemIdIsDead(itemId) \ ((itemId)->lp_flags == LP_DEAD) /* * ItemIdHasStorage - * True iff item identifier has associated storage. + * True if and only if item identifier has associated storage. */ #define ItemIdHasStorage(itemId) \ ((itemId)->lp_len != 0) diff --git a/src/include/storage/itemptr.h b/src/include/storage/itemptr.h index 67bbdbb..883e84a 100644 --- a/src/include/storage/itemptr.h +++ b/src/include/storage/itemptr.h @@ -57,7 +57,7 @@ typedef ItemPointerData *ItemPointer; /* * ItemPointerIsValid - * True iff the disk item pointer is not NULL. + * True if and only if the disk item pointer is not NULL. */ #define ItemPointerIsValid(pointer) \ ((bool) (PointerIsValid(pointer) && ((pointer)->ip_posid != 0))) diff --git a/src/include/storage/off.h b/src/include/storage/off.h index 80d2b77..15e4eab 100644 --- a/src/include/storage/off.h +++ b/src/include/storage/off.h @@ -35,7 +35,7 @@ typedef uint16 OffsetNumber; /* * OffsetNumberIsValid - * True iff the offset number is valid. + * True if and only if the offset number is valid. */ #define OffsetNumberIsValid(offsetNumber) \ ((bool) ((offsetNumber != InvalidOffsetNumber) && \ diff --git a/src/include/storage/pos.h b/src/include/storage/pos.h index bc41502..8b70966 100644 --- a/src/include/storage/pos.h +++ b/src/include/storage/pos.h @@ -35,7 +35,7 @@ typedef PositionIdData *PositionId; /* * PositionIdIsValid - * True iff the position identifier is valid. + * True if and only if the position identifier is valid. */ #define PositionIdIsValid(positionId) \ PointerIsValid(positionId) diff --git a/src/include/utils/portal.h b/src/include/utils/portal.h index 0506bde..fd5e965 100644 --- a/src/include/utils/portal.h +++ b/src/include/utils/portal.h @@ -179,7 +179,7 @@ typedef struct PortalData /* * PortalIsValid - * True iff portal is valid. + * True if and only if portal is valid. */ #define PortalIsValid(p) PointerIsValid(p) diff --git a/src/include/utils/rel.h b/src/include/utils/rel.h index 9b8a4c9..ab63093 100644 --- a/src/include/utils/rel.h +++ b/src/include/utils/rel.h @@ -297,7 +297,7 @@ typedef struct StdRdOptions /* * RelationIsValid - * True iff relation descriptor is valid. + * True if and only if relation descriptor is valid. */ #define RelationIsValid(relation) PointerIsValid(relation) @@ -305,7 +305,7 @@ typedef struct StdRdOptions /* * RelationHasReferenceCountZero - * True iff relation reference count is zero. + * True if and only if relation reference count is zero. * * Note: * Assumes relation descriptor is valid. diff --git a/src/include/utils/tqual.h b/src/include/utils/tqual.h index e34c28a..349d018 100644 --- a/src/include/utils/tqual.h +++ b/src/include/utils/tqual.h @@ -41,7 +41,7 @@ extern PGDLLIMPORT SnapshotData SnapshotToastData; /* * HeapTupleSatisfiesVisibility - * True iff heap tuple satisfies a time qual. + * True if and only if heap tuple satisfies a time qual. * * Notes: * Assumes heap tuple is valid. diff --git a/src/pl/plperl/plperl.c b/src/pl/plperl/plperl.c index a81c185..ffc682d 100644 --- a/src/pl/plperl/plperl.c +++ b/src/pl/plperl/plperl.c @@ -672,7 +672,7 @@ activate_interpreter(plperl_interp_desc *interp_desc) { Assert(interp_desc->interp); PERL_SET_CONTEXT(interp_desc->interp); - /* trusted iff user_id isn't InvalidOid */ + /* trusted if and only if user_id isn't InvalidOid */ set_interp_require(OidIsValid(interp_desc->user_id)); plperl_active_interp = interp_desc; } @@ -848,7 +848,7 @@ plperl_init_interp(void) * This is safe because it's completely unable to load any code. * If the requested file/module has already been loaded it'll return true. * If not, it'll die. - * So now "use Foo;" will work iff Foo has already been loaded. + * So now "use Foo;" will work if and only if Foo has already been loaded. */ static OP * pp_require_safe(pTHX) diff --git a/src/pl/plperl/ppport.h b/src/pl/plperl/ppport.h index 5ea0c66..55c0d44 100644 --- a/src/pl/plperl/ppport.h +++ b/src/pl/plperl/ppport.h @@ -6642,7 +6642,7 @@ DPPP_(my_grok_oct)(pTHX_ const char *start, STRLEN *len_p, I32 *flags, NV *resul goto redo; } /* Allow \octal to work the DWIM way (that is, stop scanning - * as soon as non-octal characters are seen, complain only iff + * as soon as non-octal characters are seen, complain only if and only if * someone seems to want to use the digits eight and nine). */ if (digit == 8 || digit == 9) { if (!(*flags & PERL_SCAN_SILENT_ILLDIGIT)) diff --git a/src/pl/plpgsql/src/pl_exec.c b/src/pl/plpgsql/src/pl_exec.c index 3749fac..30878c7 100644 --- a/src/pl/plpgsql/src/pl_exec.c +++ b/src/pl/plpgsql/src/pl_exec.c @@ -2125,7 +2125,7 @@ exec_stmt_forc(PLpgSQL_execstate *estate, PLpgSQL_stmt_forc *stmt) * Open the cursor just like an OPEN command * * Note: parser should already have checked that statement supplies - * args iff cursor needs them, but we check again to be safe. + * args if and only if cursor needs them, but we check again to be safe. * ---------- */ if (stmt->argquery != NULL) @@ -3773,7 +3773,7 @@ exec_stmt_open(PLpgSQL_execstate *estate, PLpgSQL_stmt_open *stmt) * This is an OPEN cursor * * Note: parser should already have checked that statement supplies - * args iff cursor needs them, but we check again to be safe. + * args if and only if cursor needs them, but we check again to be safe. * ---------- */ if (stmt->argquery != NULL) diff --git a/src/test/regress/expected/opr_sanity.out b/src/test/regress/expected/opr_sanity.out index 26abe8a..90cf2d0 100644 --- a/src/test/regress/expected/opr_sanity.out +++ b/src/test/regress/expected/opr_sanity.out @@ -73,7 +73,7 @@ WHERE proiswindow AND (proisagg OR proretset); -----+--------- (0 rows) --- pronargdefaults should be 0 iff proargdefaults is null +-- pronargdefaults should be 0 if and only if proargdefaults is null SELECT p1.oid, p1.proname FROM pg_proc AS p1 WHERE (pronargdefaults <> 0) != (proargdefaults IS NOT NULL); diff --git a/src/test/regress/expected/type_sanity.out b/src/test/regress/expected/type_sanity.out index 3616c8d..1778e61 100644 --- a/src/test/regress/expected/type_sanity.out +++ b/src/test/regress/expected/type_sanity.out @@ -482,7 +482,7 @@ WHERE p1.rngtypid = 0 OR p1.rngsubtype = 0 OR p1.rngsubopc = 0; ----------+------------ (0 rows) --- rngcollation should be specified iff subtype is collatable +-- rngcollation should be specified if and only if subtype is collatable SELECT p1.rngtypid, p1.rngsubtype, p1.rngcollation, t.typcollation FROM pg_range p1 JOIN pg_type t ON t.oid = p1.rngsubtype WHERE (rngcollation = 0) != (typcollation = 0); diff --git a/src/test/regress/sql/opr_sanity.sql b/src/test/regress/sql/opr_sanity.sql index 40e1be2..75f88b9 100644 --- a/src/test/regress/sql/opr_sanity.sql +++ b/src/test/regress/sql/opr_sanity.sql @@ -71,7 +71,7 @@ SELECT p1.oid, p1.proname FROM pg_proc AS p1 WHERE proiswindow AND (proisagg OR proretset); --- pronargdefaults should be 0 iff proargdefaults is null +-- pronargdefaults should be 0 if and only if proargdefaults is null SELECT p1.oid, p1.proname FROM pg_proc AS p1 WHERE (pronargdefaults <> 0) != (proargdefaults IS NOT NULL); diff --git a/src/test/regress/sql/type_sanity.sql b/src/test/regress/sql/type_sanity.sql index 14cb540..19d89a6 100644 --- a/src/test/regress/sql/type_sanity.sql +++ b/src/test/regress/sql/type_sanity.sql @@ -360,7 +360,7 @@ SELECT p1.rngtypid, p1.rngsubtype FROM pg_range as p1 WHERE p1.rngtypid = 0 OR p1.rngsubtype = 0 OR p1.rngsubopc = 0; --- rngcollation should be specified iff subtype is collatable +-- rngcollation should be specified if and only if subtype is collatable SELECT p1.rngtypid, p1.rngsubtype, p1.rngcollation, t.typcollation FROM pg_range p1 JOIN pg_type t ON t.oid = p1.rngsubtype