Re: Small patch to fix build on Windows

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: Dmitry Igrishin <dmitigr(at)gmail(dot)com>
Cc: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Small patch to fix build on Windows
Date: 2019-08-09 02:45:17
Message-ID: 20190809024517.GF3194@paquier.xyz
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, Aug 08, 2019 at 10:46:07PM +0300, Dmitry Igrishin wrote:
> This looks nice for a Perl hacker :-). As for me, it looks unusual and
> a bit confusing. I never
> programmed in Perl, but I was able to quickly understand where the
> problem lies due to the
> style adopted in other languages, when the contents are enclosed in
> quotation marks, and
> the quotation marks are escaped if they are part of the contents.
> So, should I fix it? Any thoughts?

FWIW, I like Alvaro's suggestion about qq{} in this case, as it makes
sure that double-quotes are correctly applied where they should.
--
Michael

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2019-08-09 03:00:21 Re: Add "password_protocol" connection parameter to libpq
Previous Message Sehrope Sarkuni 2019-08-09 02:34:26 Re: [Proposal] Table-level Transparent Data Encryption (TDE) and Key Management Service (KMS)