[jboss-cvs] JBossAS SVN: r87849 - in projects/ejb-book/trunk/ch04-firstejb/src: test/java/org/jboss/ejb3/examples/ch04/firstejb and 1 other directories.

[jboss-cvs-commits at lists.jboss.org]

Author: ALRubinger
Date: 2009-04-26 20:45:29 -0400 (Sun, 26 Apr 2009)
New Revision: 87849

Modified:
   projects/ejb-book/trunk/ch04-firstejb/src/main/java/org/jboss/ejb3/examples/ch04/firstejb/CalculatorBeanBase.java
   projects/ejb-book/trunk/ch04-firstejb/src/test/java/org/jboss/ejb3/examples/ch04/firstejb/CalculatorIntegrationTestCase.java
   projects/ejb-book/trunk/ch04-firstejb/src/test/resources/log4j.xml
Log:
[EJBBOOK-1] Fixes for comments, some review.  Nothing major here.

Modified: projects/ejb-book/trunk/ch04-firstejb/src/main/java/org/jboss/ejb3/examples/ch04/firstejb/CalculatorBeanBase.java
===================================================================
--- projects/ejb-book/trunk/ch04-firstejb/src/main/java/org/jboss/ejb3/examples/ch04/firstejb/CalculatorBeanBase.java	2009-04-26 19:56:53 UTC (rev 87848)
+++ projects/ejb-book/trunk/ch04-firstejb/src/main/java/org/jboss/ejb3/examples/ch04/firstejb/CalculatorBeanBase.java	2009-04-27 00:45:29 UTC (rev 87849)
@@ -48,14 +48,14 @@
    // Required Implementations --------------------------------------------------||
    // ---------------------------------------------------------------------------||
 
-   /**
-    * {@link CalculatorCommonBusiness#add(int...)}
+   /* (non-Javadoc)
+    * @see org.jboss.ejb3.examples.ch04.firstejb.CalculatorCommonBusiness#add(int[])
     */
    @Override
    public int add(final int... arguments)
    {
       // Initialize
-      StringBuffer sb = new StringBuffer();
+      final StringBuffer sb = new StringBuffer();
       sb.append("Adding arguments: ");
       int result = 0;
 

Modified: projects/ejb-book/trunk/ch04-firstejb/src/test/java/org/jboss/ejb3/examples/ch04/firstejb/CalculatorIntegrationTestCase.java
===================================================================
--- projects/ejb-book/trunk/ch04-firstejb/src/test/java/org/jboss/ejb3/examples/ch04/firstejb/CalculatorIntegrationTestCase.java	2009-04-26 19:56:53 UTC (rev 87848)
+++ projects/ejb-book/trunk/ch04-firstejb/src/test/java/org/jboss/ejb3/examples/ch04/firstejb/CalculatorIntegrationTestCase.java	2009-04-27 00:45:29 UTC (rev 87849)
@@ -133,7 +133,7 @@
     * Uses the supplied Calculator instance to test the addition
     * algorithm
     */
-   private void assertAdditionSucceeds(CalculatorCommonBusiness calc)
+   private void assertAdditionSucceeds(final CalculatorCommonBusiness calc)
    {
       // Initialize
       final int[] arguments = new int[]

Modified: projects/ejb-book/trunk/ch04-firstejb/src/test/resources/log4j.xml
===================================================================
--- projects/ejb-book/trunk/ch04-firstejb/src/test/resources/log4j.xml	2009-04-26 19:56:53 UTC (rev 87848)
+++ projects/ejb-book/trunk/ch04-firstejb/src/test/resources/log4j.xml	2009-04-27 00:45:29 UTC (rev 87849)
@@ -2,13 +2,9 @@
 <!DOCTYPE log4j:configuration SYSTEM "log4j.dtd">
 
 <!-- ===================================================================== -->
-<!--                                                                       -->
 <!--  Log4j Configuration                                                  -->
-<!--                                                                       -->
 <!-- ===================================================================== -->
 
-<!-- $Id: log4j.xml 68671 2008-01-08 10:04:25Z wolfc $ -->
-
 <!--
    | For more configuration infromation and examples see the Jakarta Log4j
    | website: http://jakarta.apache.org/log4j