<Swing Dev> [PATCH] 6179357-related: warnings removal
Pavel Porvatov
Pavel.Porvatov at Sun.COM
Tue Aug 26 12:29:20 UTC 2008
Hi Florian,
> Hi Pavel,
>
> great! :-) I will check it as soon as I find some spare time.
>
> What's the difference between
> http://hg.openjdk.java.net/jdk7/swing-gate/jdk
>
> and
> http://hg.openjdk.java.net/jdk7/swing/jdk
> ?
>
> I used to work on
> http://hg.openjdk.java.net/jdk7/swing/jdk
>
> Should I move to
> http://hg.openjdk.java.net/jdk7/swing-gate/jdk
> ?
>
Use http://hg.openjdk.java.net/jdk7/swing/jdk. All fixes are there...
Regards, Pavel.
> -Florian
>
> Pavel Porvatov schrieb:
>> Hi Florian,
>>
>> I've committed the last part of your fix into
>> http://hg.openjdk.java.net/jdk7/swing-gate/jdk.
>>
>> There are four CRs related to your fix: 6693507, 6722802, 6727661 and
>> 6727662. I made some changes in your patch and improved code unrelated
>> to warnings. I also reverted some your changes because they produce
>> new warnings. If you find out new compilation warnings which can be
>> removed let me know...
>>
>> Regards, Pavel.
>>
>>> Hi Pavel,
>>>
>>> it took a little longer than I thought, but now I've updated my
>>> patches to base on http://hg.openjdk.java.net/jdk7/swing/jdk and
>>> removed all the tabs.
>>>
>>> Unfortunatly it wasn't easily possible to port all patches. So I send
>>> here the big part and after integration I will try to remove some of
>>> the other warnings again before I start with adding generics to the
>>> Swing framework.
>>>
>>> When compiling with
>>> -Xlint -J-Xms80m -J-Xmx256m -Xlint:-serial -Xlint:-deprecation
>>> -Xlint:-fallthrough -Xmaxwarns 2300
>>> this patch reduces the number of reported warnings from 1263 to 240.
>>>
>>> Again, I tried only to change things related to the warnings and not
>>> to change any public or protected APIs in public or protected classes
>>> in this patch.
>>>
>>> At many places I tried to add type parameters as meaningful as
>>> possible, but with some cases I just used Object, if a more
>>> meaningful type parameter was not so trivial. You might want to
>>> refactor this if needed. (Though everything should work as it is.)
>>>
>>> -Florian
>>>
>>> Am Dienstag, 29. April 2008 schrieb Florian Brunner:
>>>> Hi Pavel,
>>>>
>>>>> Send me all remaining fixes at once. I changed my opinion because
>>>>> 1. I don't want to produce a lot of CRs for every part of fix
>>>>> 2. I'm going to do a small correction (if needed of course) in your
>>>>> fix myself. It'll save a lot of our time
>>>> Ok, great, I will do that as soon as I find some spare time (probably
>>>> during the coming long week-end).
>>>>
>>>> -Florian
>>>
>>
>
More information about the swing-dev
mailing list