Re: [COMMITTERS] pgsql: Remove QueryOperand->istrue flag, it was used only in cover

From: ITAGAKI Takahiro <itagaki(dot)takahiro(at)oss(dot)ntt(dot)co(dot)jp>
To: pgsql-patches(at)postgresql(dot)org, teodor(at)postgresql(dot)org
Subject: Re: [COMMITTERS] pgsql: Remove QueryOperand->istrue flag, it was used only in cover
Date: 2007-09-13 00:53:51
Message-ID: 20070913095046.5DBA.ITAGAKI.TAKAHIRO@oss.ntt.co.jp
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-patches

This change introduced a compiler warning. Here is a fix for it.
| tsrank.c: In function `calc_rank_cd':
| tsrank.c:710: warning: assignment from incompatible pointer type

Index: src/backend/utils/adt/tsrank.c
===================================================================
--- src/backend/utils/adt/tsrank.c (HEAD)
+++ src/backend/utils/adt/tsrank.c (working copy)
@@ -707,7 +707,7 @@
}

qr.query = query;
- qr.operandexist = (int*)palloc0(sizeof(bool) * query->size);
+ qr.operandexist = (bool *) palloc0(sizeof(bool) * query->size);

doc = get_docrep(txt, &qr, &doclen);
if (!doc)

teodor(at)postgresql(dot)org (Teodor Sigaev) wrote:

> Log Message:
> -----------
> Remove QueryOperand->istrue flag, it was used only in cover ranking
> (ts_rank_cd). Use palloc'ed array in ranking instead of flag.
>
> Modified Files:
> --------------
> pgsql/src/backend/utils/adt:
> tsrank.c (r1.5 -> r1.6)
> (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/tsrank.c?r1=1.5&r2=1.6)

Regards,
---
ITAGAKI Takahiro
NTT Open Source Software Center

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Bruce Momjian 2007-09-13 03:34:11 pgsql: Update Japanese FAQ, backpatch.
Previous Message Alvaro Herrera 2007-09-12 22:14:59 pgsql: Fix a memory leak in the autovacuum launcher code.

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2007-09-13 03:43:22 Re: [HACKERS] New Zealand - TZ change
Previous Message Bruce Momjian 2007-09-12 21:41:14 Re: actualised forgotten Magnus's patch for plpgsql MOVE statement