Re: [HACKERS] Current sources?

> I wouldn't say that the patch should be removed from the CVS. I just
> need my *private* working snapshot to make sure my patches would not
> break anything before submitting.
>
> I wish I could solve the problem by myself, but spending a few hours
> for debugging last Sunday, I have not find fixes for that yet. Sorry.

OK, here is a fix I have just applied to the tree.  It appears to meet
the intent of the original patch.  David H. will have to comment on its
accuracy.

---------------------------------------------------------------------------


Index: backend/executor/execMain.c
===================================================================
RCS file: /usr/local/cvsroot/pgsql/src/backend/executor/execMain.c,v
retrieving revision 1.46
diff -c -r1.46 execMain.c
*** execMain.c    1998/05/21 03:53:50    1.46
--- execMain.c    1998/05/26 03:33:22
***************
*** 530,536 ****
          JunkFilter *j = (JunkFilter *) ExecInitJunkFilter(targetList);
          estate->es_junkFilter = j;

!         tupType = j->jf_cleanTupType;       /*  Added by daveh@insightdist.com  5/20/98   */
      }
      else
          estate->es_junkFilter = NULL;
--- 530,537 ----
          JunkFilter *j = (JunkFilter *) ExecInitJunkFilter(targetList);
          estate->es_junkFilter = j;

!         if (operation == CMD_SELECT)
!             tupType = j->jf_cleanTupType;
      }
      else
          estate->es_junkFilter = NULL;


--
Bruce Momjian                          |  830 Blythe Avenue
maillist@candle.pha.pa.us              |  Drexel Hill, Pennsylvania 19026
  +  If your life is a hard drive,     |  (610) 353-9879(w)
  +  Christ can be your backup.        |  (610) 853-3000(h)