From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Tatsuo Ishii <ishii(at)postgresql(dot)org> |
Cc: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Why SyncRepWakeQueue is not static? |
Date: | 2015-04-02 14:46:10 |
Message-ID: | CA+Tgmob6z-NdLjcYRyrhT5AO=OFPEE=SuZVvAUCawi_+A_P3UA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Mar 25, 2015 at 10:07 PM, Tatsuo Ishii <ishii(at)postgresql(dot)org> wrote:
>>> Fix committed/pushed from master to 9.2. 9.1 declares it as a static
>>> function.
>>
>> Er, is that a good idea to back-patch that? Normally routine specs are
>> maintained stable on back-branches, and this is just a cosmetic
>> change.
>
> I'm not sure if it's a cosmetic change or not. I thought declaring
> to-be-static function as extern is against our coding
> standard. Moreover, if someone wants to change near the place in the
> source code in the future, changes made to head may not be easily back
> patched or cherry-picked to older branches if I do not back patch it.
True. But if any third-party code calls that function, you just broke
it. I don't think keeping the back-branches consistent with master is
a sufficiently good reason for such a change.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Heikki Linnakangas | 2015-04-02 14:58:23 | Re: What exactly is our CRC algorithm? |
Previous Message | Robert Haas | 2015-04-02 14:44:42 | Re: possible dsm bug in dsm_attach() |