Re: backup manifests - Mailing list pgsql-hackers

Thank you, Robert.

Getting below warning while compiling the v11-0003-pg_validatebackup-Validate-a-backup-against-the-.patch.

pg_validatebackup.c: In function ‘report_manifest_error’:
pg_validatebackup.c:356:2: warning: function might be possible candidate for ‘gnu_printf’ format attribute [-Wsuggest-attribute=format]
  pg_log_generic_v(PG_LOG_FATAL, fmt, ap);
 

To resolve this, can we use "pg_attribute_printf(2, 3)" in function declaration something like below?
e.g:

diff --git a/src/bin/pg_validatebackup/parse_manifest.h b/src/bin/pg_validatebackup/parse_manifest.h
index b0b18a5..25d140f 100644
--- a/src/bin/pg_validatebackup/parse_manifest.h
+++ b/src/bin/pg_validatebackup/parse_manifest.h
@@ -25,7 +25,7 @@ typedef void (*json_manifest_perfile_callback)(JsonManifestParseContext *,
                                                                 size_t size, pg_checksum_type checksum_type,
                                                                 int checksum_length, uint8 *checksum_payload);
 typedef void (*json_manifest_error_callback)(JsonManifestParseContext *,
-                                                                char *fmt, ...);
+                                                                char *fmt,...) pg_attribute_printf(2, 3);
 
 struct JsonManifestParseContext
 {
diff --git a/src/bin/pg_validatebackup/pg_validatebackup.c b/src/bin/pg_validatebackup/pg_validatebackup.c
index 0e7299b..6ccbe59 100644
--- a/src/bin/pg_validatebackup/pg_validatebackup.c
+++ b/src/bin/pg_validatebackup/pg_validatebackup.c
@@ -95,7 +95,7 @@ static void record_manifest_details_for_file(JsonManifestParseContext *context,
                                                                                         int checksum_length,
                                                                                         uint8 *checksum_payload);
 static void report_manifest_error(JsonManifestParseContext *context,
-                                                                 char *fmt, ...);
+                                                                 char *fmt,...) pg_attribute_printf(2, 3);
 
 static void validate_backup_directory(validator_context *context,
                                                                          char *relpath, char *fullpath);


Typos:

0004 patch
unexpctedly => unexpectedly

0005 patch
bacup => backup