On Tue, Jan 23, 2018 at 11:33:56AM -0500, Peter Eisentraut wrote:
> Here is a proposed patch set to clean this up. First, add some test
> coverage for record_image_cmp. (There wasn't any, only for
> record_image_eq as part of MV testing.) Then, remove the GET_ macros
> from record_image_{eq,cmp}. And finally remove all that byte-masking
> stuff from postgres.h.
Good catch. Coverage reports mark those areas as empty! Similarly the
functions for record_* are mostly not used. Some tests could be added
for them at the same time. The four error paths of those functions are
tested as well, which is cool to see. Even if the buildfarm explodes
after 0002 and 0003, 0001 is still a good addition. The set looks good
to me by the way.
--
Michael