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

---------------------------(end of broadcast)---------------------------
TIP 1: if posting/reading through Usenet, please send an appropriate
       subscribe-nomail command to [EMAIL PROTECTED] so that your
       message can get through to the mailing list cleanly

Reply via email to