From 4be19e2c9b04ab91e1f12ce75ba0b22d8af94916 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=81lvaro=20Herrera?= Date: Tue, 26 Nov 2024 17:10:07 +0100 Subject: [PATCH] Clean up newlines following left parentheses Most came in during the 17 cycle, so backpatch there. Some (particularly reorderbuffer.h) are very old, but backpatching doesn't seem useful. Like commits c9d297751959, c4f113e8fef9. --- contrib/bloom/bloom.h | 9 +++---- src/backend/backup/basebackup_incremental.c | 8 +++--- src/backend/utils/init/postinit.c | 3 +-- src/bin/pg_combinebackup/load_manifest.c | 3 +-- src/common/parse_manifest.c | 5 ++-- src/common/unicode/category_test.c | 24 ++++++------------ src/include/common/parse_manifest.h | 4 +-- src/include/replication/reorderbuffer.h | 27 +++++++-------------- 8 files changed, 30 insertions(+), 53 deletions(-) diff --git a/contrib/bloom/bloom.h b/contrib/bloom/bloom.h index fba3ba77711..83c81e640f9 100644 --- a/contrib/bloom/bloom.h +++ b/contrib/bloom/bloom.h @@ -110,12 +110,9 @@ typedef struct BloomOptions * FreeBlockNumberArray - array of block numbers sized so that metadata fill * all space in metapage. */ -typedef BlockNumber FreeBlockNumberArray[ - MAXALIGN_DOWN( - BLCKSZ - SizeOfPageHeaderData - MAXALIGN(sizeof(BloomPageOpaqueData)) - - MAXALIGN(sizeof(uint16) * 2 + sizeof(uint32) + sizeof(BloomOptions)) - ) / sizeof(BlockNumber) -]; +typedef BlockNumber FreeBlockNumberArray[MAXALIGN_DOWN(BLCKSZ - SizeOfPageHeaderData - MAXALIGN(sizeof(BloomPageOpaqueData)) + - MAXALIGN(sizeof(uint16) * 2 + sizeof(uint32) + sizeof(BloomOptions))) + / sizeof(BlockNumber)]; /* Metadata of bloom index */ typedef struct BloomMetaPageData diff --git a/src/backend/backup/basebackup_incremental.c b/src/backend/backup/basebackup_incremental.c index 15082c29878..ceec1f44312 100644 --- a/src/backend/backup/basebackup_incremental.c +++ b/src/backend/backup/basebackup_incremental.c @@ -207,8 +207,8 @@ AppendIncrementalManifestData(IncrementalBackupInfo *ib, const char *data, * time for an incremental parse. We'll do all but the last MIN_CHUNK * so that we have enough left for the final piece. */ - json_parse_manifest_incremental_chunk( - ib->inc_state, ib->buf.data, ib->buf.len - MIN_CHUNK, false); + json_parse_manifest_incremental_chunk(ib->inc_state, ib->buf.data, + ib->buf.len - MIN_CHUNK, false); /* now remove what we just parsed */ memmove(ib->buf.data, ib->buf.data + (ib->buf.len - MIN_CHUNK), MIN_CHUNK + 1); @@ -234,8 +234,8 @@ FinalizeIncrementalManifest(IncrementalBackupInfo *ib) oldcontext = MemoryContextSwitchTo(ib->mcxt); /* Parse the last chunk of the manifest */ - json_parse_manifest_incremental_chunk( - ib->inc_state, ib->buf.data, ib->buf.len, true); + json_parse_manifest_incremental_chunk(ib->inc_state, ib->buf.data, + ib->buf.len, true); /* Done with the buffer, so release memory. */ pfree(ib->buf.data); diff --git a/src/backend/utils/init/postinit.c b/src/backend/utils/init/postinit.c index adc21e019d5..e4ac84354ef 100644 --- a/src/backend/utils/init/postinit.c +++ b/src/backend/utils/init/postinit.c @@ -431,8 +431,7 @@ CheckMyDatabase(const char *name, bool am_superuser, bool override_allow_connect builtin_validate_locale(dbform->encoding, datlocale); - default_locale.info.builtin.locale = MemoryContextStrdup( - TopMemoryContext, datlocale); + default_locale.info.builtin.locale = MemoryContextStrdup(TopMemoryContext, datlocale); } else if (dbform->datlocprovider == COLLPROVIDER_ICU) { diff --git a/src/bin/pg_combinebackup/load_manifest.c b/src/bin/pg_combinebackup/load_manifest.c index be8e6273fcb..298c883c17f 100644 --- a/src/bin/pg_combinebackup/load_manifest.c +++ b/src/bin/pg_combinebackup/load_manifest.c @@ -204,8 +204,7 @@ load_backup_manifest(char *backup_directory) (long long int) statbuf.st_size); } bytes_left -= rc; - json_parse_manifest_incremental_chunk( - inc_state, buffer, rc, bytes_left == 0); + json_parse_manifest_incremental_chunk(inc_state, buffer, rc, bytes_left == 0); } /* Release the incremental state memory */ diff --git a/src/common/parse_manifest.c b/src/common/parse_manifest.c index 5a7b491e9a9..bc7eebb06ae 100644 --- a/src/common/parse_manifest.c +++ b/src/common/parse_manifest.c @@ -182,9 +182,8 @@ json_parse_manifest_incremental_shutdown(JsonManifestParseIncrementalState *incs */ void -json_parse_manifest_incremental_chunk( - JsonManifestParseIncrementalState *incstate, const char *chunk, size_t size, - bool is_last) +json_parse_manifest_incremental_chunk(JsonManifestParseIncrementalState *incstate, + const char *chunk, size_t size, bool is_last) { JsonParseErrorType res, expected; diff --git a/src/common/unicode/category_test.c b/src/common/unicode/category_test.c index e823044d63a..ba6aa7be574 100644 --- a/src/common/unicode/category_test.c +++ b/src/common/unicode/category_test.c @@ -74,22 +74,14 @@ icu_test() bool prop_hex_digit = pg_u_prop_hex_digit(code); bool prop_join_control = pg_u_prop_join_control(code); - bool icu_prop_alphabetic = u_hasBinaryProperty( - code, UCHAR_ALPHABETIC); - bool icu_prop_lowercase = u_hasBinaryProperty( - code, UCHAR_LOWERCASE); - bool icu_prop_uppercase = u_hasBinaryProperty( - code, UCHAR_UPPERCASE); - bool icu_prop_cased = u_hasBinaryProperty( - code, UCHAR_CASED); - bool icu_prop_case_ignorable = u_hasBinaryProperty( - code, UCHAR_CASE_IGNORABLE); - bool icu_prop_white_space = u_hasBinaryProperty( - code, UCHAR_WHITE_SPACE); - bool icu_prop_hex_digit = u_hasBinaryProperty( - code, UCHAR_HEX_DIGIT); - bool icu_prop_join_control = u_hasBinaryProperty( - code, UCHAR_JOIN_CONTROL); + bool icu_prop_alphabetic = u_hasBinaryProperty(code, UCHAR_ALPHABETIC); + bool icu_prop_lowercase = u_hasBinaryProperty(code, UCHAR_LOWERCASE); + bool icu_prop_uppercase = u_hasBinaryProperty(code, UCHAR_UPPERCASE); + bool icu_prop_cased = u_hasBinaryProperty(code, UCHAR_CASED); + bool icu_prop_case_ignorable = u_hasBinaryProperty(code, UCHAR_CASE_IGNORABLE); + bool icu_prop_white_space = u_hasBinaryProperty(code, UCHAR_WHITE_SPACE); + bool icu_prop_hex_digit = u_hasBinaryProperty(code, UCHAR_HEX_DIGIT); + bool icu_prop_join_control = u_hasBinaryProperty(code, UCHAR_JOIN_CONTROL); /* * Compare with ICU for character classes using: diff --git a/src/include/common/parse_manifest.h b/src/include/common/parse_manifest.h index ee571a568a1..ce5cafa9661 100644 --- a/src/include/common/parse_manifest.h +++ b/src/include/common/parse_manifest.h @@ -50,8 +50,8 @@ struct JsonManifestParseContext extern void json_parse_manifest(JsonManifestParseContext *context, const char *buffer, size_t size); extern JsonManifestParseIncrementalState *json_parse_manifest_incremental_init(JsonManifestParseContext *context); -extern void json_parse_manifest_incremental_chunk( - JsonManifestParseIncrementalState *incstate, const char *chunk, size_t size, +extern void json_parse_manifest_incremental_chunk(JsonManifestParseIncrementalState *incstate, + const char *chunk, size_t size, bool is_last); extern void json_parse_manifest_incremental_shutdown(JsonManifestParseIncrementalState *incstate); diff --git a/src/include/replication/reorderbuffer.h b/src/include/replication/reorderbuffer.h index 851a001c8bb..7de50462dcf 100644 --- a/src/include/replication/reorderbuffer.h +++ b/src/include/replication/reorderbuffer.h @@ -478,45 +478,38 @@ typedef void (*ReorderBufferRollbackPreparedCB) (ReorderBuffer *rb, TimestampTz prepare_time); /* start streaming transaction callback signature */ -typedef void (*ReorderBufferStreamStartCB) ( - ReorderBuffer *rb, +typedef void (*ReorderBufferStreamStartCB) (ReorderBuffer *rb, ReorderBufferTXN *txn, XLogRecPtr first_lsn); /* stop streaming transaction callback signature */ -typedef void (*ReorderBufferStreamStopCB) ( - ReorderBuffer *rb, +typedef void (*ReorderBufferStreamStopCB) (ReorderBuffer *rb, ReorderBufferTXN *txn, XLogRecPtr last_lsn); /* discard streamed transaction callback signature */ -typedef void (*ReorderBufferStreamAbortCB) ( - ReorderBuffer *rb, +typedef void (*ReorderBufferStreamAbortCB) (ReorderBuffer *rb, ReorderBufferTXN *txn, XLogRecPtr abort_lsn); /* prepare streamed transaction callback signature */ -typedef void (*ReorderBufferStreamPrepareCB) ( - ReorderBuffer *rb, +typedef void (*ReorderBufferStreamPrepareCB) (ReorderBuffer *rb, ReorderBufferTXN *txn, XLogRecPtr prepare_lsn); /* commit streamed transaction callback signature */ -typedef void (*ReorderBufferStreamCommitCB) ( - ReorderBuffer *rb, +typedef void (*ReorderBufferStreamCommitCB) (ReorderBuffer *rb, ReorderBufferTXN *txn, XLogRecPtr commit_lsn); /* stream change callback signature */ -typedef void (*ReorderBufferStreamChangeCB) ( - ReorderBuffer *rb, +typedef void (*ReorderBufferStreamChangeCB) (ReorderBuffer *rb, ReorderBufferTXN *txn, Relation relation, ReorderBufferChange *change); /* stream message callback signature */ -typedef void (*ReorderBufferStreamMessageCB) ( - ReorderBuffer *rb, +typedef void (*ReorderBufferStreamMessageCB) (ReorderBuffer *rb, ReorderBufferTXN *txn, XLogRecPtr message_lsn, bool transactional, @@ -524,16 +517,14 @@ typedef void (*ReorderBufferStreamMessageCB) ( const char *message); /* stream truncate callback signature */ -typedef void (*ReorderBufferStreamTruncateCB) ( - ReorderBuffer *rb, +typedef void (*ReorderBufferStreamTruncateCB) (ReorderBuffer *rb, ReorderBufferTXN *txn, int nrelations, Relation relations[], ReorderBufferChange *change); /* update progress txn callback signature */ -typedef void (*ReorderBufferUpdateProgressTxnCB) ( - ReorderBuffer *rb, +typedef void (*ReorderBufferUpdateProgressTxnCB) (ReorderBuffer *rb, ReorderBufferTXN *txn, XLogRecPtr lsn);