From 179cdd098196338880bdbb39c39a788abdad4dd8 Mon Sep 17 00:00:00 2001 From: Heikki Linnakangas Date: Fri, 8 May 2015 21:58:57 +0300 Subject: Add macros to check if a filename is a WAL segment or other such file. We had many instances of the strlen + strspn combination to check for that. This makes the code a bit easier to read. --- src/backend/access/transam/xlog.c | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) (limited to 'src/backend/access/transam/xlog.c') diff --git a/src/backend/access/transam/xlog.c b/src/backend/access/transam/xlog.c index 92822a192b0..5097173c609 100644 --- a/src/backend/access/transam/xlog.c +++ b/src/backend/access/transam/xlog.c @@ -3577,8 +3577,7 @@ RemoveOldXlogFiles(XLogSegNo segno, XLogRecPtr PriorRedoPtr, XLogRecPtr endptr) while ((xlde = ReadDir(xldir, XLOGDIR)) != NULL) { /* Ignore files that are not XLOG segments */ - if (strlen(xlde->d_name) != 24 || - strspn(xlde->d_name, "0123456789ABCDEF") != 24) + if (!IsXLogFileName(xlde->d_name)) continue; /* @@ -3650,8 +3649,7 @@ RemoveNonParentXlogFiles(XLogRecPtr switchpoint, TimeLineID newTLI) while ((xlde = ReadDir(xldir, XLOGDIR)) != NULL) { /* Ignore files that are not XLOG segments */ - if (strlen(xlde->d_name) != 24 || - strspn(xlde->d_name, "0123456789ABCDEF") != 24) + if (!IsXLogFileName(xlde->d_name)) continue; /* @@ -3839,10 +3837,7 @@ CleanupBackupHistory(void) while ((xlde = ReadDir(xldir, XLOGDIR)) != NULL) { - if (strlen(xlde->d_name) > 24 && - strspn(xlde->d_name, "0123456789ABCDEF") == 24 && - strcmp(xlde->d_name + strlen(xlde->d_name) - strlen(".backup"), - ".backup") == 0) + if (IsBackupHistoryFileName(xlde->d_name)) { if (XLogArchiveCheckDone(xlde->d_name)) { -- cgit v1.2.3