Author: sebb
Date: Sun Nov 25 17:16:06 2007
New Revision: 598102

URL: http://svn.apache.org/viewvc?rev=598102&view=rev
Log:
Remove unused flag; catch RuntimeException and log it

Modified:
    
jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java

Modified: 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java
URL: 
http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java?rev=598102&r1=598101&r2=598102&view=diff
==============================================================================
--- 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java 
(original)
+++ 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java 
Sun Nov 25 17:16:06 2007
@@ -222,7 +222,7 @@
                if (visualizer == null) {
                        return; // No point reading the file if there's no 
visualiser
                }
-               boolean parsedOK = false, errorDetected = false;
+               boolean parsedOK = false;
                String filename = getFilename();
         File file = new File(filename);
         if (file.exists()) {
@@ -277,10 +277,12 @@
                 log.warn("Problem reading JTL file: "+file);
                        } catch (JMeterError e){
                 log.warn("Problem reading JTL file: "+file);
+                       } catch (RuntimeException e){ // e.g. 
NullPointerException
+                log.warn("Problem reading JTL file: "+file,e);
             } finally {
                 JOrphanUtils.closeQuietly(dataReader);
                 JOrphanUtils.closeQuietly(bufferedInputStream);
-                               if (!parsedOK || errorDetected) {
+                               if (!parsedOK) {
                     GuiPackage.showErrorMessage(
                                 "Error loading results file - see log file",
                                 "Result file loader");



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to