I'm working on adding a tuple_data_record function to pageinspect to parallel tuple_data_split (returning the record type of the relation being examined rather than an array of attributes as on-disk byte data).
This my first real foray into the Postgres codebase, so an beginner question: I noticed that existing C functions in pageinspect often do error checking and then report the error with ereport but still continue on to execute the rest of the function instead of early returning. Is this standard practice? Or should I be reporting the error and then cleaning up and returning rather than continuing to execute?
