Re: Improve node type forward reference
От | Nathan Bossart |
---|---|
Тема | Re: Improve node type forward reference |
Дата | |
Msg-id | Zw5_ne4SCEABRTiU@nathan обсуждение исходный текст |
Ответ на | Re: Improve node type forward reference (Peter Eisentraut <peter@eisentraut.org>) |
Ответы |
Re: Improve node type forward reference
|
Список | pgsql-hackers |
On Tue, Oct 15, 2024 at 09:02:48AM +0200, Peter Eisentraut wrote: > On 14.10.24 23:28, Nathan Bossart wrote: >> On Mon, Oct 14, 2024 at 09:47:59AM +0200, Peter Eisentraut wrote: >> > But we can do this better by using an incomplete struct, like >> > >> > struct Query *viewQuery ...; >> > >> > That way, everything has the correct type and fewer casts are required. This >> > technique is already used elsewhere in node type definitions. >> >> I noticed that the examples in parsenodes.h are for structs defined within >> the same file. If the struct is defined in a separate file, I guess you >> might need to include another header file wherever it is used, but that >> doesn't seem too bad. > > No, you can leave the struct incomplete. You only need to provide its full > definition (= include the other header file) if you actually want to access > the struct's fields. That's what I figured. This one LGTM, too, then. -- nathan
В списке pgsql-hackers по дате отправления: