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?
-Florian
Regards, Pavel.
Ok, thanks
-Florian
Pavel Porvatov schrieb:
Hi Florian,
I've started Sun process of applying your fix. I'll notify you after
the fix will be in the mercurial repository.
Thanks, Pavel.
Hi Pavel,
what is the status of my patch? I would like to provide more patches.
-Florian
Am Dienstag, 22. Januar 2008 schrieb Pavel Porvatov:
Hi Florian,
Now your patch looks perfectly! Let's wait the moment when I will be
able to commit this patch (now nobody can make changes in the jdk7
repository) and then we'll continue with your changes.
Regards, Pavel.