Re: Fix gcc warning in sync.c (usr/src/backend/storage/sync/sync.c)

Поиск
Список
Период
Сортировка
От Kyotaro Horiguchi
Тема Re: Fix gcc warning in sync.c (usr/src/backend/storage/sync/sync.c)
Дата
Msg-id 20220711.143500.1660612605708077512.horikyota.ntt@gmail.com
обсуждение исходный текст
Ответ на Fix gcc warning in sync.c (usr/src/backend/storage/sync/sync.c)  (Ranier Vilela <ranier.vf@gmail.com>)
Ответы Re: Fix gcc warning in sync.c (usr/src/backend/storage/sync/sync.c)  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
At Sat, 9 Jul 2022 21:53:31 -0300, Ranier Vilela <ranier.vf@gmail.com> wrote in 
> sync.c: In function ‘RememberSyncRequest’:
> sync.c:528:10: warning: assignment to ‘PendingFsyncEntry *’ {aka ‘struct
> <anonymous> *’} from incompatible pointer type ‘PendingUnlinkEntry *’ {aka
> ‘struct <anonymous> *’} [-Wincompatible-pointer-types]
>   528 |    entry = (PendingUnlinkEntry *) lfirst(cell);
> 
> Although the structures are identical, gcc bothers to assign a pointer from
> one to the other.

If the entry were of really PendingSyncEntry, it would need a fix, but
at the same time everyone should see the same warning at their hand.

Actually, I already see the following line (maybe) at the place instead.

529@master,REL14, 508@REL13
>        PendingUnlinkEntry *entry = (PendingUnlinkEntry *) lfirst(cell);


regards.

-- 
Kyotaro Horiguchi
NTT Open Source Software Center

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

Предыдущее
От: Thomas Munro
Дата:
Сообщение: Re: pgsql: dshash: Add sequential scan support.
Следующее
От: John Naylor
Дата:
Сообщение: Re: NAMEDATALEN increase because of non-latin languages