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
