libiberty warning patch

Jeffrey A Law law@cygnus.com
Tue Aug 31 22:41:00 GMT 1999


  In message < 199908041503.LAA18233@caip.rutgers.edu >you write:
  > 	The following warnings from the gcc directory:
  > 
  >  > cxxmain.c: In function `gnu_special':
  >  > cxxmain.c:2451: warning: comparison between signed and unsigned
  >  > cxxmain.c: In function `arm_special':
  >  > cxxmain.c:2618: warning: comparison between signed and unsigned
  >  > cxxmain.c: In function `demangle_fund_type':
  >  > cxxmain.c:3384: warning: comparison between signed and unsigned
  >  > cxxmain.c: In function `do_hpacc_template_const_value':
  >  > cxxmain.c:3453: warning: unused parameter `work'
  >  > cxxmain.c: In function `main':
  >  > cxxmain.c:4527: warning: assignment discards qualifiers from pointer
  >  > 		target type
  >  > cxxmain.c:4530: warning: assignment discards qualifiers from pointer
  >  > 		target type
  > 
  > 
  > are due to the file libiberty/cplus-dem.c, (which cxxmain.c is a
  > symlink to.)
  > 
  > Here is a patch to clean them up.  Note libiberty has the ATTRIBUTE_*
  > stuff now so its safe to use in a file compiled in both directories.
  > 
  > 	Okay to install?
  > 
  > 		Thanks,
  > 		--Kaveh
  > 
  > 
  > 1999-08-04  Kaveh R. Ghazi  <ghazi@caip.rutgers.edu>
  > 
  > 	* cplus-dem.c (gnu_special): Cast a `size_t' to `long' when
  > 	comparing against a signed quantity.
  > 	(arm_special): Likewise.
  > 	(demangle_fund_type): Likewise.
  > 	(do_hpacc_template_const_value): Mark parameter `work' with
  > 	ATTRIBUTE_UNUSED.	
  > 	(main): Constify variable `valid_symbols'.
This is fine.  Please install it.

Thanks,
jeff



More information about the Gcc-patches mailing list