Fwd: JDK 8 Warnings Cleanup Day -- Dec. 1st, 2011

Stuart Marks stuart.marks at oracle.com
Tue Nov 29 02:45:55 UTC 2011


Folks here might be interested in this effort. This will be a JDK8 activity, so 
if you want to participate, please respond on the jdk8-dev list.

Thanks.

s'marks

-------- Original Message --------
Subject: JDK 8 Warnings Cleanup Day -- Dec. 1st, 2011
Date: Mon, 28 Nov 2011 18:10:06 -0800 (PST)
From: Xiomara Jayasena <xiomara.jayasena at oracle.com>
To: jdk8-dev at openjdk.java.net, jdk7-dev at openjdk.java.net
CC: build-dev at openjdk.java.net

A few teams are planning on cleaning up warnings generated during the JDK build 
on Dec. 1st.  We would like to invite you to participate.



To be able to participate a prerequisite is that you are able build the JDK. 
The source that we will target is JDK8.  If you decide to participate then 
please reply to this e-mail with the area that you intend to work on.  I will 
keep track of the areas so that there is no overlap of work.



.         A group of us working on this project will create bugs for the areas 
that developers sign up for.

.         The focus will be on java warnings, since these are less risky.

.         For this exercise please avoid making any API changes or any large 
scale refactorings.

.         Avoid @SuppressWarnings except where it is clearly necessary.

.         We would like to have the code changes completed by 5pm (PST) on Dec. 
1st.

.         A suggested area to work on is the build step defined by: 
jdk/make/java/java/ which compiles close to 500 .java files and produces over 
1200 warnings.  The java files are in java.io, java.lang, java.util and a 
variety of related packages.



Process to follow:



1.       Pick a file or a few files to work on and send mail to jdk8-dev by 8am 
PT on Thursday to let us know which area you would like to work on.

2.       A bug ID will be generated for this.

3.       Pull this repo to base your changes on it: 
http://hg.openjdk.java.net/jdk8/tl/jdk

4.       Post your changes for review.  If you have webrev, please use it. If 
you don't have webrev, please generate a patch and post it here as a .txt 
attachment.  If the patch is too cumbersome to review, we can generate a webrev 
for you.

5.       All of the changes must go through the usual build, test and review 
process before being pushed.  If you are a JDK8 committer you may push after 
completing the usual process.  If you aren't a committer we will help you 
review and test your changes.

6.       In order to streamlines the process, Stuart Marks will be 
consolidating patches and running build and test jobs - thanks to Stuart for 
doing this!



Lastly to get a build with warnings enable use the following flags:

make JAVAC_MAX_WARNINGS=true JAVAC_WARNINGS_FATAL= OTHER_JAVACFLAGS="-Xmaxwarns 
10000"



Tip: Make sure to redirect output to a file.  To find the 
jdk/make/java/java/Makefile build step mentioned above, open the log file and 
look for the first occurrence of '# Running javac'. Voluminous warning output 
should appear below."



-Xiomara



More information about the discuss mailing list