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]

Re: [PATCH] Fix -save-temps from clobbering input file@apple.com


At 10:10 PM +0100 11/11/01, Andreas Schwab wrote:

>Casts are bad if you can easily avoid them, since they can hide bugs.

I do not see how EXPLICIT casts can hide a bug any more than using
a temp variable made from a cast can hide bugs.  Indeed the latter is
harder to detect when reading code than the explicit case.  But I
am not going to get into a debate with you here.  In the context of
THIS patch it is not worth creating unnecessary temporary variables
and I have no intention of doing so! 

It's one thing to keep resubmitting a patch for some technical reason
like I did with the -### patch.  It another to resubmit because it's
doesn't agree with one's personal style.  It's either accepted as it
stands or it is not (barring, like I said, for some technical reason).
Also see below.


>|> Let's not get into a stylistic debate here.
>
>Please read <http://gcc.gnu.org/codingconventions.html>, this is on-topic.

I see nothing there or <http://www.gnu.org/prep/standards_toc.html> that precludes casts.

Ira


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