Hello
2012/12/31 Pavel Stehule <pavel.steh...@gmail.com>:
> Hello
>
> 2012/12/31 Stephen Frost <sfr...@snowman.net>:
>> Pavel,
>>
>> * Pavel Stehule (pavel.steh...@gmail.com) wrote:
>>> A result from ours previous talk was a completely disabling mixing
>>> positional and ordered placeholders - like is requested by man and gcc
>>> raises warnings there.
>>>
>>> But mixing is not explicitly disallowed in doc, and mixing was tested
>>> in our regress tests. There are tests where placeholders are mixed -
>>> so anybody can use it.
>>> select format('Hello %s %1$s %s', 'World', 'Hello again'); -- is
>>> enabled and supported and result is expected
>>
>> Alright, then I agree that raising a warning in that case makes sense
>> and let's update the docs to reflect that it shouldn't be done (like
>> what glibc/gcc do).
>
> so there are two patches - first is fix in logic when positional and
> ordered parameters are mixed + add warning in this situation. Second
> patch enables possibility to specify width for %s conversion.
>
> I didn't finalize documentation due my net good English skills -
> probably there is necessary new paragraph about function "format"
> elsewhere than in table
>
> Regards
>
> Pavelupdated patches due changes for better variadic "any" function. apply fix_mixing_positinal_ordered_placeholders_warnings_20130126.patch first Regards Pavel
fix_mixing_positinal_ordered_placeholders_warnings_20130126.patch
Description: Binary data
format_width_20130126.patch
Description: Binary data
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
