Modified: 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/components/Choose.java
URL: 
http://svn.apache.org/viewcvs/jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/components/Choose.java?rev=368196&r1=368195&r2=368196&view=diff
==============================================================================
--- 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/components/Choose.java
 (original)
+++ 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/components/Choose.java
 Wed Jan 11 16:47:57 2006
@@ -19,43 +19,41 @@
 import org.apache.tapestry.components.Conditional;
 
 /**
- *  This component is a container for [EMAIL PROTECTED] When} or Otherwise 
components;
- *  it provides the context for mutually exclusive conditional evaluation.
- *
- *  [<a 
href="../../../../../../ComponentReference/contrib.Choose.html">Component 
Reference</a>]
- *
- *  @author David Solis
+ * This component is a container for [EMAIL PROTECTED] When} or Otherwise 
components; it
+ * provides the context for mutually exclusive conditional evaluation. [<a
+ * href="../../../../../../ComponentReference/contrib.Choose.html">Component
+ * Reference</a>]
  * 
+ * @author David Solis
  */
 public abstract class Choose extends Conditional {
 
+    public void addBody(IRender element)
+    {
+        super.addBody(element);
+        if (element instanceof When) ((When)element).setChoose(this);
+    }
 
-       public void addBody(IRender element)
-       {
-               super.addBody(element);
-               if (element instanceof When)
-                       ((When) element).setChoose(this);       
-       }
-       
-       protected void cleanupAfterRender(IRequestCycle cycle)
-       {
-               setConditionMet(false);
-               super.cleanupAfterRender(cycle);
-       }
-       
-       protected boolean evaluateCondition()
-       {
-               return getCondition();
-       }
-
-       public boolean getInvert()
-       {
-               // This component doesn't require invert parameter.
-               return false;
-       }
-
-       public abstract boolean getCondition();
-       
-       public abstract boolean isConditionMet();
-       public abstract void setConditionMet(boolean value);
+    protected void cleanupAfterRender(IRequestCycle cycle)
+    {
+        setConditionMet(false);
+        super.cleanupAfterRender(cycle);
+    }
+
+    protected boolean evaluateCondition()
+    {
+        return getCondition();
+    }
+
+    public boolean getInvert()
+    {
+        // This component doesn't require invert parameter.
+        return false;
+    }
+
+    public abstract boolean getCondition();
+
+    public abstract boolean isConditionMet();
+
+    public abstract void setConditionMet(boolean value);
 }

Modified: 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/ejb/XCreateException.java
URL: 
http://svn.apache.org/viewcvs/jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/ejb/XCreateException.java?rev=368196&r1=368195&r2=368196&view=diff
==============================================================================
--- 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/ejb/XCreateException.java
 (original)
+++ 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/ejb/XCreateException.java
 Wed Jan 11 16:47:57 2006
@@ -16,23 +16,22 @@
 
 import javax.ejb.CreateException;
 
-
 /**
- *  Extended version of [EMAIL PROTECTED] CreateException} that includes a 
root cause.
- *
- *  @author Howard Lewis Ship
- *
- **/
+ * Extended version of [EMAIL PROTECTED] CreateException} that includes a root 
cause.
+ * 
+ * @author Howard Lewis Ship
+ */
 
 public class XCreateException extends CreateException
 {
+
     private static final long serialVersionUID = 6807032467099102587L;
-    
-       private Throwable rootCause;
+
+    private final Throwable rootCause;
 
     public XCreateException(String message)
     {
-        super(message);
+        this(message, null);
     }
 
     public XCreateException(String message, Throwable rootCause)
@@ -44,13 +43,11 @@
 
     public XCreateException(Throwable rootCause)
     {
-        super(rootCause.getMessage());
-
-        this.rootCause = rootCause;
+        this(rootCause.getMessage(), rootCause);
     }
 
     public Throwable getRootCause()
     {
         return rootCause;
     }
-}
\ No newline at end of file
+}

Modified: 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/ejb/XEJBException.java
URL: 
http://svn.apache.org/viewcvs/jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/ejb/XEJBException.java?rev=368196&r1=368195&r2=368196&view=diff
==============================================================================
--- 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/ejb/XEJBException.java
 (original)
+++ 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/ejb/XEJBException.java
 Wed Jan 11 16:47:57 2006
@@ -1,4 +1,4 @@
-// Copyright 2004, 2005 The Apache Software Foundation
+// Copyright 2004, 2005, 2006 The Apache Software Foundation
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
@@ -17,24 +17,24 @@
 import javax.ejb.EJBException;
 
 /**
- *  Extended version of [EMAIL PROTECTED] EJBException} that includes a root 
cause.
- *  [EMAIL PROTECTED] EJBException} doesn't have quite the right constructor 
for this ...
- *  it has an equivalent to the rootCause property, (causedByException), but
- *  doesn't have a constructor that allows us to set a custom message.
- *
- *  @author Howard Lewis Ship
- *
- **/
+ * Extended version of [EMAIL PROTECTED] EJBException} that includes a root 
cause.
+ * [EMAIL PROTECTED] EJBException} doesn't have quite the right constructor 
for this ... it
+ * has an equivalent to the rootCause property, (causedByException), but 
doesn't
+ * have a constructor that allows us to set a custom message.
+ * 
+ * @author Howard Lewis Ship
+ */
 
 public class XEJBException extends EJBException
 {
+
     private static final long serialVersionUID = 3712108893575174833L;
-    
-       private Throwable rootCause;
+
+    private final Throwable rootCause;
 
     public XEJBException(String message)
     {
-        super(message);
+        this(message, null);
     }
 
     public XEJBException(String message, Throwable rootCause)
@@ -55,4 +55,4 @@
     {
         return rootCause;
     }
-}
\ No newline at end of file
+}

Modified: 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/ejb/XFinderException.java
URL: 
http://svn.apache.org/viewcvs/jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/ejb/XFinderException.java?rev=368196&r1=368195&r2=368196&view=diff
==============================================================================
--- 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/ejb/XFinderException.java
 (original)
+++ 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/ejb/XFinderException.java
 Wed Jan 11 16:47:57 2006
@@ -1,4 +1,4 @@
-// Copyright 2004, 2005 The Apache Software Foundation
+// Copyright 2004, 2005, 2006 The Apache Software Foundation
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
@@ -17,21 +17,21 @@
 import javax.ejb.FinderException;
 
 /**
- *  Extended version of [EMAIL PROTECTED] FinderException} that includes a 
root cause.
- *
- *  @author Howard Lewis Ship
- *
- **/
+ * Extended version of [EMAIL PROTECTED] FinderException} that includes a root 
cause.
+ * 
+ * @author Howard Lewis Ship
+ */
 
 public class XFinderException extends FinderException
 {
+
     private static final long serialVersionUID = -7761100160348957271L;
-    
-       private Throwable rootCause;
+
+    private final Throwable rootCause;
 
     public XFinderException(String message)
     {
-        super(message);
+        this(message, null);
     }
 
     public XFinderException(String message, Throwable rootCause)
@@ -43,13 +43,11 @@
 
     public XFinderException(Throwable rootCause)
     {
-        super(rootCause.getMessage());
-
-        this.rootCause = rootCause;
+        this(rootCause.getMessage(), rootCause);
     }
 
     public Throwable getRootCause()
     {
         return rootCause;
     }
-}
\ No newline at end of file
+}

Modified: 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/ejb/XRemoveException.java
URL: 
http://svn.apache.org/viewcvs/jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/ejb/XRemoveException.java?rev=368196&r1=368195&r2=368196&view=diff
==============================================================================
--- 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/ejb/XRemoveException.java
 (original)
+++ 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/ejb/XRemoveException.java
 Wed Jan 11 16:47:57 2006
@@ -1,4 +1,4 @@
-// Copyright 2004, 2005 The Apache Software Foundation
+// Copyright 2004, 2005, 2006 The Apache Software Foundation
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
@@ -17,21 +17,21 @@
 import javax.ejb.RemoveException;
 
 /**
- *  Extended version of [EMAIL PROTECTED] RemoveException} that includes a 
root cause.
- *
- *  @author Howard Lewis Ship
- *
- **/
+ * Extended version of [EMAIL PROTECTED] RemoveException} that includes a root 
cause.
+ * 
+ * @author Howard Lewis Ship
+ */
 
 public class XRemoveException extends RemoveException
 {
+
     private static final long serialVersionUID = -8940644648555335217L;
-    
-       private Throwable rootCause;
+
+    private final Throwable rootCause;
 
     public XRemoveException(String message)
     {
-        super(message);
+        this(message, null);
     }
 
     public XRemoveException(String message, Throwable rootCause)
@@ -43,13 +43,11 @@
 
     public XRemoveException(Throwable rootCause)
     {
-        super(rootCause.getMessage());
-
-        this.rootCause = rootCause;
+        this(rootCause.getMessage(), rootCause);
     }
 
     public Throwable getRootCause()
     {
         return rootCause;
     }
-}
\ No newline at end of file
+}

Modified: 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/form/checkboxes/CheckboxGroup.java
URL: 
http://svn.apache.org/viewcvs/jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/form/checkboxes/CheckboxGroup.java?rev=368196&r1=368195&r2=368196&view=diff
==============================================================================
--- 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/form/checkboxes/CheckboxGroup.java
 (original)
+++ 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/form/checkboxes/CheckboxGroup.java
 Wed Jan 11 16:47:57 2006
@@ -1,4 +1,4 @@
-// Copyright 2004, 2005 The Apache Software Foundation
+// Copyright 2004, 2005, 2006 The Apache Software Foundation
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
@@ -28,29 +28,33 @@
  */
 public abstract class CheckboxGroup extends BaseComponent
 {
+
     public final static String CHECKBOX_GROUP_ATTRIBUTE = 
"org.apache.tapestry.contrib.form.CheckboxGroup";
 
     public abstract Collection getCheckboxNames();
+
     public abstract String getFunctionName();
-    public abstract void setFunctionName(String functionName);    
+
+    public abstract void setFunctionName(String functionName);
 
     public void registerControlledCheckbox(ControlledCheckbox checkbox)
     {
-       String name = checkbox.getCheckboxName();
-       String form = checkbox.getForm().getName();
+        String name = checkbox.getCheckboxName();
+        String form = checkbox.getForm().getName();
         getCheckboxNames().add(form + "." + name);
     }
-    
+
     /**
-     * @see 
org.apache.tapestry.BaseComponent#renderComponent(org.apache.tapestry.IMarkupWriter,
 org.apache.tapestry.IRequestCycle)
+     * @see 
org.apache.tapestry.BaseComponent#renderComponent(org.apache.tapestry.IMarkupWriter,
+     *      org.apache.tapestry.IRequestCycle)
      */
     protected void renderComponent(IMarkupWriter writer, IRequestCycle cycle)
     {
         Object objOtherGroup = cycle.getAttribute(CHECKBOX_GROUP_ATTRIBUTE);
         cycle.setAttribute(CHECKBOX_GROUP_ATTRIBUTE, this);
-        
+
         initialize(cycle);
-        
+
         super.renderComponent(writer, cycle);
 
         // clear the registered checkbox names after rendering
@@ -64,13 +68,13 @@
     {
         String functionName = "setCheckboxGroup";
 
-        if (!cycle.isRewinding()) {
-               PageRenderSupport body = 
TapestryUtils.getPageRenderSupport(cycle, this);
-               if (body != null)
-                   functionName = body.getUniqueString("setCheckboxGroup");
+        if (!cycle.isRewinding())
+        {
+            PageRenderSupport body = TapestryUtils.getPageRenderSupport(cycle, 
this);
+            if (body != null) functionName = 
body.getUniqueString("setCheckboxGroup");
         }
-        
+
         setFunctionName(functionName);
     }
-    
+
 }

Modified: 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/form/checkboxes/ControlledCheckbox.java
URL: 
http://svn.apache.org/viewcvs/jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/form/checkboxes/ControlledCheckbox.java?rev=368196&r1=368195&r2=368196&view=diff
==============================================================================
--- 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/form/checkboxes/ControlledCheckbox.java
 (original)
+++ 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/form/checkboxes/ControlledCheckbox.java
 Wed Jan 11 16:47:57 2006
@@ -1,4 +1,4 @@
-// Copyright 2004, 2005 The Apache Software Foundation
+// Copyright 2004, 2005, 2006 The Apache Software Foundation
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
@@ -27,29 +27,31 @@
  */
 public abstract class ControlledCheckbox extends BaseComponent
 {
+
     public abstract CheckboxGroup getGroup();
-    
+
     public String getCheckboxName()
     {
-        Checkbox checkbox = (Checkbox) getComponent("checkbox");
+        Checkbox checkbox = (Checkbox)getComponent("checkbox");
         String name = checkbox.getName();
         return name;
     }
-    
+
     public IForm getForm()
     {
-       Checkbox checkbox = (Checkbox) getComponent("checkbox");
-       IForm form = checkbox.getForm();
-       return form;
+        Checkbox checkbox = (Checkbox)getComponent("checkbox");
+        IForm form = checkbox.getForm();
+        return form;
     }
-    
+
     /**
-     * @see 
org.apache.tapestry.BaseComponent#renderComponent(org.apache.tapestry.IMarkupWriter,
 org.apache.tapestry.IRequestCycle)
+     * @see 
org.apache.tapestry.BaseComponent#renderComponent(org.apache.tapestry.IMarkupWriter,
+     *      org.apache.tapestry.IRequestCycle)
      */
     protected void renderComponent(IMarkupWriter writer, IRequestCycle cycle)
     {
         super.renderComponent(writer, cycle);
-        
+
         registerCheckbox();
     }
 
@@ -57,16 +59,18 @@
     {
         getCheckboxGroup().registerControlledCheckbox(this);
     }
-    
+
     public CheckboxGroup getCheckboxGroup()
     {
         CheckboxGroup group = getGroup();
-        if (group == null) {
+        if (group == null)
+        {
             IRequestCycle cycle = getPage().getRequestCycle();
-            group = (CheckboxGroup) 
cycle.getAttribute(CheckboxGroup.CHECKBOX_GROUP_ATTRIBUTE);
+            group = 
(CheckboxGroup)cycle.getAttribute(CheckboxGroup.CHECKBOX_GROUP_ATTRIBUTE);
         }
         if (group == null)
-            throw new ApplicationRuntimeException("The component " + 
getExtendedId() + " must be wrapped by a CheckboxGroup or the 'group' parameter 
must be set.");
+            throw new ApplicationRuntimeException("The component " + 
getExtendedId()
+                    + " must be wrapped by a CheckboxGroup or the 'group' 
parameter must be set.");
 
         return group;
     }

Modified: 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/inspector/ShowSpecification.java
URL: 
http://svn.apache.org/viewcvs/jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/inspector/ShowSpecification.java?rev=368196&r1=368195&r2=368196&view=diff
==============================================================================
--- 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/inspector/ShowSpecification.java
 (original)
+++ 
jakarta/tapestry/trunk/contrib/src/java/org/apache/tapestry/contrib/inspector/ShowSpecification.java
 Wed Jan 11 16:47:57 2006
@@ -1,4 +1,4 @@
-// Copyright 2004, 2005 The Apache Software Foundation
+// Copyright 2004, 2005, 2006 The Apache Software Foundation
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
@@ -33,15 +33,15 @@
 import org.apache.tapestry.spec.IParameterSpecification;
 
 /**
- * Component of the [EMAIL PROTECTED] Inspector} page used to display the 
specification, parameters and
- * bindings and assets of the inspected component.
+ * Component of the [EMAIL PROTECTED] Inspector} page used to display the 
specification,
+ * parameters and bindings and assets of the inspected component.
  * 
  * @author Howard Lewis Ship
  */
 
-public abstract class ShowSpecification extends BaseComponent implements 
PageBeginRenderListener,
-        PageEndRenderListener
+public abstract class ShowSpecification extends BaseComponent implements 
PageBeginRenderListener, PageEndRenderListener
 {
+
     private IComponent _inspectedComponent;
 
     private IComponentSpecification _inspectedSpecification;
@@ -68,8 +68,10 @@
 
     private IBeanSpecification _beanSpecification;
 
+    /** Sorts two components by their id's. */
     private static class ComponentComparitor implements Comparator
     {
+
         public int compare(Object left, Object right)
         {
             IComponent leftComponent;
@@ -77,11 +79,10 @@
             IComponent rightComponent;
             String rightId;
 
-            if (left == right)
-                return 0;
+            if (left == right) return 0;
 
-            leftComponent = (IComponent) left;
-            rightComponent = (IComponent) right;
+            leftComponent = (IComponent)left;
+            rightComponent = (IComponent)right;
 
             leftId = leftComponent.getId();
             rightId = rightComponent.getId();
@@ -91,8 +92,8 @@
     }
 
     /**
-     * Clears all cached information about the component and such after each 
render (including the
-     * rewind phase render used to process the tab view).
+     * Clears all cached information about the component and such after each
+     * render (including the rewind phase render used to process the tab view).
      * 
      * @since 1.0.5
      */
@@ -115,14 +116,15 @@
     }
 
     /**
-     * Gets the inspected component and specification from the [EMAIL 
PROTECTED] Inspector} page.
+     * Gets the inspected component and specification from the [EMAIL 
PROTECTED] Inspector}
+     * page.
      * 
      * @since 1.0.5
      */
 
     public void pageBeginRender(PageEvent event)
     {
-        Inspector inspector = (Inspector) getPage();
+        Inspector inspector = (Inspector)getPage();
 
         _inspectedComponent = inspector.getInspectedComponent();
         _inspectedSpecification = _inspectedComponent.getSpecification();
@@ -144,21 +146,19 @@
 
     public List getFormalParameterNames()
     {
-        if (_formalParameterNames == null)
-            _formalParameterNames = 
sort(_inspectedSpecification.getParameterNames());
+        if (_formalParameterNames == null) _formalParameterNames = 
sort(_inspectedSpecification.getParameterNames());
 
         return _formalParameterNames;
     }
 
     /**
-     * Returns a sorted list of informal parameter names. This is the list of 
all bindings, with the
-     * list of parameter names removed, sorted.
+     * Returns a sorted list of informal parameter names. This is the list of
+     * all bindings, with the list of parameter names removed, sorted.
      */
 
     public List getInformalParameterNames()
     {
-        if (_informalParameterNames != null)
-            return _informalParameterNames;
+        if (_informalParameterNames != null) return _informalParameterNames;
 
         Collection names = _inspectedComponent.getBindingNames();
         if (names != null && names.size() > 0)
@@ -171,8 +171,7 @@
             // name).
 
             names = _inspectedSpecification.getParameterNames();
-            if (names != null)
-                _informalParameterNames.removeAll(names);
+            if (names != null) _informalParameterNames.removeAll(names);
 
             Collections.sort(_informalParameterNames);
         }
@@ -191,8 +190,8 @@
     }
 
     /**
-     * Returns the [EMAIL PROTECTED] 
org.apache.tapestry.spec.ParameterSpecification} corresponding to the
-     * value of the parameterName property.
+     * Returns the [EMAIL PROTECTED] 
org.apache.tapestry.spec.ParameterSpecification}
+     * corresponding to the value of the parameterName property.
      */
 
     public IParameterSpecification getParameterSpecification()
@@ -201,7 +200,8 @@
     }
 
     /**
-     * Returns the [EMAIL PROTECTED] IBinding} corresponding to the value of 
the parameterName property.
+     * Returns the [EMAIL PROTECTED] IBinding} corresponding to the value of 
the
+     * parameterName property.
      */
 
     public IBinding getBinding()
@@ -220,32 +220,32 @@
     }
 
     /**
-     * Returns the [EMAIL PROTECTED] IAsset} corresponding to the value of the 
assetName property.
+     * Returns the [EMAIL PROTECTED] IAsset} corresponding to the value of the 
assetName
+     * property.
      */
 
     public IAsset getAsset()
     {
-        return (IAsset) _inspectedComponent.getAssets().get(_assetName);
+        return (IAsset)_inspectedComponent.getAssets().get(_assetName);
     }
 
     /**
-     * Returns a sorted list of asset names, or null if the component contains 
no assets.
+     * Returns a sorted list of asset names, or null if the component contains
+     * no assets.
      */
 
     public List getAssetNames()
     {
-        if (_assetNames == null)
-            _assetNames = sort(_inspectedComponent.getAssets().keySet());
+        if (_assetNames == null) _assetNames = 
sort(_inspectedComponent.getAssets().keySet());
 
         return _assetNames;
     }
 
     public List getSortedComponents()
     {
-        if (_sortedComponents != null)
-            return _sortedComponents;
+        if (_sortedComponents != null) return _sortedComponents;
 
-        Inspector inspector = (Inspector) getPage();
+        Inspector inspector = (Inspector)getPage();
         IComponent inspectedComponent = inspector.getInspectedComponent();
 
         // Get a Map of the components and simply return null if there
@@ -265,14 +265,13 @@
     public abstract IComponent getCurrentComponent();
 
     /**
-     * Returns a list of the properties for the component (from its 
specification), or null if the
-     * component has no properties.
+     * Returns a list of the properties for the component (from its
+     * specification), or null if the component has no properties.
      */
 
     public List getSortedPropertyNames()
     {
-        if (_sortedPropertyNames == null)
-            _sortedPropertyNames = 
sort(_inspectedSpecification.getPropertyNames());
+        if (_sortedPropertyNames == null) _sortedPropertyNames = 
sort(_inspectedSpecification.getPropertyNames());
 
         return _sortedPropertyNames;
     }
@@ -294,8 +293,7 @@
 
     public List getBeanNames()
     {
-        if (_beanNames == null)
-            _beanNames = sort(_inspectedSpecification.getBeanNames());
+        if (_beanNames == null) _beanNames = 
sort(_inspectedSpecification.getBeanNames());
 
         return _beanNames;
     }
@@ -318,8 +316,7 @@
 
     private List sort(Collection c)
     {
-        if (c == null || c.size() == 0)
-            return null;
+        if (c == null || c.size() == 0) return null;
 
         List result = new ArrayList(c);
 
@@ -327,4 +324,4 @@
 
         return result;
     }
-}
\ No newline at end of file
+}

Added: jakarta/tapestry/trunk/tapestry-checkstyle.xml
URL: 
http://svn.apache.org/viewcvs/jakarta/tapestry/trunk/tapestry-checkstyle.xml?rev=368196&view=auto
==============================================================================
--- jakarta/tapestry/trunk/tapestry-checkstyle.xml (added)
+++ jakarta/tapestry/trunk/tapestry-checkstyle.xml Wed Jan 11 16:47:57 2006
@@ -0,0 +1,112 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!DOCTYPE module PUBLIC "-//Puppy Crawl//DTD Check Configuration 1.2//EN" 
"http://www.puppycrawl.com/dtds/configuration_1_2.dtd";>
+<module name="Checker">
+<property name="severity" value="warning"/>
+<module name="TreeWalker">
+<module name="FileContentsHolder">
+</module>
+<module name="JavadocStyle">
+<property name="excludeScope" value="nothing"/>
+</module>
+<module name="ConstantName">
+</module>
+<module name="LocalFinalVariableName">
+</module>
+<module name="LocalVariableName">
+</module>
+<module name="MemberName">
+<metadata name="com.atlassw.tools.eclipse.checkstyle.comment" value="Normal 
naming, but allow either &quot;m_&quot; or &quot;_&quot; as a prefix."/>
+<property name="format" value="^(m_|_)?[a-z][a-zA-Z0-9]*$"/>
+</module>
+<module name="MethodName">
+</module>
+<module name="PackageName">
+<property name="format" value="^[a-z]+(\.[a-z][a-z0-9]*)*$"/>
+</module>
+<module name="ParameterName">
+<metadata name="com.atlassw.tools.eclipse.checkstyle.comment" value="Either a 
normal name (w/o underscores) or an all uppercase acronym (rarely used)"/>
+<property name="format" value="^([a-z][a-zA-Z0-9]*|[A-Z]+)$"/>
+</module>
+<module name="StaticVariableName">
+<property name="format" value="^_?[a-z][a-zA-Z0-9]*$"/>
+</module>
+<module name="TypeName">
+</module>
+<module name="AvoidStarImport">
+</module>
+<module name="ImportOrder">
+</module>
+<module name="RedundantImport">
+</module>
+<module name="UnusedImports">
+<property name="severity" value="error"/>
+</module>
+<module name="TabCharacter">
+<property name="severity" value="error"/>
+</module>
+<module name="ModifierOrder">
+</module>
+<module name="RedundantModifier">
+</module>
+<module name="DefaultComesLast">
+</module>
+<module name="DeclarationOrder">
+</module>
+<module name="DoubleCheckedLocking">
+<property name="severity" value="error"/>
+</module>
+<module name="EmptyStatement">
+</module>
+<module name="HiddenField">
+</module>
+<module name="JUnitTestCase">
+<property name="severity" value="error"/>
+</module>
+<module name="MissingSwitchDefault">
+<property name="severity" value="error"/>
+</module>
+<module name="PackageDeclaration">
+<property name="severity" value="error"/>
+</module>
+<module name="ParameterAssignment">
+</module>
+<module name="RedundantThrows">
+</module>
+<module name="SimplifyBooleanExpression">
+<property name="severity" value="error"/>
+</module>
+<module name="SimplifyBooleanReturn">
+<property name="severity" value="error"/>
+</module>
+<module name="StringLiteralEquality">
+<property name="severity" value="error"/>
+</module>
+<module name="SuperClone">
+<property name="severity" value="error"/>
+</module>
+<module name="SuperFinalize">
+<property name="severity" value="error"/>
+</module>
+<module name="FinalClass">
+<property name="severity" value="error"/>
+</module>
+<module name="HideUtilityClassConstructor">
+<property name="severity" value="error"/>
+</module>
+<module name="InterfaceIsType">
+</module>
+<module name="MutableException">
+</module>
+<module name="ArrayTypeStyle">
+</module>
+<module name="UpperEll">
+</module>
+<module name="JavadocType">
+<property name="severity" value="error"/>
+</module>
+</module>
+<module name="PackageHtml">
+</module>
+<module name="NewlineAtEndOfFile">
+</module>
+</module>



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

Reply via email to