Author: sebb
Date: Tue Jan 8 04:02:29 2008
New Revision: 609943
URL: http://svn.apache.org/viewvc?rev=609943&view=rev
Log:
Tidy Exception handling
Modified:
jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/util/accesslog/StandardGenerator.java
Modified:
jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/util/accesslog/StandardGenerator.java
URL:
http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/util/accesslog/StandardGenerator.java?rev=609943&r1=609942&r2=609943&view=diff
==============================================================================
---
jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/util/accesslog/StandardGenerator.java
(original)
+++
jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/util/accesslog/StandardGenerator.java
Tue Jan 8 04:02:29 2008
@@ -27,7 +27,9 @@
import org.apache.jmeter.protocol.http.sampler.HTTPSamplerBase;
import org.apache.jmeter.protocol.http.sampler.HTTPSamplerFactory;
+import org.apache.jorphan.logging.LoggingManager;
import org.apache.jorphan.util.JOrphanUtils;
+import org.apache.log.Logger;
/**
* Description:<br>
@@ -56,6 +58,8 @@
public class StandardGenerator implements Generator, Serializable {
+ private static final Logger log = LoggingManager.getLoggerForClass();
+
protected HTTPSamplerBase SAMPLE = null;
transient protected FileWriter WRITER = null;
@@ -108,7 +112,7 @@
try {
this.OUTPUT = new FileOutputStream(FILE);
} catch (IOException exception) {
- // do nothing
+ log.error(exception.getMessage());
}
}
@@ -209,11 +213,7 @@
* @see
org.apache.jmeter.protocol.http.util.accesslog.Generator#generateRequest()
*/
public Object generateRequest() {
- try {
- SAMPLE = HTTPSamplerFactory.newInstance();
- } catch (NullPointerException e) {
- e.printStackTrace();
- }
+ SAMPLE = HTTPSamplerFactory.newInstance();
return SAMPLE;
}
@@ -222,14 +222,11 @@
* saved.
*/
public void save() {
- try {
- // no implementation at this time, since
- // we bypass the idea of having a console
- // tool to generate test plans. Instead
- // I decided to have a sampler that uses
- // the generator and parser directly
- } catch (Exception exception) {
- }
+ // no implementation at this time, since
+ // we bypass the idea of having a console
+ // tool to generate test plans. Instead
+ // I decided to have a sampler that uses
+ // the generator and parser directly
}
/**
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
