Commit3422f5f

committed

Update comment about ExprState.escontext

The updated comment provides more helpful guidance by mentioning thatescontext should be set when soft error handling is needed.Reported-by: Jian He <jian.universality@gmail.com>Discussion:https://postgr.es/m/CACJufxEo4sUjKCYtda0_qt9tazqqKPmF1cqhW9KBOUeJFqQd2g@mail.gmail.comBackpatch-through: 17

1 parent7829f85 commit3422f5fCopy full SHA for 3422f5f

File tree

-2

lines changed

-2

lines changed

Lines changed: 4 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -132,8 +132,10 @@ typedef struct ExprState`
`132`	`132`	`bool*innermost_domainnull;`
`133`	`133`
`134`	`134`	`/*`
`135`		`- * For expression nodes that support soft errors. Should be set to NULL`
`136`		`- * before calling ExecInitExprRec() if the caller wants errors thrown.`
	`135`	`+ * For expression nodes that support soft errors. Should be set to NULL if`
	`136`	`+ * the caller wants errors to be thrown. Callers that do not want errors`
	`137`	`+ * thrown should set it to a valid ErrorSaveContext before calling`
	`138`	`+ * ExecInitExprRec().`
`137`	`139`	`*/`
`138`	`140`	`ErrorSaveContext*escontext;`
`139`	`141`	`}ExprState;`

Comments

(0)