On 10/30/17, 11:14 AM, Roger Riggs wrote:
Hi Joe,
+1
Is there a useful comment on the @SuppressWarnings like in other files:
XSDHandler.java: 3028
DTMDocumentImpl: 1700
DOM2DTM.java: 1654
FilterExprWalker.java: 65
For the methods with long or very long switch statements, I added a note
following the SuppressWarnings annotation to indicate where fallthrough
would happen and in which case warnings were suppressed. But for the
pretty short ones like the above, I thought it's quite obvious where
fallthrough might happen, I didn't therefore add any comment.
A few of the added breaks would have been hiding bugs.
It might be worth mentioning them in the issue.
I added a note to the issue.
Thanks,
Joe
Roger
On 10/30/2017 1:09 PM, Lance Andersen wrote:
Hi Joe
The changes look OK
Best
Lance
On Oct 30, 2017, at 1:03 PM, Joe Wang <[email protected]> wrote:
Hi,
Please review a cleanup of fallthrough and static warnings. For
fallthrough, the majority of the changes are suppressing the
warnings, while for static, replacing the instances with the class.
All jaxp tests and JCK passed.
JBS: https://bugs.openjdk.java.net/browse/JDK-8181155
webrevs:
http://cr.openjdk.java.net/~joehw/jdk10/8181155/webrev/index.html
Thanks,
Joe
<http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance
Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
[email protected] <mailto:[email protected]>
