My only thought is that `record_data` should be declared on a new line (or maybe just padded for legibility) but otherwise this LGTM