Tom Lane wrote:
Teodor Sigaev <[EMAIL PROTECTED]> writes:
! pbuf = buf;
! while( !isspace( *pbuf ) )
! pbuf++;
! *pbuf = '\0';
Surely the loop needs to look like
while (*pbuf && !is
Teodor Sigaev <[EMAIL PROTECTED]> writes:
> ! pbuf = buf;
> ! while( !isspace( *pbuf ) )
> ! pbuf++;
> ! *pbuf = '\0';
Surely the loop needs to look like
while (*pbuf && !isspace(*pbuf))
> > Yup, that solved the problem, thanks.> I'll commit extended patch - there
> > is one more place with the same bug.
Ok, thanks.
> Wouldn't it be more efficiently written to walk the string backwards until
> > !isspace instead? Not sure that it matters at all, but then you'll
> > normallyi n
Yup, that solved the problem, thanks.
I'll commit extended patch - there is one more place with the same bug.
Wouldn't it be more efficiently written to walk the string backwards until
!isspace instead? Not sure that it matters at all, but then you'll
normallyi never step over more than two by
On Mon, Mar 26, 2007 at 02:32:26PM +0400, Teodor Sigaev wrote:
> >FWIW, it looks like it failed to reject stopwords. Is it possible you
> Right.
>
> I suppose the problem is with '\r\n'... Try attached patch.
> --
> Teodor Sigaev E-mail: [EMAIL PROTECTED]
>
FWIW, it looks like it failed to reject stopwords. Is it possible you
Right.
I suppose the problem is with '\r\n'... Try attached patch.
--
Teodor Sigaev E-mail: [EMAIL PROTECTED]
WWW: http://www.sigaev.ru/
***
Magnus Hagander <[EMAIL PROTECTED]> writes:
> tsearch2 regression tests are also failing on win32/msvc, with attached
> diffs.
> Any pointers on where to start? ;)
FWIW, it looks like it failed to reject stopwords. Is it possible you
ran it in an environment that would make it pick the Russian st
