Request for Review: 7116914 (Miscellaneous warnings (sun.text))

[Stuart Marks]

Looks good. Thanks for making the updates. Go ahead and push.

s'marks

On 12/5/11 3:05 AM, Yuka Kamiya wrote:
> Hi Stuart,
> 
> All right. Here's a new webrev which includes two changes:
>    - 'length=(srcIndex-prevSrc);' in normalizer/NormalizerImpl.java
>    - @SuppressWarnings("serial") in bidi/BidiBase.java
> 
> http://cr.openjdk.java.net/~peytoia/7116914/webrev.01/
> 
> Could you please review it?
> 
> Thanks,
> --
> Yuka
> 
> 
> (11/12/05 15:39), Stuart Marks wrote:
>> I guess we agree that the only use of serialVersionUID would be to remove the
>> warning, as it's not useful for serialization compatibility. Wouldn't it be
>> better just to suppress the warning instead?
>>
>> Regarding the "throw syntaxError()" change, yes that would be a bit too much of
>> a refactoring to perform in the context of warnings cleanup.
>>
>> s'marks
>>
>> On 12/4/11 7:24 PM, Yuka Kamiya wrote:
>>> Hi,
>>>
>>> Mike, Stuart, Alan, and Masayoshi:
>>> Thank you for your comments.
>>>
>>> If no one has any objections, I'd like to fix only
>>>      >    - The parens are probably not needed around 'length=(srcIndex-prevSrc);'
>>> pointed out by Mike.
>>>
>>> Both the serialVersionUID of AttributedCharacterIterator.Attribute and breaks in UnicodeSet.java seem harmless.
>>>
>>> I like Stuart's suggestion
>>>      >    case 1:
>>>      >            ...
>>>      >           throw syntaxError(...);
>>> but we have a local maintenance policy about these files imported from the third party.
>>> As long as the code works correctly (and doesn't issue a warning), smaller change is better.
>>> (The policy may change in the future, though.)
>>>
>>> Thanks,
>>> --
>>> Yuka
>>>
>>>
>>> (11/12/02 16:51), Yuka Kamiya wrote:
>>>> Hello,
>>>>
>>>> Could someone please review this fix?
>>>>
>>>> http://cr.openjdk.java.net/~peytoia/7116914/webrev.00/
>>>>
>>>> Some warnings are still issued even after this fix, and that's intentional.
>>>> I'd like to solve them in another way rather than using @SuppressWarnings("deprecation") in the future.
>>>>
>>>> The original number of warnings in this area was 70. Now 16.
>>>>
>>>> Thanks,
>>>> --
>>>> Yuka Kamiya