This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC 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]

Re: [C++ PATCH] Fix 21280


Nathan Sidwell wrote:
this patch fixes a regression with synthesized functions and #pragma interface. We were mistakingly thinking a synthesized function's interface status was determined by where the function was first needed.

When tweaking the patch so it applied to mainline, I realised the new error message should really be a warning. So this is how I've amended the 4.0 branch.


nathan

--
Nathan Sidwell    ::   http://www.codesourcery.com   ::     CodeSourcery LLC
nathan@codesourcery.com    ::     http://www.planetfall.pwp.blueyonder.co.uk

2005-06-02  Nathan Sidwell  <nathan@codesourcery.com>

	* method.c (synthesize_method): Change synthesize error to warning
	for previous patch.

Index: cp/method.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/cp/method.c,v
retrieving revision 1.322.4.3
diff -c -3 -p -r1.322.4.3 method.c
*** cp/method.c	2 Jun 2005 17:31:03 -0000	1.322.4.3
--- cp/method.c	2 Jun 2005 17:56:10 -0000
*************** synthesize_method (tree fndecl)
*** 777,784 ****
    pop_deferring_access_checks ();
  
    if (error_count != errorcount || warning_count != warningcount)
!     error ("%Hsynthesized method %qD first required here ",
! 	   &input_location, fndecl);
  }
  
  /* Use EXTRACTOR to locate the relevant function called for each base &
--- 777,784 ----
    pop_deferring_access_checks ();
  
    if (error_count != errorcount || warning_count != warningcount)
!     warning ("%Hsynthesized method %qD first required here ",
! 	     &input_location, fndecl);
  }
  
  /* Use EXTRACTOR to locate the relevant function called for each base &

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