SingingBush commented on code in PR #2:
URL: https://github.com/apache/xerces-j/pull/2#discussion_r2539826984
##########
samples/xni/SecuritySupport.java:
##########
@@ -120,21 +108,19 @@ public Object run() {
}
static boolean getFileExists(final File f) {
- return ((Boolean)
- AccessController.doPrivileged(new PrivilegedAction() {
- public Object run() {
- return f.exists() ? Boolean.TRUE : Boolean.FALSE;
- }
- })).booleanValue();
+ return AccessController.doPrivileged(new PrivilegedAction<Boolean>() {
+ public Boolean run() {
Review Comment:
The source and target JDK is set to 1.8 in build.xml (despite comment
stating "designed to support Java 1.4+") so no need to support the older
syntax. I've built locally with JDK 8
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
