Craig Ringer wrote: > Revised patch attached. > > I have _not_ rewritten to use sscanf yet. I'll do that next, so you can > choose the fewer-changes patch for backpatching if desired.
Pushed, with a further change: it seems more sensible to centralize the whole operation of building the path, rather than just the format string, so I created a new function to do that. The code looks cleaner IMO this way IMO. All tests pass in all branches. BTW the way the XLogSegNoOffsetToRecPtr() et al macros were modified to accept wal_segment_size at the end of the argument list, *after* the output argument, seems pretty bad style. -- Álvaro Herrera https://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services