Suppress possibly-uninitialized-variable warnings from gcc 4.5.

It appears that gcc 4.5 can issue such warnings for whole structs, not
just scalar variables as in the past.  Refactor some pg_dump code slightly
so that the OutputContext local variables are always initialized, even
if they won't be used.  It's cheap enough to not be worth worrying about.

diff --git a/src/bin/pg_dump/pg_backup_archiver.c b/src/bin/pg_dump/pg_backup_archiver.c
index 64d8d93dda81805540ac70dbd7afcf11390c19fc..e230c4e1b00dd52ee3b39e82813f8191fc9f8ec9 100644 (file)
--- a/src/bin/pg_dump/pg_backup_archiver.c
+++ b/src/bin/pg_dump/pg_backup_archiver.c
@@ -75,6 +75,13 @@ typedef struct _parallel_slot
 
 #define NO_SLOT (-1)
 
+/* state needed to save/restore an archive's output target */
+typedef struct _outputContext
+{
+       void       *OF;
+       int                     gzOut;
+} OutputContext;
+
 const char *progname;
 
 static const char *modulename = gettext_noop("archiver");
@@ -114,8 +121,9 @@ static void _write_msg(const char *modulename, const char *fmt, va_list ap);
 static void _die_horribly(ArchiveHandle *AH, const char *modulename, const char *fmt, va_list ap);
 
 static void dumpTimestamp(ArchiveHandle *AH, const char *msg, time_t tim);
-static OutputContext SetOutput(ArchiveHandle *AH, char *filename, int compression);
-static void ResetOutput(ArchiveHandle *AH, OutputContext savedContext);
+static void SetOutput(ArchiveHandle *AH, char *filename, int compression);
+static OutputContext SaveOutput(ArchiveHandle *AH);
+static void RestoreOutput(ArchiveHandle *AH, OutputContext savedContext);
 
 static int restore_toc_entry(ArchiveHandle *AH, TocEntry *te,
                                  RestoreOptions *ropt, bool is_parallel);
@@ -299,8 +307,9 @@ RestoreArchive(Archive *AHX, RestoreOptions *ropt)
        /*
         * Setup the output file if necessary.
         */
+       sav = SaveOutput(AH);
        if (ropt->filename || ropt->compression)
-               sav = SetOutput(AH, ropt->filename, ropt->compression);
+               SetOutput(AH, ropt->filename, ropt->compression);
 
        ahprintf(AH, "--\n-- PostgreSQL database dump\n--\n\n");
 
@@ -420,7 +429,7 @@ RestoreArchive(Archive *AHX, RestoreOptions *ropt)
        AH->stage = STAGE_FINALIZING;
 
        if (ropt->filename || ropt->compression)
-               ResetOutput(AH, sav);
+               RestoreOutput(AH, sav);
 
        if (ropt->useDB)
        {
@@ -782,8 +791,9 @@ PrintTOCSummary(Archive *AHX, RestoreOptions *ropt)
        OutputContext sav;
        char       *fmtName;
 
+       sav = SaveOutput(AH);
        if (ropt->filename)
-               sav = SetOutput(AH, ropt->filename, 0 /* no compression */ );
+               SetOutput(AH, ropt->filename, 0 /* no compression */ );
 
        ahprintf(AH, ";\n; Archive created at %s", ctime(&AH->createDate));
        ahprintf(AH, ";     dbname: %s\n;     TOC Entries: %d\n;     Compression: %d\n",
@@ -839,7 +849,7 @@ PrintTOCSummary(Archive *AHX, RestoreOptions *ropt)
        }
 
        if (ropt->filename)
-               ResetOutput(AH, sav);
+               RestoreOutput(AH, sav);
 }
 
 /***********
@@ -1117,16 +1127,11 @@ archprintf(Archive *AH, const char *fmt,...)
  * Stuff below here should be 'private' to the archiver routines
  *******************************/
 
-static OutputContext
+static void
 SetOutput(ArchiveHandle *AH, char *filename, int compression)
 {
-       OutputContext sav;
        int                     fn;
 
-       /* Replace the AH output file handle */
-       sav.OF = AH->OF;
-       sav.gzOut = AH->gzOut;
-
        if (filename)
                fn = -1;
        else if (AH->FH)
@@ -1182,12 +1187,21 @@ SetOutput(ArchiveHandle *AH, char *filename, int compression)
                        die_horribly(AH, modulename, "could not open output file: %s\n",
                                                 strerror(errno));
        }
+}
+
+static OutputContext
+SaveOutput(ArchiveHandle *AH)
+{
+       OutputContext sav;
+
+       sav.OF = AH->OF;
+       sav.gzOut = AH->gzOut;
 
        return sav;
 }
 
 static void
-ResetOutput(ArchiveHandle *AH, OutputContext sav)
+RestoreOutput(ArchiveHandle *AH, OutputContext savedContext)
 {
        int                     res;
 
@@ -1200,8 +1214,8 @@ ResetOutput(ArchiveHandle *AH, OutputContext sav)
                die_horribly(AH, modulename, "could not close output file: %s\n",
                                         strerror(errno));
 
-       AH->gzOut = sav.gzOut;
-       AH->OF = sav.OF;
+       AH->gzOut = savedContext.gzOut;
+       AH->OF = savedContext.OF;
 }
 
 

diff --git a/src/bin/pg_dump/pg_backup_archiver.h b/src/bin/pg_dump/pg_backup_archiver.h
index d9378df55f3ad95a971af6bafb1e37b53d8bbd61..502e7410a1b55b256938087daa8bda60ba8eb6e2 100644 (file)
--- a/src/bin/pg_dump/pg_backup_archiver.h
+++ b/src/bin/pg_dump/pg_backup_archiver.h
@@ -132,12 +132,6 @@ typedef void (*DeClonePtr) (struct _archiveHandle * AH);
 
 typedef size_t (*CustomOutPtr) (struct _archiveHandle * AH, const void *buf, size_t len);
 
-typedef struct _outputContext
-{
-       void       *OF;
-       int                     gzOut;
-} OutputContext;
-
 typedef enum
 {
        SQL_SCAN = 0,                           /* normal */