Pending Character-related work - Character-warnings

[Ulf Zibis]

Oops, it's never superfluous to review again:

Line 245: too much spaces
Compare line 363 vs 376 etc.: inconsistent indentation after @since
Line 598: can be deleted
Line 600: 2 spaces after @exception
Line 600: 2 spaces after NullPointerException
Line 600: better use @throws
Line 652: shoudn't you add a @suppressWarnings before // But I still 
think, a little comment would be better.
Line 4706: 2 spaces after ch should be enough (you can donate it to line 
4736 after codePoint ;-) )
Line 5297, 5330: one more space after @return
Line 5812: another type of indentation iregularity
... (there are some more irregularities in this class on indentation and 
spacing)
... I think, the time has come to replace things like: 
<code>'\u005A'</code>

-Ulf



Am 30.06.2010 19:50, schrieb Martin Buchholz:
> On Wed, Jun 30, 2010 at 01:22, Ulf Zibis<Ulf.Zibis at gmx.de>  wrote:
>    
>> Am 29.06.2010 02:29, schrieb Martin Buchholz:
>>      
>    
>>> I tried to do that, but Character.java is one of those classes
>>> that needs to be compilable by the bootstrap JDK,
>>> so this change ist leider nicht moeglich.
>>>
>>>
>>>        
>> I think, there should be a note about this in the sources of those classes.
>>      
> Such a note is likely to become obsolete soon.
>
>    
>> Maybe it's possible to have 2 versions, one for the bootstrap + another for
>> the final compilation.
>>      
> It's not a crime to implement language FOO(N) in language FOO(N-1).
>
> Martin
>
>
>