This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

windres bug fix


Hi Guys,

  DJ very kindly regenerated the recently created *.rsd files in the
  windres testsuite using Microsoft's tools, rather than windres.
  This exposed a small bug in rcparse.y - the MEMFLAG_PURE was not
  being set by default, so that patch below fixes this and checks in
  DJ's regenerated files.

Cheers
        Nick

ChangeLog
2002-04-10  Nick Clifton  <nickc@cambridge.redhat.com>

	* rcparse.y: Set MEMFLAG_PURE by default.

testsuite/ChangeLog
2002-04-10  Nick Clifton  <nickc@cambridge.redhat.com>

	* binutils-all/windres/capstyle.rsd: Regernate using MSVC.
	* binutils-all/windres/deflang.rsd: Regernate using MSVC.
	* binutils-all/windres/dialogsignature.rsd: Regernate using MSVC.
	* binutils-all/windres/escapea.rsd: Regernate using MSVC.
	* binutils-all/windres/escapex.rsd: Regernate using MSVC.
	* binutils-all/windres/nocaption.rsd: Regernate using MSVC.
	* binutils-all/windres/printstyle.rsd: Regernate using MSVC.
	* binutils-all/windres/sublang.rsd: Regernate using MSVC.

Index: binutils/rcparse.y
===================================================================
RCS file: /cvs/src/src/binutils/rcparse.y,v
retrieving revision 1.11
diff -c -3 -p -w -r1.11 rcparse.y
*** binutils/rcparse.y	9 Apr 2002 17:23:18 -0000	1.11
--- binutils/rcparse.y	10 Apr 2002 08:10:45 -0000
*************** suboptions:
*** 1320,1326 ****
  	    memset (&$$, 0, sizeof (struct res_res_info));
  	    $$.language = language;
  	    /* FIXME: Is this the right default?  */
! 	    $$.memflags = MEMFLAG_MOVEABLE;
  	  }
  	| suboptions memflag
  	  {
--- 1320,1326 ----
  	    memset (&$$, 0, sizeof (struct res_res_info));
  	    $$.language = language;
  	    /* FIXME: Is this the right default?  */
! 	    $$.memflags = MEMFLAG_MOVEABLE | MEMFLAG_PURE;
  	  }
  	| suboptions memflag
  	  {
*************** memflags_move:
*** 1369,1375 ****
  	  {
  	    memset (&$$, 0, sizeof (struct res_res_info));
  	    $$.language = language;
! 	    $$.memflags = MEMFLAG_MOVEABLE;
  	  }
  	| memflags_move memflag
  	  {
--- 1369,1375 ----
  	  {
  	    memset (&$$, 0, sizeof (struct res_res_info));
  	    $$.language = language;
! 	    $$.memflags = MEMFLAG_MOVEABLE | MEMFLAG_PURE;
  	  }
  	| memflags_move memflag
  	  {


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]