[rfc][icedtea-web] Code cleanup patches
Andrew Azores
aazores at redhat.com
Mon Feb 3 12:32:19 PST 2014
Hi,
The larger two of the three attached patches do a few simple things:
1) make variables/parameters/fields final, where applicable. I feel this
makes it more clear which variables are used eg as accumulators or will
be returned by the method, etc. This also allows the removal of some
local variables which were created simply to have "final versions" of
other variables, in order for them to be captured by anonymous inner
classes for example.
2) prefix fields with "this" in very long methods. In shorter methods it
can remain clear that fields are fields since no declaration for that
name can be seen, but in very long methods, it can become more confusing.
3) fix formatting. Eg remove useless comments, remove whitespace from
"empty" lines, remove trailing whitespace
There should be no changes at all in functionality, since this is just
cleanup and not refactoring.
The smallest patch simply removes an unneeded 'ls' that I found in a
custom reproducer's Makefile.
Thanks,
--
Andrew A
-------------- next part --------------
A non-text attachment was scrubbed...
Name: classloader_cleanup.patch
Type: text/x-patch
Size: 78856 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20140203/9f4f3c11/classloader_cleanup-0001.patch
-------------- next part --------------
A non-text attachment was scrubbed...
Name: extensionjnlpsinapplet_makefile_ls.patch
Type: text/x-patch
Size: 462 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20140203/9f4f3c11/extensionjnlpsinapplet_makefile_ls-0001.patch
-------------- next part --------------
A non-text attachment was scrubbed...
Name: securitydialogmessagehandler_cleanup.patch
Type: text/x-patch
Size: 8149 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20140203/9f4f3c11/securitydialogmessagehandler_cleanup-0001.patch
More information about the distro-pkg-dev
mailing list