so 3. 9. 2022 v 7:57 odesílatel Tom Lane <t...@sss.pgh.pa.us> napsal:
> John Naylor <john.nay...@enterprisedb.com> writes: > > On Sat, Sep 3, 2022 at 12:30 PM Pavel Stehule <pavel.steh...@gmail.com> > wrote: > >> /usr/local/pgsql/master/include/server/port/simd.h: In function > ‘vector8_has’: > >> /usr/local/pgsql/master/include/server/port/simd.h:168:27: warning: > comparison of integer expressions of different signedness: ‘int’ and ‘long > unsigned int’ [-Wsign-compare] > >> 168 | for (int i = 0; i < sizeof(Vector8); i++) > > > "int" should probably be "Size" -- does that remove the warning? > > Agreed, should be Size or size_t, or else cast the sizeof() result. > But I wonder why none of the buildfarm is showing such a warning. > I got this warning when I compiled plgsql_check against master with enabled asserts https://github.com/okbob/plpgsql_check In file included from /usr/local/pgsql/master/include/server/mb/pg_wchar.h:22, from src/format.c:17: /usr/local/pgsql/master/include/server/port/simd.h: In function ‘vector8_has_le’: /usr/local/pgsql/master/include/server/port/simd.h:219:27: warning: comparison of integer expressions of different signedness: ‘int’ and ‘long unsigned int’ [-Wsign-compare] 219 | for (int i = 0; i < sizeof(Vector8); i++) | ^ > regards, tom lane >
