Author: sebb Date: Fri Aug 14 21:41:19 2009 New Revision: 804382 URL: http://svn.apache.org/viewvc?rev=804382&view=rev Log: Java 1.5 always has nanoTime() Fix up some raw types
Modified: jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleResult.java Modified: jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleResult.java URL: http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleResult.java?rev=804382&r1=804381&r2=804382&view=diff ============================================================================== --- jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleResult.java (original) +++ jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleResult.java Fri Aug 14 21:41:19 2009 @@ -26,8 +26,6 @@ import java.util.HashSet; import java.util.List; import java.util.Set; -import java.lang.reflect.Method; -import java.lang.reflect.InvocationTargetException; import org.apache.avalon.framework.configuration.Configuration; import org.apache.jmeter.assertions.AssertionResult; @@ -140,48 +138,22 @@ private long pauseTime = 0;// Start of pause (if any) - private List assertionResults; + private List<AssertionResult> assertionResults; - private List subResults; + private List<SampleResult> subResults; private String dataType=""; // Don't return null if not set private boolean success; //@GuardedBy("this"") - private final Set files = new HashSet(); // files that this sample has been saved in + private final Set<String> files = new HashSet<String>(); // files that this sample has been saved in private String dataEncoding;// (is this really the character set?) e.g. // ISO-8895-1, UTF-8 - private static Method initNanoTimeMethod() { - try { - return System.class.getMethod("nanoTime", null); - } catch (NoSuchMethodException e) { - return null; - } - } - - private static boolean haveNanoTime() { - return nanoTimeMethod != null; - } - - private static long nanoTime() { - Long result = null; - try { - result = (Long) nanoTimeMethod.invoke(null, null); - } catch (IllegalAccessException e) { - throw new RuntimeException(e); - } catch (InvocationTargetException e) { - throw new RuntimeException(e); - } - return result.longValue(); - } - - private static final Method nanoTimeMethod = initNanoTimeMethod(); - // a reference time from the nanosecond clock - private static final long referenceTimeNsClock = haveNanoTime() ? sampleNsClockInMs() : Long.MIN_VALUE; + private static final long referenceTimeNsClock = sampleNsClockInMs(); // a reference time from the millisecond clock private static final long referenceTimeMsClock = System.currentTimeMillis(); @@ -331,17 +303,13 @@ } private static long sampleNsClockInMs() { - return nanoTime() / 1000000; + return System.nanoTime() / 1000000; } // Helper method to get 1 ms resolution timing. public static long currentTimeInMs() { - if (haveNanoTime()) { - long elapsedInMs = sampleNsClockInMs() - referenceTimeNsClock; - return referenceTimeMsClock + elapsedInMs; - } else { - return System.currentTimeMillis(); - } + long elapsedInMs = sampleNsClockInMs() - referenceTimeNsClock; + return referenceTimeMsClock + elapsedInMs; } // Helper method to maintain timestamp relationships @@ -458,7 +426,7 @@ public void addAssertionResult(AssertionResult assertResult) { if (assertionResults == null) { - assertionResults = new ArrayList(); + assertionResults = new ArrayList<AssertionResult>(); } assertionResults.add(assertResult); } @@ -473,7 +441,7 @@ if (assertionResults == null) { return EMPTY_AR; } - return (AssertionResult[]) assertionResults.toArray(new AssertionResult[0]); + return assertionResults.toArray(new AssertionResult[0]); } public void addSubResult(SampleResult subResult) { @@ -484,7 +452,7 @@ } subResult.setThreadName(tn); if (subResults == null) { - subResults = new ArrayList(); + subResults = new ArrayList<SampleResult>(); } subResults.add(subResult); // Extend the time to the end of the added sample @@ -503,7 +471,7 @@ */ public void storeSubResult(SampleResult subResult) { if (subResults == null) { - subResults = new ArrayList(); + subResults = new ArrayList<SampleResult>(); } subResults.add(subResult); subResult.setParent(this); @@ -519,7 +487,7 @@ if (subResults == null) { return EMPTY_SR; } - return (SampleResult[]) subResults.toArray(new SampleResult[0]); + return subResults.toArray(new SampleResult[0]); } public void configure(Configuration info) { --------------------------------------------------------------------- To unsubscribe, e-mail: jmeter-dev-unsubscr...@jakarta.apache.org For additional commands, e-mail: jmeter-dev-h...@jakarta.apache.org