Re: plpgsql.warn_shadow

Поиск
Список
Период
Сортировка
От Pavel Stehule
Тема Re: plpgsql.warn_shadow
Дата
Msg-id CAFj8pRDNp0wEK9OrKSsjB2bYqzi+Lsb0LVqY5TV5AsLwh48dQw@mail.gmail.com
обсуждение исходный текст
Ответ на Re: plpgsql.warn_shadow  (Petr Jelinek <petr@2ndquadrant.com>)
Список pgsql-hackers



2014-03-18 20:49 GMT+01:00 Petr Jelinek <petr@2ndquadrant.com>:

On 18/03/14 20:45, Pavel Stehule wrote:


2014-03-18 20:44 GMT+01:00 Petr Jelinek <petr@2ndquadrant.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?

There is something like that said 2 paragraphs down the line:
+  The default is <literal>"none"</>. Currently the list of available checks
+  includes only one:
+  <variablelist>
+   <varlistentry>
+    <term><varname>shadowed-variables</varname></term>

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

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

Предыдущее
От: Alvaro Herrera
Дата:
Сообщение: Re: Patch: show relation and tuple infos of a lock to acquire
Следующее
От: Josh Berkus
Дата:
Сообщение: Re: Wiki Page Draft for upcoming release