Since we were breaking our usual rule by re-indenting the 8.1 branch,
I took the time to eyeball the whole "cvs diff" for changes that weren't
just comment block fixes. I found a few things that need attention.
This change is disturbing first because it seems completely unnecessary,
and second because I'm not convinced that every C preprocessor will deal
correctly with a comment continued off a #endif line:
Index: contrib/pgbench/pgbench.c
***************
*** 1110,1116 **** fprintf(stderr, "Use limit/ulimt to increase the limit before using
pgbench.\n"); exit(1); }
! #endif /* #if !(defined(__CYGWIN__) || defined(__MINGW32__)) */ break; case 'C':
is_connect = 1;
--- 1110,1117 ---- fprintf(stderr, "Use limit/ulimt to increase the limit before using
pgbench.\n"); exit(1); }
! #endif /* #if !(defined(__CYGWIN__) ||
! * defined(__MINGW32__)) */ break; case 'C':
is_connect = 1;
This change seems odd and unnecessary as well:
Index: src/backend/access/transam/slru.c
***************
*** 252,258 **** /* indeed, the I/O must have failed */ if (shared->page_status[slotno] ==
SLRU_PAGE_READ_IN_PROGRESS) shared->page_status[slotno] = SLRU_PAGE_EMPTY;
! else /* write_in_progress */ { shared->page_status[slotno] =
SLRU_PAGE_VALID; shared->page_dirty[slotno] = true;
--- 253,260 ---- /* indeed, the I/O must have failed */ if (shared->page_status[slotno] ==
SLRU_PAGE_READ_IN_PROGRESS) shared->page_status[slotno] = SLRU_PAGE_EMPTY;
! else
! /* write_in_progress */ { shared->page_status[slotno] = SLRU_PAGE_VALID;
shared->page_dirty[slotno] = true;
And what happened here?
Index: src/interfaces/libpq/libpq-fe.h
***************
*** 35,41 **** /* Application-visible enum types */
! typedef enum { /* * Although it is okay to add to this list, values which become unused
--- 35,41 ---- /* Application-visible enum types */
! typedef enum { /* * Although it is okay to add to this list, values which become unused
regards, tom lane