This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB 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: [patch] Delete #if 0'd out code in language.c


> 2011-12-28  Doug Evans  <dje@google.com>
> 
> 	Delete #if 0'd out code.
> 	* language.c (binop_result_type): Delete.
> 	(simple_type, ordered_type, same_type, integral_type): Delete.
> 	(numeric_type, character_type, string_type, boolean_type): Delete.
> 	(float_type, structured_type): Delete.
> 	* language.h: Update.

I wouldn't wait. Any commented out code, especially code that has
been commented out for 10 years, is fair game for instant removal
as an obvious change (IMO).  My reason is that it's easy to put back
any part of it should the need arise again.

-- 
Joel


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