Re: Remove an obsolete comment in gistinsert()

Поиск
Список
Период
Сортировка
От Tender Wang
Тема Re: Remove an obsolete comment in gistinsert()
Дата
Msg-id CAHewXNnMV1fxvmt9ZWYrD1y0UbjAzJutqpsXeBO9t_kCDw2s=w@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Remove an obsolete comment in gistinsert()  (Aleksander Alekseev <aleksander@timescale.com>)
Список pgsql-hackers


Aleksander Alekseev <aleksander@timescale.com> 于2024年11月5日周二 22:08写道:
Hi Tender,

> While learning the GIST codes, I find an obsolete comment in gistinsert ().
>
> itup = gistFormTuple(giststate, r,
>                                   values, isnull, true /* size is currently bogus */ );

Thanks for reporting. I agree that this is an oversight of commit 1f7ef54.

The commit changed the signature of gistFormTuple():

```
 IndexTuple
 gistFormTuple(GISTSTATE *giststate, Relation r,
-                         Datum attdata[], int datumsize[], bool isnull[])
+                         Datum attdata[], bool isnull[], bool newValues)
```

... but left the comment for the `datumsize` argument:

```
        itup = gistFormTuple(&buildstate->giststate, index,
-               values, NULL /* size is currently bogus */, isnull);
+               values, isnull, true /* size is currently bogus */);
```

I checked the rest of gistFormTuple() calls and also looked for other
comments like this. There seems to be only one call like this to fix.

Thanks for reviewing this. I have added it to the 2015-01 commitfest. 

--
Thanks,
Tender Wang

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