On 9/26/2015 6:42 PM, Sebastian Sickelmann wrote:
Hello,
my name is Sebastian Sickelmann and i signed the OCA.
Actually I am searching through the JBS for low hanging fruits.
Right now i am looking through the openjdk-sources and try to evaluate
if i can make something about JDK-5108778.
As i am not an author, i am actually not able to host webrevs on
cr.openjdk.java.net.
Is there someone who would support at hosting the macosx-part of
JDK-5108778 on cr.openjdk.java.net for reviewing?
Thank you for the contribution.
Here is the created link:
http://cr.openjdk.java.net/~alexsch/sebastian.sickelmann/5108778/webrev.00/
I placed the macosx part in my dropbox at:
https://dl.dropboxusercontent.com/u/43692695/oss-patches/openjdk/jdk-5108778/macosx_0/webrev/index.html
or as zip:
https://dl.dropboxusercontent.com/u/43692695/oss-patches/openjdk/jdk-5108778/macosx_0/webrev.zip
Is it possible to use autoboxing in the fix instead of
Boolean.valueOf(someBooleanVariable)?
It is required to have automated tests for a fix where it is possible.
Can an automated test be written which checks that number of Boolean
valueas are not too much?
The issue 5108778 is too broad. Are you going to fix it only in JDK
client area? If so, you can file a specific issue on it.
Unfortunately I am not able to compile or run the tests for this on my
machine because I don't have a macosx running on my hardware.
I compiled the code and at least it does not have errors.
Maybe
someone can support here, too?
Is it possible to use a virtual machine on your desktop?
Thanks,
Alexandr.
-- Sebastian
