Re: pgsql: Add putenv support for msvcrt from Visual Studio 2013

Поиск
Список
Период
Сортировка
От Michael Paquier
Тема Re: pgsql: Add putenv support for msvcrt from Visual Studio 2013
Дата
Msg-id CAB7nPqSRfAYLeHso-g1RtF50jKcTuN-vESHxLbdvp2w0bxgsCg@mail.gmail.com
обсуждение исходный текст
Ответ на Re: pgsql: Add putenv support for msvcrt from Visual Studio 2013  (Noah Misch <noah@leadboat.com>)
Список pgsql-hackers
On Sun, Dec 4, 2016 at 5:58 AM, Noah Misch <noah@leadboat.com> wrote:
> On Wed, Nov 30, 2016 at 04:24:34PM +0000, Christian Ullrich wrote:
>> * From: Michael Paquier
>> > would be nice to mention in a code comment that this what Noah has
>> > mentioned upthread: if a CRT loads while pgwin32_putenv() is
>> > executing, the newly-loaded CRT will always have the latest value.
>>
>> I fixed the existing comment by removing the last sentence that is in the wrong place now, but I don't see the point
insuddenly starting to explain why it is done this way and not the other.
 
>>
>> > Based on that 0006 will need a rebase, nothing huge though.
>>
>> Done, new revisions attached.
>
> I committed patches 1-7 with some comment changes, a pgindent, and other
> cosmetic trivia.  (The file was pgindent-clean before this work.  Patch 6
> still achieved the more-compact formatting you sought.)

Thanks all for helping in closing this item. We have a fine result here.
-- 
Michael



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Ian Barwick
Дата:
Сообщение: Re: [DOCS] monitoring.sgml - clarify length of query text displayed in pg_stat_statements
Следующее
От: Amit Kapila
Дата:
Сообщение: Re: Speed up Clog Access by increasing CLOG buffers