Backups/TwelveMonkeys
2
0
Fork 0
mirror of https://github.com/haraldk/TwelveMonkeys.git synced 2025-08-04 20:15:28 -04:00
Code Issues Projects Releases Wiki Activity

Merge pull request #505 from KoenDG/intellij_performance_analysis

Browse Source 
Changes suggested by Intellij in the category "Performance".
This commit is contained in:
Harald Kuhr 2019-11-04 08:43:22 +01:00 committed by GitHub
commit 306d8ac5f6
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 9 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
common/common-io/src/main/java/com/twelvemonkeys/io/FileUtil.java
View File

@ -922,7 +922,7 @@ public final class FileUtil {
// A URL should never be able to represent an opaque URI, test anyway // A URL should never be able to represent an opaque URI, test anyway
throw new IllegalArgumentException("URI is not hierarchical"); throw new IllegalArgumentException("URI is not hierarchical");
} }
if (path.equals("")) { if (path.isEmpty()) {
throw new IllegalArgumentException("URI path component is empty"); throw new IllegalArgumentException("URI path component is empty");
} }

4
common/common-lang/src/main/java/com/twelvemonkeys/lang/BeanUtil.java
View File

@ -88,7 +88,7 @@ public final class BeanUtil {
while (begIdx < pProperty.length() && begIdx >= 0) { while (begIdx < pProperty.length() && begIdx >= 0) {
endIdx = pProperty.indexOf(".", endIdx + 1); endIdx = pProperty.indexOf('.', endIdx + 1);
if (endIdx > 0) { if (endIdx > 0) {
subProp = pProperty.substring(begIdx, endIdx); subProp = pProperty.substring(begIdx, endIdx);
begIdx = endIdx + 1; begIdx = endIdx + 1;
@ -106,7 +106,7 @@ public final class BeanUtil {
Class[] paramClass = new Class[0]; Class[] paramClass = new Class[0];
int begBracket; int begBracket;
if ((begBracket = subProp.indexOf("[")) > 0) { if ((begBracket = subProp.indexOf('[')) > 0) {
// An error if there is no matching bracket // An error if there is no matching bracket
if (!subProp.endsWith("]")) { if (!subProp.endsWith("]")) {
return null; return null;

2
common/common-lang/src/main/java/com/twelvemonkeys/util/regex/WildcardStringParser.java
View File

@ -622,7 +622,7 @@ public class WildcardStringParser {
buffer.append("\n"); buffer.append("\n");
buffer.append(" Format: <state index>: <character> (<last free state>)"); buffer.append(" Format: <state index>: <character> (<last free state>)");
buffer.append("\n"); buffer.append("\n");
buffer.append(" Number of strings parsed: " + totalNumberOfStringsParsed); buffer.append(" Number of strings parsed: ").append(totalNumberOfStringsParsed);
buffer.append("\n"); buffer.append("\n");
} }
return buffer.toString(); return buffer.toString();

9
imageio/imageio-pict/src/main/java/com/twelvemonkeys/imageio/plugins/pict/PICTImageReader.java
View File

@ -78,6 +78,7 @@ import java.awt.image.*;
import java.io.*; import java.io.*;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.Arrays; import java.util.Arrays;
import java.util.Collections;
import java.util.Iterator; import java.util.Iterator;
import java.util.List; import java.util.List;
@ -181,7 +182,7 @@ public final class PICTImageReader extends ImageReaderBase {
readPICTHeader0(pStream); readPICTHeader0(pStream);
} }
} }
private void readPICTHeader0(final ImageInputStream pStream) throws IOException { private void readPICTHeader0(final ImageInputStream pStream) throws IOException {
// Get size // Get size
picSize = pStream.readUnsignedShort(); picSize = pStream.readUnsignedShort();
@ -325,7 +326,7 @@ public final class PICTImageReader extends ImageReaderBase {
/** /**
* Reads the PICT stream. * Reads the PICT stream.
* The contents of the stream will be drawn onto the supplied graphics * The contents of the stream will be drawn onto the supplied graphics
* object. * object.
* <p> * <p>
* If "DEBUG" is true, the elements read are listed on stdout. * If "DEBUG" is true, the elements read are listed on stdout.
@ -510,7 +511,7 @@ public final class PICTImageReader extends ImageReaderBase {
ovSize.setLocation(x, y); ovSize.setLocation(x, y);
/* /*
ovSize.x *= 2;// Don't know why, but has to be multiplied by 2 ovSize.x *= 2;// Don't know why, but has to be multiplied by 2
ovSize.y *= 2; ovSize.y *= 2;
*/ */
if (DEBUG) { if (DEBUG) {
@ -2627,7 +2628,7 @@ public final class PICTImageReader extends ImageReaderBase {
public Iterator<ImageTypeSpecifier> getImageTypes(int pIndex) throws IOException { public Iterator<ImageTypeSpecifier> getImageTypes(int pIndex) throws IOException {
// TODO: The images look slightly different in Preview.. Could indicate the color space is wrong... // TODO: The images look slightly different in Preview.. Could indicate the color space is wrong...
return Arrays.asList( return Collections.singletonList(
ImageTypeSpecifiers.createPacked( ImageTypeSpecifiers.createPacked(
ColorSpace.getInstance(ColorSpace.CS_sRGB), ColorSpace.getInstance(ColorSpace.CS_sRGB),
0xff0000, 0xff00, 0xff, 0xff000000, DataBuffer.TYPE_INT, false 0xff0000, 0xff00, 0xff, 0xff000000, DataBuffer.TYPE_INT, false