windres bug fix

Nick Clifton nickc@cambridge.redhat.com
Wed Apr 10 01:17:00 GMT 2002


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
  	  {



More information about the Binutils mailing list