Re: patch for space around the FragmentDelimiter

yeah you are right. I did not know that you can pass space using double
quotes.

-Sushant.

On Sun, 2009-03-01 at 20:49 -0500, Tom Lane wrote:
> Sushant Sinha <sushant354(at)gmail(dot)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