<Swing Dev> [PATCH] 6179357-related: warnings removal in com.sun.java.swing.plaf.motif
Pavel Porvatov
Pavel.Porvatov at Sun.COM
Tue Apr 29 13:02:48 UTC 2008
Hi Florian,
> Hi Pavel!
>>
>> I've pushed your changes into
>> http://hg.openjdk.java.net/jdk7/swing/jdk, changeset is here
>> http://hg.openjdk.java.net/jdk7/swing/jdk/rev/5b1734431fa5. Note that
>> these changes can appear in http://hg.openjdk.java.net/jdk7/jdk7/jdk
>> in June in the build 28 (have a look at integration schedule
>> http://openjdk.java.net/projects/jdk7/builds/). To avoid conflicts use
>> http://hg.openjdk.java.net/jdk7/swing/jdk, please.
> Ok, thanks for the information.
>>
>> So you can send me the last part of the fix with ALL remained changes.
>> Don't put in the sources TABs characters and trailing whitespaces,
>> please.
> You once told me:
> >Note, that I cannot review diffs with size 300kb, I'm sorry :( . I
> reviewed 20 kb and it took some time. And it will be difficult to review
> new changes in Your diffs, so try to separate Your changes into files
> not more than 5-10 kb. Let's >start with the first change with size
> about 50 kb (select separation as you want)
> >3. You send me Your changes, I review it and return feedback
> >4. If Your change is excellent, I will commit these changes via our
> internal procedure (don't worry about this step :))
> >5. We repeat the steps 3-4 and so on... (We can increase a next diffs)
>
> So how should we continue? Send you all remaining fixes at once or in
> small portions? What do you like more?
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
Regards, Pavel.
More information about the swing-dev
mailing list