Code review request: 7077672 jdk8_tl nightly fail in step-2 build on 8/10/11
Rémi Forax
forax at univ-mlv.fr
Fri Aug 12 12:28:44 UTC 2011
On 08/12/2011 01:29 AM, Jonathan Gibbons wrote:
> On 08/11/2011 02:56 PM, Rémi Forax wrote:
>> On 08/11/2011 10:55 PM, Jonathan Gibbons wrote:
>>> Rémi, you wrote:
>>>
>>>> On 08/11/2011 09:07 PM, Alexandre Boulgakov wrote:
>>>> >/ Please review the attached patch to fix a build break.
>>>> />/
>>>> />/ The fix changes new Class<>[] to new Class<?>[] in two places.
>>>> />/
>>>> />/ Thanks,
>>>> />/ Sasha
>>>> /
>>>> Hi Sasha,
>>>> you can rewrite:
>>>>
>>>> connectMethod = corbaStubClass.getMethod("connect",
>>>> new Class<>[] {org.omg.CORBA.ORB.class});
>>>>
>>>> to
>>>>
>>>> connectMethod = corbaStubClass.getMethod("connect",
>>>> org.omg.CORBA.ORB.class);
>>>>
>>>> because getMethod is a varargs.
>>>>
>>>> Anyways, the changes are ok for me without that.
>>>>
>>>> Rémi
>>>
>>> Thanks for the suggestion. In this case we wanted a minimal fix to
>>> go back as safely and as quickly as possible. But your suggestion
>>> leads to another, broader suggestion for a cleanup of places where
>>> the array declaration is no longer required. Another one for the list!
>>>
>>> -- Jon
>>
>> Hi Jon,
>> first I'm really happy of these clean up,
>> I used to use the JDK source code as an example of well written code
>> for my students,
>> so any improvements is great.
>>
>> Also I agree with you, the proposed change above is not as important as
>> the changes Sasha is doing because it just improves the readability
>> not the type safety.
>> So it can be done later.
>>
>> cheers,
>> Rémi
>>
>
> Rémi,
>
> "I used to use ...." ? :-(
To be crystal clear, the code deliver with JDK 1.5 was full of raw
types, rare types etc
not something you want beginners be exposed.
The code is now in better shape.
>
> If you see any more opportunities for cleanup, I hope you'll let us
> know. Patches (or even changesets) would be even better :-) You
> could even have fun with the compiler and tree API to find examples of
> unclean code that are candidates for cleanup.
I think you have found a good project for my students :)
> After all, why should we have all the fun?
>
> -- Jon
Rémi
More information about the core-libs-dev
mailing list