On Fri, Feb 13, 2026 at 06:06:26PM +1100, Peter Smith wrote: > At the same time, I combined all previous patches.
That's a practice better than sending a handful bag of patches that each touch one single file. :) Seems globally clean to me, except for two changes. - zp = gzipcs->zp = (z_streamp) pg_malloc(sizeof(z_stream)); + zp = gzipcs->zp = (z_streamp) pg_malloc_object(z_stream); [...] - zp = (z_streamp) pg_malloc(sizeof(z_stream)); + zp = (z_streamp) pg_malloc_object(z_stream); These two defeat the purpose of the change. _object is useful because we can enforce type checks based on their size. Note that We have this declaration in zlib: zlib.h:typedef z_stream FAR *z_streamp; So we should be OK with dropping the casts entirely, with I guess compilers not warning if types do not match as an effect of this typedef declaration. I have removed these, and things seem OK here, now to say if the buildfarm is entirely OK will be a different thing. I'll see about that if/once required. -- Michael
signature.asc
Description: PGP signature
