Commit244dd95

committed

Update some obsolete comments.

Fix a few stray references to expression eval functions that don'texist anymore or don't take the same input representation they used to.

1 parent6a101b0 commit244dd95Copy full SHA for 244dd95

File tree

-4

lines changed

-4

lines changed

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -354,8 +354,8 @@ make_and_qual(Node qual1, Node qual2)`
`354`	`354`	`}`
`355`	`355`
`356`	`356`	`/*`
`357`		`- *Sometimes (such as in the input of ExecQual), we use lists of expression`
`358`		`- *nodes with implicit AND semantics.`
	`357`	`+ *The planner frequently prefers to represent qualification expressions`
	`358`	`+ *as lists of boolean expressions with implicit AND semantics.`
`359`	`359`	`*`
`360`	`360`	`* These functions convert between an AND-semantics expression list and the`
`361`	`361`	`* ordinary representation of a boolean expression.`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -4743,7 +4743,7 @@ RelationGetIndexExpressions(Relation relation)`
`4743`	`4743`	`* RelationGetIndexPredicate -- get the index predicate for an index`
`4744`	`4744`	`*`
`4745`	`4745`	`* We cache the result of transforming pg_index.indpred into an implicit-AND`
`4746`		`- * node tree (suitable forExecQual).`
	`4746`	`+ * node tree (suitable foruse in planning).`
`4747`	`4747`	`* If the rel is not an index or has no predicate, we return NIL.`
`4748`	`4748`	`* Otherwise, the returned tree is copied into the caller's memory context.`
`4749`	`4749`	`* (We don't want to return a pointer to the relcache copy, since it could`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -4759,7 +4759,7 @@ exec_assign_value(PLpgSQL_execstate *estate,`
`4759`	`4759`	`* fixed-length array types we skip the assignment. We can't`
`4760`	`4760`	`* support assignment of a null entry into a fixed-length`
`4761`	`4761`	`* array, either, so that's a no-op too. This is all ugly but`
`4762`		`- * corresponds to the current behavior ofExecEvalArrayRef().`
	`4762`	`+ * corresponds to the current behavior ofexecExpr*.c.`
`4763`	`4763`	`*/`
`4764`	`4764`	`if (arrayelem->arraytyplen>0&&/* fixed-length array? */`
`4765`	`4765`	`(oldarrayisnull\|\|isNull))`

Comments

(0)