You might want to run pg_indent on pg_backup_db.c however ...

tom lane

src/bin/pg_dump/pg_backup_archiver.c

@@ -15,7 +15,7 @@
  *
  *
  * IDENTIFICATION
- *		$Header: /cvsroot/pgsql/src/bin/pg_dump/pg_backup_archiver.c,v 1.39 2002/01/18 17:13:50 tgl Exp $
+ *		$Header: /cvsroot/pgsql/src/bin/pg_dump/pg_backup_archiver.c,v 1.40 2002/01/18 19:17:04 momjian Exp $
  *
  * Modifications - 28-Jun-2000 - pjw@rhyme.com.au
  *
@@ -1922,12 +1922,12 @@ _tocEntryRequired(TocEntry *te, RestoreOptions *ropt)
 			res = res & ~REQ_DATA;
 	}
 
-    /* Special case: <Init> type with <Max OID> name; this is part of
+	/*
-     * a DATA restore even though it has SQL.
+	 * Special case: <Init> type with <Max OID> name; this is part of a
+	 * DATA restore even though it has SQL.
 	 */
-	if (  ( strcmp(te->desc, "<Init>") == 0 ) && ( strcmp(te->name, "Max OID") == 0) ) {
+	if ((strcmp(te->desc, "<Init>") == 0) && (strcmp(te->name, "Max OID") == 0))
 		res = REQ_DATA;
-	}
 
 	/* Mask it if we only want schema */
 	if (ropt->schemaOnly)

src/bin/pg_dump/pg_backup_db.c

@@ -5,7 +5,7 @@
  *	Implements the basic DB functions used by the archiver.
  *
  * IDENTIFICATION
- *	  $Header: /cvsroot/pgsql/src/bin/pg_dump/pg_backup_db.c,v 1.30 2002/01/18 17:13:51 tgl Exp $
+ *	  $Header: /cvsroot/pgsql/src/bin/pg_dump/pg_backup_db.c,v 1.31 2002/01/18 19:17:05 momjian Exp $
  *
  * NOTES
  *
@@ -556,7 +556,8 @@ _sendCopyLine( ArchiveHandle *AH, char *qry, char *eos)
 	int			isEnd = 0;
 
 	/* loop to find unquoted newline ending the line of COPY data */
-    for (;;) {
+	for (;;)
+	{
 		loc = strcspn(&qry[pos], "\n") + pos;
 
 		/* If no match, then wait */
@@ -579,8 +580,8 @@ _sendCopyLine( ArchiveHandle *AH, char *qry, char *eos)
 		sPos = loc - sPos;
 
 		/*
-		 * If an odd number of preceding slashes, then \n was escaped
+		 * If an odd number of preceding slashes, then \n was escaped so
-		 * so set the next search pos, and loop (if any left).
+		 * set the next search pos, and loop (if any left).
 		 */
 		if ((sPos & 1) == 1)
 		{
@@ -591,9 +592,9 @@ _sendCopyLine( ArchiveHandle *AH, char *qry, char *eos)
 				appendBinaryPQExpBuffer(AH->pgCopyBuf, qry, (eos - qry));
 				return eos;
 			}
-		} else {
-			break;
 		}
+		else
+			break;
 	}
 
 	/* We found an unquoted newline */
@@ -613,8 +614,7 @@ _sendCopyLine( ArchiveHandle *AH, char *qry, char *eos)
 	resetPQExpBuffer(AH->pgCopyBuf);
 
 	/*
-	 * fprintf(stderr, "Buffer is '%s'\n",
+	 * fprintf(stderr, "Buffer is '%s'\n", AH->pgCopyBuf->data);
-	 * AH->pgCopyBuf->data);
 	 */
 
 	if (isEnd)
@@ -637,15 +637,14 @@ _sendSQLLine( ArchiveHandle *AH, char *qry, char *eos)
 	int			pos = 0;		/* Current position */
 
 	/*
-	 * The following is a mini state machine to assess the end of an
+	 * The following is a mini state machine to assess the end of an SQL
-	 * SQL statement. It really only needs to parse good SQL, or at
+	 * statement. It really only needs to parse good SQL, or at least
-	 * least that's the theory... End-of-statement is assumed to be an
+	 * that's the theory... End-of-statement is assumed to be an unquoted,
-	 * unquoted, un commented semi-colon.
+	 * un commented semi-colon.
 	 */
 
 	/*
-	 * fprintf(stderr, "Buffer at start is: '%s'\n\n",
+	 * fprintf(stderr, "Buffer at start is: '%s'\n\n", AH->sqlBuf->data);
-	 * AH->sqlBuf->data);
 	 */
 
 	for (pos = 0; pos < (eos - qry); pos++)
@@ -670,8 +669,9 @@ _sendSQLLine( ArchiveHandle *AH, char *qry, char *eos)
 					resetPQExpBuffer(AH->sqlBuf);
 					AH->sqlparse.lastChar = '\0';
 
-					/* Remove any following newlines - so that embedded COPY commands don't get a 
+					/*
-					 * starting newline.
+					 * Remove any following newlines - so that embedded
+					 * COPY commands don't get a starting newline.
 					 */
 					pos++;
 					for (; pos < (eos - qry) && qry[pos] == '\n'; pos++);
@@ -741,7 +741,10 @@ _sendSQLLine( ArchiveHandle *AH, char *qry, char *eos)
 		/* fprintf(stderr, "\n"); */
 	}
 
-    /* If we get here, we've processed entire string with no complete SQL stmt */
+	/*
+	 * If we get here, we've processed entire string with no complete SQL
+	 * stmt
+	 */
 	return eos;
 
 }
@@ -762,12 +765,11 @@ ExecuteSqlCommandBuf(ArchiveHandle *AH, void *qryv, int bufLen)
 	/* Could switch between command and COPY IN mode at each line */
 	while (qry < eos)
 	{
-		if (AH->pgCopyIn) {
+		if (AH->pgCopyIn)
 			qry = _sendCopyLine(AH, qry, eos);
-		} else {
+		else
 			qry = _sendSQLLine(AH, qry, eos);
 	}
-	}
 
 	return 1;
 }

src/bin/pg_dump/pg_dump.c

@@ -22,7 +22,7 @@
  *
  *
  * IDENTIFICATION
- *	  $Header: /cvsroot/pgsql/src/bin/pg_dump/pg_dump.c,v 1.237 2002/01/11 23:21:55 tgl Exp $
+ *	  $Header: /cvsroot/pgsql/src/bin/pg_dump/pg_dump.c,v 1.238 2002/01/18 19:17:05 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -2127,14 +2127,15 @@ getTables(int *numTables, FuncInfo *finfo, int numFuncs, const char* tablename)
 
 	/*
 	 * First pass: extract data from result and lock tables.  We do the
-	 * locking before anything else, to minimize the window wherein a table
+	 * locking before anything else, to minimize the window wherein a
-	 * could disappear under us.
+	 * table could disappear under us.
 	 *
 	 * Note that we have to collect info about all tables here, even when
 	 * dumping only one, because we don't know which tables might be
 	 * inheritance ancestors of the target table.  Possible future
 	 * improvement: suppress later collection of schema info about tables
-	 * that are determined not to be either targets or ancestors of targets.
+	 * that are determined not to be either targets or ancestors of
+	 * targets.
 	 */
 	tblinfo = (TableInfo *) malloc(ntups * sizeof(TableInfo));
 
@@ -2167,12 +2168,13 @@ getTables(int *numTables, FuncInfo *finfo, int numFuncs, const char* tablename)
 		 *
 		 * If no target tablename was specified, lock all tables we see,
 		 * otherwise lock only the specified table.  (This is incomplete
-		 * because we'll still try to collect schema info about all tables,
+		 * because we'll still try to collect schema info about all
-		 * and could possibly lose during that phase.  But for the typical
+		 * tables, and could possibly lose during that phase.  But for the
-		 * use where we're dumping all tables anyway, it matters not.)
+		 * typical use where we're dumping all tables anyway, it matters
+		 * not.)
 		 *
-		 * NOTE: it'd be kinda nice to lock views and sequences too, not
+		 * NOTE: it'd be kinda nice to lock views and sequences too, not only
-		 * only plain tables, but the backend doesn't presently allow that.
+		 * plain tables, but the backend doesn't presently allow that.
 		 */
 		if ((tblinfo[i].relkind == RELKIND_RELATION) &&
 		(tablename == NULL || strcmp(tblinfo[i].relname, tablename) == 0))
@@ -2198,8 +2200,8 @@ getTables(int *numTables, FuncInfo *finfo, int numFuncs, const char* tablename)
 	res = NULL;
 
 	/*
-	 * Second pass: pick up additional information about each table,
+	 * Second pass: pick up additional information about each table, as
-	 * as required.
+	 * required.
 	 */
 	for (i = 0; i < *numTables; i++)
 	{

src/bin/pg_dump/pg_restore.c

@@ -34,7 +34,7 @@
  *
  *
  * IDENTIFICATION
- *		$Header: /cvsroot/pgsql/src/bin/pg_dump/pg_restore.c,v 1.32 2002/01/10 01:11:45 tgl Exp $
+ *		$Header: /cvsroot/pgsql/src/bin/pg_dump/pg_restore.c,v 1.33 2002/01/18 19:17:05 momjian Exp $
  *
  * Modifications - 28-Jun-2000 - pjw@rhyme.com.au
  *