Re: Small patch to fix build on Windows - Mailing list pgsql-hackers

From Dmitry Igrishin
Subject Re: Small patch to fix build on Windows
Date
Msg-id CAAfz9KPZbPjoWTqOb5moi_YWvdbSjAMZsrVBW0cBw33Q560CLw@mail.gmail.com
Whole thread Raw
In response to Re: Small patch to fix build on Windows  (Michael Paquier <michael@paquier.xyz>)
Responses Re: Small patch to fix build on Windows
List pgsql-hackers
пт, 9 авг. 2019 г. в 05:45, Michael Paquier <michael@paquier.xyz>:
>
> 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.
The attached 4rd version of the patch uses qq||. I used qq|| instead
of qq{} for consistency because qq|| is already used in Solution.pm:

  return qq|VisualStudioVersion = $self->{VisualStudioVersion}
  MinimumVisualStudioVersion = $self->{MinimumVisualStudioVersion}
  |;

Attachment

pgsql-hackers by date:

Previous
From: Jeff Davis
Date:
Subject: Re: Add "password_protocol" connection parameter to libpq
Next
From: Kyotaro Horiguchi
Date:
Subject: Re: Small patch to fix build on Windows