Re: Is "Window" really safe as a global typedef name?

From: Greg Stark <greg(dot)stark(at)enterprisedb(dot)com>
To: Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Is "Window" really safe as a global typedef name?
Date: 2008-12-20 08:18:37
Message-ID: 4129BBB5-0AB3-44D2-8125-52B673EA1FCF@enterprisedb.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

OLAPWindow? Gives a clue what meaning of "window" it's referring to...

--
Greg

On 20 Dec 2008, at 03:13, Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com
> wrote:

> Tom Lane wrote:
>> I'm looking at the window-functions patch and wondering just what
>> kind
>> of trouble we'll get into if we leave its new plan node type named
>> just
>> "Window". I've already confirmed that this is a direct conflict
>> against
>> a typedef in <X11/X.h>, and I'd be not the least bit surprised if
>> it's
>> used in even-more-popular system headers on Windows or Darwin. Now
>> maybe you could always get away with not including such headers
>> together
>> with plannodes.h, but it sure looks like problems waiting to happen.
>> So I'm thinking we'd better rename it, but I'm not coming up with
>> anything good; the best I can do after a long day is "EvalWindow",
>> and that doesn't seem particularly inspired. Any suggestions?
>
> EvalWindow sounds like a function in src/backend/executor/.
>
> WindowAgg?
>
> --
> Heikki Linnakangas
> EnterpriseDB http://www.enterprisedb.com
>
> --
> Sent via pgsql-hackers mailing list (pgsql-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Simon Riggs 2008-12-20 09:02:39 Re: Hot standby and b-tree killed items
Previous Message Heikki Linnakangas 2008-12-20 08:13:34 Re: Is "Window" really safe as a global typedef name?