aboutsummaryrefslogtreecommitdiff
path: root/velocity-engine-examples
diff options
context:
space:
mode:
authorClaude Brisson <cbrisson@apache.org>2017-01-28 19:21:08 +0000
committerClaude Brisson <cbrisson@apache.org>2017-01-28 19:21:08 +0000
commit23e6d0faeb4648ec700481fb3f01214bd81f4f79 (patch)
tree9dcf8c9be8710fa85f2dd31ea18604584445882f /velocity-engine-examples
parent34e78d04e1edf70a291f2acbf9d891466bc9582a (diff)
downloadapache-velocity-engine-23e6d0faeb4648ec700481fb3f01214bd81f4f79.tar.gz
[engine] full code review (foreach loops and some other problems detected by code analysis tools)
git-svn-id: https://svn.apache.org/repos/asf/velocity/engine/trunk@1780734 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'velocity-engine-examples')
-rw-r--r--velocity-engine-examples/src/main/java/org/apache/velocity/example/DBContext.java2
-rw-r--r--velocity-engine-examples/src/main/java/org/apache/velocity/example/EventExample.java5
-rw-r--r--velocity-engine-examples/src/main/java/org/apache/velocity/example/Example.java2
3 files changed, 2 insertions, 7 deletions
diff --git a/velocity-engine-examples/src/main/java/org/apache/velocity/example/DBContext.java b/velocity-engine-examples/src/main/java/org/apache/velocity/example/DBContext.java
index 2dcbfb34..cd64e833 100644
--- a/velocity-engine-examples/src/main/java/org/apache/velocity/example/DBContext.java
+++ b/velocity-engine-examples/src/main/java/org/apache/velocity/example/DBContext.java
@@ -179,8 +179,6 @@ public class DBContext extends AbstractContext
{
System. out.println(e);
}
-
- return;
}
}
diff --git a/velocity-engine-examples/src/main/java/org/apache/velocity/example/EventExample.java b/velocity-engine-examples/src/main/java/org/apache/velocity/example/EventExample.java
index 607d6ffd..45eb87ab 100644
--- a/velocity-engine-examples/src/main/java/org/apache/velocity/example/EventExample.java
+++ b/velocity-engine-examples/src/main/java/org/apache/velocity/example/EventExample.java
@@ -276,10 +276,7 @@ public class EventExample extends MarkerIgnoringBase
*/
public boolean shouldLogOnNullSet( String lhs, String rhs )
{
- if (lhs.equals("$settest"))
- return false;
-
- return true;
+ return !lhs.equals("$settest");
}
public Object methodException( Context context, Class claz, String method, Exception e, Info info ) {
diff --git a/velocity-engine-examples/src/main/java/org/apache/velocity/example/Example.java b/velocity-engine-examples/src/main/java/org/apache/velocity/example/Example.java
index 3cb33e68..7717f382 100644
--- a/velocity-engine-examples/src/main/java/org/apache/velocity/example/Example.java
+++ b/velocity-engine-examples/src/main/java/org/apache/velocity/example/Example.java
@@ -88,7 +88,7 @@ public class Example
* of the template and the data to produce the output stream.
*/
- BufferedWriter writer = writer = new BufferedWriter(
+ BufferedWriter writer = new BufferedWriter(
new OutputStreamWriter(System.out));
if ( template != null)