Not sure if I missed it in my first round of review, or if this is new in this version, but this now generats a warning when asserts are disabled, given that the lip variable in xlog_recover_process_intents is only used in asserts. We'll need to remove it and just open code the dereference in the two asserts that use it.
