Re: plpgsql.warn_shadow

From: Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>
To: Petr Jelinek <petr(at)2ndquadrant(dot)com>
Cc: Simon Riggs <simon(at)2ndquadrant(dot)com>, Marko Tiikkaja <marko(at)joh(dot)to>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Florian Pflug <fgp(at)phlo(dot)org>, Marti Raudsepp <marti(at)juffo(dot)org>
Subject: Re: plpgsql.warn_shadow
Date: 2014-03-18 19:45:53
Message-ID: CAFj8pRCOzDTek4GVo3iuAbP6ZLJT_HcRDs9k3RyCqqVRS5LZyg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

2014-03-18 20:44 GMT+01:00 Petr Jelinek <petr(at)2ndquadrant(dot)com>:

>
> On 18/03/14 20:36, Pavel Stehule wrote:
>
>>
>> <para>
>> + To aid the user in finding instances of simple but common problems
>> before
>> + they cause harm, <application>PL/PgSQL</> provides a number of
>> additional
>> + <replaceable>checks</>. When enabled, depending on the
>> configuration, they
>> + can be used to emit either a <literal>WARNING</> or an
>> <literal>ERROR</>
>> + during the compilation of a function.
>> + </para>
>>
>> >>>provides a number of additional<<<
>>
>> There will be only one check in 9.4, so this sentence is confusing and
>> should be reformulated.
>>
>
> Thanks, yeah I left that sentence unchanged from original patch, maybe I
> should remove the word "number" there as it implies that there are a lot of
> them, but I don't really want to change everything to singular when the
> input is specified as list.

What about add sentence: in this moment only "shadowed-variables" is
available?

Pavel

>
>
> --
> Petr Jelinek http://www.2ndQuadrant.com/
> PostgreSQL Development, 24x7 Support, Training & Services
>
>

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Merlin Moncure 2014-03-18 19:48:26 Re: Planner hints in Postgresql
Previous Message Petr Jelinek 2014-03-18 19:44:05 Re: plpgsql.warn_shadow