Re: patch for space around the FragmentDelimiter

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: patch for space around the FragmentDelimiter
Дата
Msg-id 18181.1235958563@sss.pgh.pa.us
обсуждение исходный текст
Ответ на patch for space around the FragmentDelimiter  (Sushant Sinha <sushant354@gmail.com>)
Ответы Re: patch for space around the FragmentDelimiter
Список pgsql-hackers
Sushant Sinha <sushant354@gmail.com> writes:
> FragmentDelimiter is an argument for ts_headline function to separates
> different headline fragments. The default delimiter is " ... ".
> Currently if someone specifies the delimiter as an option to the
> function, no extra space is added around the delimiter. However, it does
> not look good without space around the delimter. 

Maybe not to you, for the particular delimiter you happen to be working
with, but it doesn't follow that spaces are always appropriate.

> Since the option parsing function removes any space around the  given
> value, it is not possible to add any desired space. The attached patch
> adds space when a FragmentDelimiter is specified.

I think this is a pretty bad idea.  Better would be to document how to
get spaces into the delimiter, ie, use double quotes:
... FragmentDelimiter = " ... " ...

Hmm, actually, it looks to me that the documentation already shows this,
in the example of the default values.
        regards, tom lane


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

Предыдущее
От: Bruce Momjian
Дата:
Сообщение: Re: cardinality()
Следующее
От: Tom Lane
Дата:
Сообщение: Re: WIP: named and mixed notation support