[email protected] (Michael Meskes) writes: > Log Message: > ----------- > Removed comparison of unsigned expression < 0.
I believe that this change is a bad idea and should be reverted: > fe-exec.c (r1.200 -> r1.201) > > (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/libpq/fe-exec.c?r1=1.200&r2=1.201) This is correct only if enum ExecStatusType is considered unsigned by the compiler, an assumption that I don't believe is portable. (In fact, Zdenek was just complaining about something else that seems to prove Solaris' compiler treats enums as signed always.) The other two changes seem ok, since they are for size_t variables which are definitely supposed to be unsigned. regards, tom lane -- Sent via pgsql-committers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-committers
