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
signature.asc
Description: PGP signature