This is the mail archive of the binutils@sourceware.org mailing list for the binutils 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]: Sparc GOTDATA optimizations.


From: Nick Clifton <nickc@redhat.com>
Date: Thu, 11 Feb 2010 12:32:46 +0000

> Note - there appear to be a few, very minor, formatting issues with
> the patch, where a whitespace character and an operator symbol appear
> to have swapped positions.  Eg:
> 
>> +		  if (local_got_refcounts[r_symndx]>  0)
> 
> I would have expected:
> 
>> +		  if (local_got_refcounts[r_symndx] > 0)
> 

Nick, I took a second look and I do not see this in my copy
of the patch posting.

Perhaps your email client or similar did something with the
formatting?

Even in the archives:

http://sourceware.org/ml/binutils/2010-02/msg00176.html

The spacing is there on either side of the ">"

Similarly for the other spacing issues you mentioned.


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