pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Vik Fearing <vik.fearing@2ndquadrant.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
To: pgsql-hackers@postgreSQL.org
Subject: Re: Inconsistent syntax for NumericOnly grammar production
Date: Mon, 29 May 2017 15:31:15 +0200
Message-ID: <9dd9dbcf-69aa-a093-e1d1-64007306678b@2ndquadrant.com> (raw)
In-Reply-To: <30908.1496006184@sss.pgh.pa.us>
References: <30908.1496006184@sss.pgh.pa.us>
List-Unsubscribe: <mailto:majordomo@postgresql.org?body=unsub%20pgsql-hackers>

On 05/28/2017 11:16 PM, Tom Lane wrote:
> The inconsistency here means that you can do, for example,
> 
> regression=# set random_page_cost = +4;
> SET
> regression=# set random_page_cost = 4.2;
> SET
> 
> but not
> 
> regression=# set random_page_cost = +4.2;
> ERROR:  syntax error at or near "4.2"
> LINE 1: set random_page_cost = +4.2;
>                                 ^
> Any objections to allowing "+ FCONST" here?  I'm inclined to
> fix this and back-patch it as well.

Seems like the right thing to do; no objections here.
-- 
Vik Fearing                                          +33 6 46 75 15 36
http://2ndQuadrant.fr     PostgreSQL : Expertise, Formation et Support


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers



view thread (2+ messages)

Message-ID: <9dd9dbcf-69aa-a093-e1d1-64007306678b@2ndquadrant.com>
Permalink:  ../9dd9dbcf-69aa-a093-e1d1-64007306678b@2ndquadrant.com/
Also on:    postgresql.org/message-id/9dd9dbcf-69aa-a093-e1d1-64007306678b@2ndquadrant.com

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-hackers@postgresql.org
  Cc: vik.fearing@2ndquadrant.com, tgl@sss.pgh.pa.us
  Subject: Re: Inconsistent syntax for NumericOnly grammar production
  In-Reply-To: <9dd9dbcf-69aa-a093-e1d1-64007306678b@2ndquadrant.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox