RFR 8054213: Class name repeated in output of Type.toString()
Svetlana Nikandrova
svetlana.nikandrova at oracle.com
Tue Jun 28 18:25:37 UTC 2016
May be someone can find a minute?
On 27.06.2016 21:25, Svetlana Nikandrova wrote:
> Kindly reminder
>
> On 24.06.2016 14:58, Svetlana Nikandrova wrote:
>> Hello,
>>
>> let me try again with updated version of webrev:
>> Webrev:
>> http://cr.openjdk.java.net/~snikandrova/8054213/webrev.01/
>> <http://cr.openjdk.java.net/%7Esnikandrova/8054213/webrev.01/>
>> Issue:
>> https://bugs.openjdk.java.net/browse/JDK-8054213
>>
>> Many thanks goes to Sergey Ustimenko for his valuable advises. I
>> believe fix looks nicer now.
>> JPRT tested.
>>
>> Thank you,
>> Svetlana
>>
>> On 17.06.2016 21:45, Svetlana Nikandrova wrote:
>>> Hello,
>>>
>>> could you please review this fix for toString() method of
>>> ParameterizedTypeImpl.
>>> The problem is that when we obtain simple name of nested type shared
>>> prefix is only removed for ParameterizedType owner. We need to
>>> remove it for other cases too.
>>>
>>> Please note that I also changed delimiter between outer and inner
>>> class from "." to "$". I believe as "$" is usually used to separate
>>> nested class name "." looks inconsistent and confusing.
>>> Let me know if you have any objections.
>>>
>>> Webrev:
>>> http://cr.openjdk.java.net/~msolovie/8054213/webrev.00/
>>> <http://cr.openjdk.java.net/%7Emsolovie/8054213/webrev.00/>
>>> Issue:
>>> https://bugs.openjdk.java.net/browse/JDK-8054213
>>>
>>> Thank you,
>>> Svetlana
>>
>
More information about the core-libs-dev
mailing list