Kirill Reshke <reshkekir...@gmail.com> writes:
> On Thu, 21 Aug 2025 at 17:00, jian he <jian.universal...@gmail.com> wrote:
>> RangeVar   *rel = (RangeVar *) linitial(stmt->relations);
>> if (!IsA(rel, RangeVar))

> These two lines are weird. Looks like  linitial(stmt->relations)
> should be assigned to variable with type Node* first?

We take that sort of shortcut in many places.  If there's not any need
for the code to deal with more than one node type, an extra variable
that's used just for the IsA test seems like a lot of notational
overhead.

                        regards, tom lane


Reply via email to