You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by hl...@apache.org on 2007/11/12 23:23:28 UTC

svn commit: r594319 [12/40] - in /tapestry/tapestry5/trunk: ./ tapestry-annotations/ tapestry-annotations/src/main/java/org/apache/tapestry/beaneditor/ tapestry-annotations/src/site/ tapestry-component-report/ tapestry-component-report/src/main/java/or...

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/ComponentEvent.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/ComponentEvent.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/ComponentEvent.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/ComponentEvent.java Mon Nov 12 14:22:31 2007
@@ -20,7 +20,7 @@
 /**
  * An event that may originate in application logic, or as a result of a client interaction (a GET
  * or POST from the client).
- * 
+ *
  * @see ComponentResourcesCommon#triggerEvent(String, Object[],
  *      org.apache.tapestry.ComponentEventHandler)
  * @see ComponentEventHandler
@@ -30,7 +30,7 @@
     /**
      * Returns true if the component event's type matches any of the provided values. Comparison is
      * caseless.
-     * 
+     *
      * @param eventType
      * @return true if there is any match
      */
@@ -45,27 +45,26 @@
 
     /**
      * Returns true if the event context contains the specified number of parameters (or more).
-     * 
+     *
      * @param parameterCount number of parameters in the event handler method
-     * @return true if the event can 
+     * @return true if the event can
      */
     boolean matchesByParameterCount(int parameterCount);
 
     /**
      * Coerces a context value to a particular type. The context is an array of objects; typically
      * it is an array of strings of extra path information encoded into the action URL.
-     * 
+     *
      * @param <T>
-     * @param index
-     *            the index of the context value
-     * @param desiredTypeName
-     *            the desired type
-     * @param methodDescription
-     *            the method for which the conversion will take place (used if reporting an error)
+     * @param index             the index of the context value
+     * @param desiredTypeName   the desired type
+     * @param methodDescription the method for which the conversion will take place (used if reporting an error)
      * @return the coerced value (a wrapper type if the desired type is a primitive)
      */
     Object coerceContext(int index, String desiredTypeName);
 
-    /** Returns the raw context as a (possibly empty) array. */
+    /**
+     * Returns the raw context as a (possibly empty) array.
+     */
     Object[] getContext();
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/ComponentResourcesAware.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/ComponentResourcesAware.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/ComponentResourcesAware.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/ComponentResourcesAware.java Mon Nov 12 14:22:31 2007
@@ -1,20 +1,20 @@
-// Copyright 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.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
+// Copyright 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.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
 package org.apache.tapestry.runtime;
 
-import org.apache.tapestry.ComponentResources;
+import org.apache.tapestry.ComponentResources;
 
 /**
  * Interface implemented by components (after they have been transformed at load time). Component
@@ -22,6 +22,8 @@
  */
 public interface ComponentResourcesAware
 {
-    /** Returns the resources associated with this component class. */
+    /**
+     * Returns the resources associated with this component class.
+     */
     ComponentResources getComponentResources();
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/Event.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/Event.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/Event.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/Event.java Mon Nov 12 14:22:31 2007
@@ -28,7 +28,7 @@
     /**
      * Returns true if the event has been aborted (meaning that the return value from some event
      * handler method was accepted, and processing of the event was terminated).
-     * 
+     *
      * @return true if no further event handler methods should be invoked
      */
     boolean isAborted();
@@ -36,9 +36,8 @@
     /**
      * Invoke to identify, to the event, what component and method is being acted upon (used for
      * some kinds of exception reporting).
-     * 
-     * @param component
-     *            the component instance from which the result was obtained
+     *
+     * @param component         the component instance from which the result was obtained
      * @param methodDescription
      */
     void setSource(Component component, String methodDescription);
@@ -46,9 +45,8 @@
     /**
      * Stores a result for the event. Storing a non-null result value may abort the event (at the
      * discretion of the {@link ComponentEventHandler}).
-     * 
-     * @param result
-     *            the result obtained from a method invocations
+     *
+     * @param result the result obtained from a method invocations
      * @return true if the event is now aborted
      */
     boolean storeResult(Object result);

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/RenderCommand.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/RenderCommand.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/RenderCommand.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/RenderCommand.java Mon Nov 12 14:22:31 2007
@@ -1,20 +1,20 @@
-// Copyright 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.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
+// Copyright 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.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
 package org.apache.tapestry.runtime;
 
-import org.apache.tapestry.MarkupWriter;
+import org.apache.tapestry.MarkupWriter;
 
 /**
  * A command used during rendering of a page.
@@ -25,7 +25,7 @@
      * Invoked on an object to request that it render itself. This involves a mix of invoking
      * methods on the writer, and queueing up additional commands (often, representing children of
      * the object that was invoked) to perform additional rendering.
-     * <p>
+     * <p/>
      * In this way, rendering is a tail recursive algorithm, but is not implemented using tail
      * recursion.
      */

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/RenderQueue.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/RenderQueue.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/RenderQueue.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/RenderQueue.java Mon Nov 12 14:22:31 2007
@@ -20,6 +20,8 @@
  */
 public interface RenderQueue
 {
-    /** Adds the new command to the front of the queue. */
+    /**
+     * Adds the new command to the front of the queue.
+     */
     void push(RenderCommand command);
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/package.html
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/package.html?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/package.html (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/runtime/package.html Mon Nov 12 14:22:31 2007
@@ -1,4 +1,5 @@
 <html>
-    <body>Contains interfaces that are added to component classes at <em>runtime</em>. These
-        interfaces are added using Tapestry built-in AOP support.</body>
+<body>Contains interfaces that are added to component classes at <em>runtime</em>. These
+interfaces are added using Tapestry built-in AOP support.
+</body>
 </html>

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ActionResponseGenerator.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ActionResponseGenerator.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ActionResponseGenerator.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ActionResponseGenerator.java Mon Nov 12 14:22:31 2007
@@ -19,7 +19,7 @@
 /**
  * An object that can generate the final response sent to the client as part of an action request.
  * This is almost always a redirect request.
- * 
+ *
  * @see ComponentEventResultProcessor
  */
 public interface ActionResponseGenerator

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/AliasContribution.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/AliasContribution.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/AliasContribution.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/AliasContribution.java Mon Nov 12 14:22:31 2007
@@ -15,11 +15,10 @@
 package org.apache.tapestry.services;
 
 import static org.apache.tapestry.ioc.internal.util.Defense.notNull;
+import org.apache.tapestry.ioc.internal.util.InternalUtils;
 
 import java.util.Formatter;
 
-import org.apache.tapestry.ioc.internal.util.InternalUtils;
-
 /**
  * A contribution into the {@link Alias} or AliasOverride service configuration.
  */
@@ -80,7 +79,9 @@
         return _contributionType;
     }
 
-    /** The contributed object, which will be made available. */
+    /**
+     * The contributed object, which will be made available.
+     */
     public T getObject()
     {
         return _object;

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/AliasManager.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/AliasManager.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/AliasManager.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/AliasManager.java Mon Nov 12 14:22:31 2007
@@ -27,7 +27,7 @@
      * identify a mode. Only contributions where the mode is blank or the mode matches the provided
      * mode are returned. Mode specific contributions quietly override non-specific contributions
      * (where the mode is blank).
-     * 
+     *
      * @param mode
      * @return map from contribution type to contribution object
      */

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationInitializer.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationInitializer.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationInitializer.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationInitializer.java Mon Nov 12 14:22:31 2007
@@ -1,23 +1,22 @@
-// Copyright 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.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
+// Copyright 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.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
 package org.apache.tapestry.services;
 
 /**
  * Service interface for initializing Tapestry for the application.
- * 
- * 
+ *
  * @see org.apache.tapestry.services.ApplicationInitializerFilter
  */
 public interface ApplicationInitializer

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationInitializerFilter.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationInitializerFilter.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationInitializerFilter.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationInitializerFilter.java Mon Nov 12 14:22:31 2007
@@ -1,23 +1,21 @@
-// Copyright 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.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
+// Copyright 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.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
 package org.apache.tapestry.services;
 
 /**
  * Filter interface for {@link org.apache.tapestry.services.ApplicationInitializer}.
- * 
- * 
  */
 public interface ApplicationInitializerFilter
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationStateContribution.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationStateContribution.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationStateContribution.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationStateContribution.java Mon Nov 12 14:22:31 2007
@@ -48,7 +48,9 @@
         return _creator;
     }
 
-    /** The name of the strategy used to control where the ASO is stored. */
+    /**
+     * The name of the strategy used to control where the ASO is stored.
+     */
     public String getStrategy()
     {
         return _strategy;

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationStateCreator.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationStateCreator.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationStateCreator.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationStateCreator.java Mon Nov 12 14:22:31 2007
@@ -17,11 +17,13 @@
 /**
  * Used by {@link ApplicationStateManager} and {@link ApplicationStatePersistenceStrategy} to create
  * an application state object on demand.
- * 
+ *
  * @param <T>
  */
 public interface ApplicationStateCreator<T>
 {
-    /** Create a new instance of an application state object. */
+    /**
+     * Create a new instance of an application state object.
+     */
     T create();
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationStateManager.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationStateManager.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationStateManager.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationStateManager.java Mon Nov 12 14:22:31 2007
@@ -26,19 +26,17 @@
      * configuration that determines how an instance is stored and created as needed. A requested
      * ASO not in the configuration is assumed to be created via a no-args constructor, and stored
      * in the session.
-     * 
+     *
      * @param <T>
-     * @param asoClass
-     *            identifies the ASO to access or create
+     * @param asoClass identifies the ASO to access or create
      * @return the ASO instance
      */
     <T> T get(Class<T> asoClass);
 
     /**
      * Returns true if the ASO already exists, false if it has not yet been created.
-     * 
-     * @param asoClass
-     *            used to select the ASO
+     *
+     * @param asoClass used to select the ASO
      * @return true if ASO exists, false if null
      */
     <T> boolean exists(Class<T> asoClass);
@@ -46,12 +44,10 @@
     /**
      * Stores a new ASO, replacing the existing ASO (if any). Storing the value null will delete the
      * ASO so that it may be re-created later.
-     * 
+     *
      * @param <T>
-     * @param asoClass
-     *            the type of ASO
-     * @param aso
-     *            the ASO instance
+     * @param asoClass the type of ASO
+     * @param aso      the ASO instance
      */
     <T> void set(Class<T> asoClass, T aso);
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationStatePersistenceStrategy.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationStatePersistenceStrategy.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationStatePersistenceStrategy.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationStatePersistenceStrategy.java Mon Nov 12 14:22:31 2007
@@ -28,14 +28,15 @@
 
     /**
      * Stores a new ASO, possibly replacing the existing one.
-     * 
+     *
      * @param <T>
      * @param asoClass
-     * @param aso
-     *            instance to store, or null to delete existing
+     * @param aso      instance to store, or null to delete existing
      */
     <T> void set(Class<T> asoClass, T aso);
 
-    /** Returns true if the ASO already exists, false if null. */
+    /**
+     * Returns true if the ASO already exists, false if null.
+     */
     <T> boolean exists(Class<T> asoClass);
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationStatePersistenceStrategySource.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationStatePersistenceStrategySource.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationStatePersistenceStrategySource.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ApplicationStatePersistenceStrategySource.java Mon Nov 12 14:22:31 2007
@@ -16,7 +16,7 @@
 
 /**
  * Used to provide access to stategies via a logical name for the stategy, such as "session".
- * <p>
+ * <p/>
  * Note: Ideally, this would just be internal to {@link ApplicationStateManager}, but Tapestry
  * IOC's one-configuration-per-service makes that awkward, so this part of that functionality
  * becomes its own service.
@@ -25,12 +25,10 @@
 {
     /**
      * Returns the named strategy.
-     * 
-     * @param name
-     *            of strategy to access
+     *
+     * @param name of strategy to access
      * @return the strategy
-     * @throws RuntimeException
-     *             if the name does not match a configured strategy
+     * @throws RuntimeException if the name does not match a configured strategy
      */
     ApplicationStatePersistenceStrategy get(String name);
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/AssetSource.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/AssetSource.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/AssetSource.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/AssetSource.java Mon Nov 12 14:22:31 2007
@@ -14,12 +14,12 @@
 
 package org.apache.tapestry.services;
 
-import java.util.Locale;
-
 import org.apache.tapestry.Asset;
 import org.apache.tapestry.ioc.Resource;
 import org.apache.tapestry.ioc.services.ThreadLocale;
 
+import java.util.Locale;
+
 /**
  * Used to find or create an {@link Asset} with a given path.
  */
@@ -30,27 +30,22 @@
      * the base resource) <em>or</em> it may have a prefix, such as "context:" or "classpath:", in
      * which case it is treated as a complete path within the indicated domain. The resulting
      * Resource is then localized (to the provided Locale) and returned as an Asset.
-     * <p>
+     * <p/>
      * The AssetSource caches its results, so a single Asset instance may be shared among many
      * different components.
-     * 
-     * @param baseResource
-     *            base resource for computing relative paths, or null to search the classpath
-     * @param path
-     *            relative to the base resource
-     * @param locale
-     *            locale to localize the final resource to, or null for the thread's current locale
+     *
+     * @param baseResource base resource for computing relative paths, or null to search the classpath
+     * @param path         relative to the base resource
+     * @param locale       locale to localize the final resource to, or null for the thread's current locale
      * @return the asset
      */
     Asset findAsset(Resource baseResource, String path, Locale locale);
 
     /**
      * Convienience for finding assets on the classpath.
-     * 
-     * @param path
-     *            path to the base resource, relative to classpath root
-     * @param locale
-     *            to localize the resource to
+     *
+     * @param path   path to the base resource, relative to classpath root
+     * @param locale to localize the resource to
      * @return the asset
      */
     Asset getClasspathAsset(String path, Locale locale);
@@ -58,7 +53,7 @@
     /**
      * Obtains a classpath alias in the current locale (as defined by the {@link ThreadLocale}
      * service).
-     * 
+     *
      * @param path
      * @return the asset
      */

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/BeanBlockContribution.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/BeanBlockContribution.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/BeanBlockContribution.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/BeanBlockContribution.java Mon Nov 12 14:22:31 2007
@@ -14,9 +14,8 @@
 
 package org.apache.tapestry.services;
 
-import static org.apache.tapestry.ioc.internal.util.Defense.notBlank;
-
 import org.apache.tapestry.corelib.components.Label;
+import static org.apache.tapestry.ioc.internal.util.Defense.notBlank;
 
 /**
  * A contribution to the {@link BeanBlockSource} service, defining a page name and block id (within
@@ -52,7 +51,9 @@
         return _dataType;
     }
 
-    /** The id of the block within the page. */
+    /**
+     * The id of the block within the page.
+     */
     public String getBlockId()
     {
         return _blockId;
@@ -68,7 +69,9 @@
         return _edit;
     }
 
-    /** The logical name of the page containing the block. */
+    /**
+     * The logical name of the page containing the block.
+     */
     public String getPageName()
     {
         return _pageName;

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/BeanBlockSource.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/BeanBlockSource.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/BeanBlockSource.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/BeanBlockSource.java Mon Nov 12 14:22:31 2007
@@ -22,7 +22,7 @@
  * A source of {@link Block}s used to display the properties of a bean (used by the {@link Grid}
  * component), or to edit the properties of a bean (used by the {@link BeanEditForm} component).
  * Contributions to this service define what properties may be editted.
- * 
+ *
  * @see DataTypeAnalyzer
  */
 public interface BeanBlockSource
@@ -30,31 +30,26 @@
     /**
      * Returns a block which can be used to present an editor for the given data type, in the form
      * of a field label and input field.
-     * 
-     * @param datatype
-     *            logical name for the type of data to be displayed
+     *
+     * @param datatype logical name for the type of data to be displayed
      * @return the Block
-     * @throws RuntimeException
-     *             if no appropriate block is available
+     * @throws RuntimeException if no appropriate block is available
      */
     Block getEditBlock(String datatype);
 
     /**
      * Returns a block which can be used to present an output for the given data type.
-     * 
-     * @param datatype
-     *            logical name for the type of data to be displayed
+     *
+     * @param datatype logical name for the type of data to be displayed
      * @return the Block
-     * @throws RuntimeException
-     *             if no appropriate block is available
+     * @throws RuntimeException if no appropriate block is available
      */
     Block getDisplayBlock(String datatype);
 
     /**
      * Checks to see if there is a display block for the indicated data type.
-     * 
-     * @param datatype
-     *            to check for
+     *
+     * @param datatype to check for
      * @return true if a block is available
      */
     boolean hasDisplayBlock(String datatype);

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/BeanModelSource.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/BeanModelSource.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/BeanModelSource.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/BeanModelSource.java Mon Nov 12 14:22:31 2007
@@ -30,21 +30,18 @@
      * {@link OrderBefore} annotation on the getter or setter methods. The labels for the properties are
      * derived from the property names, but if the component's message catalog has keys of the form
      * <code>propertyName-label</code>, then those will be used instead.
-     * <p>
+     * <p/>
      * Models are <em>mutable</em>, so they are not cached, a fresh instance is created each
      * time.
-     * 
-     * @param beanClass
-     *            class of object to be edited
-     * @param filterReadOnlyProperties
-     *            if true, then properties that are read-only will be skipped (leaving only
-     *            read-write properties). If false, then both read-only and read-write properties
-     *            will be included.
-     * @param resources
-     *            used when resolving resources, especially component messages (used to access
-     *            labels)
+     *
+     * @param beanClass                class of object to be edited
+     * @param filterReadOnlyProperties if true, then properties that are read-only will be skipped (leaving only
+     *                                 read-write properties). If false, then both read-only and read-write properties
+     *                                 will be included.
+     * @param resources                used when resolving resources, especially component messages (used to access
+     *                                 labels)
      * @return a model
      */
     BeanModel create(Class beanClass, boolean filterReadOnlyProperties,
-            ComponentResources resources);
+                     ComponentResources resources);
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/BindingFactory.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/BindingFactory.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/BindingFactory.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/BindingFactory.java Mon Nov 12 14:22:31 2007
@@ -25,25 +25,21 @@
 {
     /**
      * Creates a new binding instance.
-     * <p>
+     * <p/>
      * The binding represents a connection between the container and the component (the component is
      * usually the child of the component, though in a few cases, it is the component itself). In
      * most cases, the expression is evaluated in terms of the resources of the <em>container</em>
      * and the component is ignored.
-     * 
-     * @param description
-     *            of the binding, such as, "parameter foo"
-     * @param container
-     *            the component, as represented by its resources, for which a binding is to be
-     *            created.
-     * @param component
-     *            the component whose parameter is to be bound by the resulting binding (rarely
-     *            used)
+     *
+     * @param description of the binding, such as, "parameter foo"
+     * @param container   the component, as represented by its resources, for which a binding is to be
+     *                    created.
+     * @param component   the component whose parameter is to be bound by the resulting binding (rarely
+     *                    used)
      * @param expression
-     * @param location
-     *            from which the binding was generate, or null if not known
+     * @param location    from which the binding was generate, or null if not known
      * @return the new binding instance
      */
     Binding newBinding(String description, ComponentResources container,
-            ComponentResources component, String expression, Location location);
+                       ComponentResources component, String expression, Location location);
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/BindingSource.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/BindingSource.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/BindingSource.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/BindingSource.java Mon Nov 12 14:22:31 2007
@@ -28,47 +28,37 @@
      * appropriate {@link BindingFactory}, which recieves the description, the expression (after
      * the prefix), and the location. If the prefix doesn't exist, or if there's no prefix, then the
      * factory for the default prefix (often "literal") is used (and passed the full prefix).
-     * <p>
+     * <p/>
      * The binding represents a connection between the container and the component (the component is
      * usually the child of the container, though in a few cases, it is the component itself). In
      * most cases, the expression is evaluated in terms of the resources of the <em>container</em>
      * and the component is ignored.
-     * 
-     * @param description
-     *            description of the binding, such as "parameter foo"
-     * @param container
-     *            typically, the parent of the component
-     * @param component
-     *            the component whose parameter is to be bound
-     * @param defaultPrefix
-     *            the default prefix used when the expression itself does not have a prefix
-     * @param expression
-     *            the binding
-     * @param location
-     *            location assigned to the binding (or null if not known)
+     *
+     * @param description   description of the binding, such as "parameter foo"
+     * @param container     typically, the parent of the component
+     * @param component     the component whose parameter is to be bound
+     * @param defaultPrefix the default prefix used when the expression itself does not have a prefix
+     * @param expression    the binding
+     * @param location      location assigned to the binding (or null if not known)
      * @return a binding
      */
     Binding newBinding(String description, ComponentResources container,
-            ComponentResources component, String defaultPrefix, String expression, Location location);
+                       ComponentResources component, String defaultPrefix, String expression, Location location);
 
     /**
      * A simpler version of
      * {@link #newBinding(String, ComponentResources, ComponentResources, String, String, Location)}
      * that defaults the values for several parameters. This is used in most cases. The default
      * binding prefix will be "prop". Most often, this is used to create a new default binding.
-     * 
-     * @param description
-     *            description of the binding, such as "parameter foo"
-     * @param container
-     *            typically, the parent of the component. This value will be used as the container
-     *            <em>and</em> the component, so whatever type of expression is evaluated, will be
-     *            evaulated in terms of this component
-     * @param defaultPrefix
-     *            the default prefix used when the expression itself does not have a prefix
-     * @param expression
-     *            the binding
+     *
+     * @param description   description of the binding, such as "parameter foo"
+     * @param container     typically, the parent of the component. This value will be used as the container
+     *                      <em>and</em> the component, so whatever type of expression is evaluated, will be
+     *                      evaulated in terms of this component
+     * @param defaultPrefix the default prefix used when the expression itself does not have a prefix
+     * @param expression    the binding
      * @return a binding
      */
     Binding newBinding(String description, ComponentResources container, String defaultPrefix,
-            String expression);
+                       String expression);
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ClassTransformation.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ClassTransformation.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ClassTransformation.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ClassTransformation.java Mon Nov 12 14:22:31 2007
@@ -14,30 +14,29 @@
 
 package org.apache.tapestry.services;
 
-import java.lang.annotation.Annotation;
-import java.util.List;
-
 import javassist.CtBehavior;
-
 import org.apache.tapestry.ioc.AnnotationProvider;
 import org.slf4j.Logger;
 
+import java.lang.annotation.Annotation;
+import java.util.List;
+
 /**
  * Contains class-specific information used when transforming an raw class into an executable class.
  * Much of this information is somewhat like ordinary reflection, but applies to a class that has
  * not yet been loaded.
- * <p>
+ * <p/>
  * Transformation is primarily about identifying annotations on fields and on methods and changing
  * the class, adding new interfaces, fields and methods, and deleting some existing fields.
- * <p>
+ * <p/>
  * A ClassTransformation contains all the state data specific to a particular class being
  * transformed. A number of <em>workers</em> will operate upon the ClassTransformation to effect
  * the desired changes before the true class is loaded into memory.
- * <p>
+ * <p/>
  * Instances of this class are not designed to be thread safe, access to an instance should be
  * restricted to a single thread. In fact, the design of this type is to allow stateless singletons
  * in multiple threads to work on thread-specific data (within the ClassTransformation). *
- * <p>
+ * <p/>
  * The majority of methods concern the <em>declared</em> members (field and methods) of a specific
  * class, rather than any fields or methods inherited from a base class.
  */
@@ -52,9 +51,8 @@
      * Returns the name of a new member (field or method). Ensures that the resulting name does not
      * conflict with any existing member (declared by the underlying class, or inherited from a base
      * class).
-     * 
-     * @param suggested
-     *            the suggested value for the member
+     *
+     * @param suggested the suggested value for the member
      * @return a unique name for the member
      */
     String newMemberName(String suggested);
@@ -62,11 +60,9 @@
     /**
      * As with {@link #newMemberName(String)}, but the suggested name is constructed from the
      * prefix and base name. An underscore will seperate the prefix from the base name.
-     * 
-     * @param prefix
-     *            for the generated name
-     * @param baseName
-     *            an name, often of an existing field or method
+     *
+     * @param prefix   for the generated name
+     * @param baseName an name, often of an existing field or method
      * @return a unique name
      */
     String newMemberName(String prefix, String baseName);
@@ -81,7 +77,7 @@
 
     /**
      * Finds all methods defined in the class that are marked with the provided annotation.
-     * 
+     *
      * @param annotationClass
      * @return a list of method signature (which may be empty) in ascending order
      * @see #findMethods(MethodFilter)
@@ -90,9 +86,8 @@
 
     /**
      * Finds all methods matched by the provided filter.
-     * 
-     * @param filter
-     *            Passed each method signature, it may include or exclude each potential
+     *
+     * @param filter Passed each method signature, it may include or exclude each potential
      * @return a list of matching method signatures (which may be empty) in ascending order (by
      *         method name), but descending order (by parameter count) within overrides of a single
      *         method name.
@@ -102,40 +97,31 @@
     /**
      * Finds all unclaimed fields matched by the provided filter. Only considers unclaimed, private,
      * instance fields.
-     * 
-     * @param filter
-     *            passed each field name and field type
+     *
+     * @param filter passed each field name and field type
      * @return the names of all matched fields, in ascending order
      */
     List<String> findFields(FieldFilter filter);
 
     /**
      * Finds an annotation on a declared instance field.
-     * 
-     * @param <T>
-     *            constrains parameter and return value to Annotation types
-     * @param fieldName
-     *            the name of the field, which must exist
-     * @param annotationClass
-     *            the type of annotation to access
+     *
+     * @param <T>             constrains parameter and return value to Annotation types
+     * @param fieldName       the name of the field, which must exist
+     * @param annotationClass the type of annotation to access
      * @return the annotation if present, or null otherwise
-     * @throws IllegalArgumentException
-     *             if the fieldName does not correspond to a declared field
+     * @throws IllegalArgumentException if the fieldName does not correspond to a declared field
      */
     <T extends Annotation> T getFieldAnnotation(String fieldName, Class<T> annotationClass);
 
     /**
      * Finds an annotation on a declared method.
-     * 
-     * @param <T>
-     *            constrains parameter and return value to Annotation types
-     * @param method
-     *            the method signature to search
-     * @param annotationClass
-     *            the type of annotation to access
+     *
+     * @param <T>             constrains parameter and return value to Annotation types
+     * @param method          the method signature to search
+     * @param annotationClass the type of annotation to access
      * @return the annotation if present, or null otherwise
-     * @throws IllegalArgumentException
-     *             if the method signature does not correspond to a declared method
+     * @throws IllegalArgumentException if the method signature does not correspond to a declared method
      */
     <T extends Annotation> T getMethodAnnotation(TransformMethodSignature method, Class<T> annotationClass);
 
@@ -143,25 +129,20 @@
      * Claims a field so as to ensure that only a single annotation is applied to any single field.
      * When a transformation occurs (driven by a field annotation), the first thing that occurs is
      * to claim the field, on behalf of the annotation.
-     * 
-     * @param fieldName
-     *            the name of the field that is being claimed
-     * @param tag
-     *            a non-null object that represents why the field is being tagged (this is typically
-     *            a specific annotation on the field)
-     * @throws IllegalArgumentException
-     *             if the fieldName does not correspond to a declared instance field
-     * @throws IllegalStateException
-     *             if the field is already claimed for some other tag
+     *
+     * @param fieldName the name of the field that is being claimed
+     * @param tag       a non-null object that represents why the field is being tagged (this is typically
+     *                  a specific annotation on the field)
+     * @throws IllegalArgumentException if the fieldName does not correspond to a declared instance field
+     * @throws IllegalStateException    if the field is already claimed for some other tag
      */
     void claimField(String fieldName, Object tag);
 
     /**
      * Changes the field to be read only. Any existing code that changes the field will cause a
      * runtime exception.
-     * 
-     * @param fieldName
-     *            name of field to so change
+     *
+     * @param fieldName name of field to so change
      */
     void makeReadOnly(String fieldName);
 
@@ -174,17 +155,16 @@
 
     /**
      * Obtains the type of a declared instance field.
-     * 
+     *
      * @param fieldName
      * @return the type of the field, as a string
-     * @throws IllegalArgumentException
-     *             if the fieldName does not correspond to a declared instance field
+     * @throws IllegalArgumentException if the fieldName does not correspond to a declared instance field
      */
     String getFieldType(String fieldName);
 
     /**
      * Returns true if the indicated name is a private instance field.
-     * 
+     *
      * @param fieldName
      * @return true if field exists
      */
@@ -193,14 +173,11 @@
     /**
      * Defines a new declared field for the class. The suggestedName may be modified to ensure
      * uniqueness.
-     * 
-     * @param modifiers
-     *            modifiers for the field (typically, {@link java.lang.reflect.Modifier#PRIVATE})
-     * @param type
-     *            the type for the field, as a string
-     * @param suggestedName
-     *            the desired name for the field, which may be modified (for uniqueness) when
-     *            returned
+     *
+     * @param modifiers     modifiers for the field (typically, {@link java.lang.reflect.Modifier#PRIVATE})
+     * @param type          the type for the field, as a string
+     * @param suggestedName the desired name for the field, which may be modified (for uniqueness) when
+     *                      returned
      * @return the (uniqued) name for the field
      */
     String addField(int modifiers, String type, String suggestedName);
@@ -211,13 +188,10 @@
      * this method repeatedly with the same type and value will return the same field name. Caching
      * extends to the parent transformation, so that a value injected into a parent class will be
      * available (via the protected instance variable) to subclasses.
-     * 
-     * @param type
-     *            the type of object to inject
-     * @param suggestedName
-     *            the suggested name for the new field
-     * @param value
-     *            to be injected. This value is retained.
+     *
+     * @param type          the type of object to inject
+     * @param suggestedName the suggested name for the new field
+     * @param value         to be injected. This value is retained.
      * @return the actual name of the injected field
      */
     String addInjectedField(Class type, String suggestedName, Object value);
@@ -225,11 +199,9 @@
     /**
      * Converts the field into a read only field whose value is the provided value. This is used
      * when converting an existing field into a read-only injected value.
-     * 
-     * @param fieldName
-     *            name of field to convert
-     * @param value
-     *            the value provided by the field
+     *
+     * @param fieldName name of field to convert
+     * @param value     the value provided by the field
      */
     void injectField(String fieldName, Object value);
 
@@ -237,14 +209,12 @@
      * Transforms the class to implement the indicated interface. If the class (or its super class)
      * does not already implement the interface, then the interface is added, and default
      * implementations of any methods of the interface are added.
-     * <p>
+     * <p/>
      * TODO: Checking that the names of methods in the interface do not conflict with the names of
      * methods present in the (unmodified) class.
-     * 
-     * @param interfaceClass
-     *            the interface to be implemented by the class
-     * @throws IllegalArgumentException
-     *             if the interfaceClass argument does not represent an interface
+     *
+     * @param interfaceClass the interface to be implemented by the class
+     * @throws IllegalArgumentException if the interfaceClass argument does not represent an interface
      */
     void addImplementedInterface(Class interfaceClass);
 
@@ -252,40 +222,37 @@
      * Extends an existing method. The provided method body is inserted at the end of the existing
      * method (i.e. {@link javassist.CtBehavior#insertAfter(java.lang.String)}). To access or
      * change the return value, use the <code>$_</code> pseudo variable.
-     * <p>
+     * <p/>
      * The method may be declared in the class, or may be inherited from a super-class. For
      * inherited methods, a method is added that first invokes the super implementation. Use
      * {@link #addMethod(TransformMethodSignature, String)} when it is necessary to control when the
      * super-class method is invoked.
-     * 
-     * @param signature
-     *            the signature of the method to extend
-     * @param methodBody
-     *            the body of code
-     * @throws IllegalArgumentException
-     *             if the provided Javassist method body can not be compiled
+     *
+     * @param signature  the signature of the method to extend
+     * @param methodBody the body of code
+     * @throws IllegalArgumentException if the provided Javassist method body can not be compiled
      */
     void extendMethod(TransformMethodSignature methodSignature, String methodBody);
 
     /**
      * Inserts code at the beginning of a method body (i.e. {@link CtBehavior#insertBefore(String)}.
-     * <p>
+     * <p/>
      * The method may be declared in the class, or may be inherited from a super-class. For
      * inherited methods, a method is added that first invokes the super implementation. Use
      * {@link #addMethod(TransformMethodSignature, String)} when it is necessary to control when the
      * super-class method is invoked.
-     * 
+     *
      * @param methodSignature
      * @param methodBody
      * @throws IllegalArgumentException If the provided Javassist method body could not be compiled
      */
     void prefixMethod(TransformMethodSignature methodSignature, String methodBody);
-    
+
     /**
      * Returns the name of a field that provides the {@link org.apache.tapestry.ComponentResources}
      * for the transformed component. This will be a protected field, accessible to the class and
      * subclasses.
-     * 
+     *
      * @return name of field
      */
     String getResourcesFieldName();
@@ -301,9 +268,8 @@
 
     /**
      * Adds a statement to the constructor. The statement is added as is, though a newline is added.
-     * 
-     * @param statement
-     *            the statement to add, which should end with a semicolon
+     *
+     * @param statement the statement to add, which should end with a semicolon
      */
     void extendConstructor(String statement);
 
@@ -325,9 +291,8 @@
     /**
      * Removes a field entirely; this is useful for fields that are replaced entirely by computed
      * values.
-     * 
-     * @param fieldName
-     *            the name of the field to remove
+     *
+     * @param fieldName the name of the field to remove
      * @see #replaceReadAccess(String, String)
      * @see #replaceWriteAccess(String, String)
      */
@@ -346,14 +311,16 @@
      */
     Logger getLogger();
 
-    /** Returns the modifiers for the named field. */
+    /**
+     * Returns the modifiers for the named field.
+     */
     int getFieldModifiers(String fieldName);
 
     /**
      * Converts a signature to a string used to identify the method; this consists of the
      * {@link TransformMethodSignature#getMediumDescription()} appended with source file information and line
      * number information (when available).
-     * 
+     *
      * @param signature
      * @return a string that identifies the class, method name, types of parameters, source file and
      *         source line number

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ClasspathAssetAliasManager.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ClasspathAssetAliasManager.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ClasspathAssetAliasManager.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ClasspathAssetAliasManager.java Mon Nov 12 14:22:31 2007
@@ -19,9 +19,8 @@
     /**
      * Takes a resource path to a classpath resource and adds the asset path prefix to the path. May
      * also convert part of the path to an alias (based on the manager's configuration).
-     * 
-     * @param resourcePath
-     *            resource path on the classpath (with no leading slash)
+     *
+     * @param resourcePath resource path on the classpath (with no leading slash)
      * @return URL ready to send to the client
      */
     String toClientURL(String resourcePath);

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ClasspathProvider.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ClasspathProvider.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ClasspathProvider.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ClasspathProvider.java Mon Nov 12 14:22:31 2007
@@ -13,12 +13,12 @@
 // limitations under the License.
 
 package org.apache.tapestry.services;
-import static java.lang.annotation.ElementType.FIELD;
-import static java.lang.annotation.ElementType.PARAMETER;
-import static java.lang.annotation.RetentionPolicy.RUNTIME;
 
 import java.lang.annotation.Documented;
+import static java.lang.annotation.ElementType.FIELD;
+import static java.lang.annotation.ElementType.PARAMETER;
 import java.lang.annotation.Retention;
+import static java.lang.annotation.RetentionPolicy.RUNTIME;
 import java.lang.annotation.Target;
 
 
@@ -27,7 +27,7 @@
  * assets located on the classpath.
  */
 @Target(
-{ PARAMETER, FIELD })
+        {PARAMETER, FIELD})
 @Retention(RUNTIME)
 @Documented
 public @interface ClasspathProvider

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentActionRequestFilter.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentActionRequestFilter.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentActionRequestFilter.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentActionRequestFilter.java Mon Nov 12 14:22:31 2007
@@ -21,22 +21,16 @@
 {
     /**
      * Filter for a component action request.
-     * 
-     * @param logicalPageName
-     *            the page name containing the component, and the default component to render the
-     *            response
-     * @param nestedComponentId
-     *            the id of the component within the page
-     * @param eventType
-     *            the type of event to trigger on the component
-     * @param context
-     *            context information to provide to the event handler
-     * @param activationContext
-     *            activation context for the page
-     * @param handler
-     *            to delegate to
+     *
+     * @param logicalPageName   the page name containing the component, and the default component to render the
+     *                          response
+     * @param nestedComponentId the id of the component within the page
+     * @param eventType         the type of event to trigger on the component
+     * @param context           context information to provide to the event handler
+     * @param activationContext activation context for the page
+     * @param handler           to delegate to
      */
     ActionResponseGenerator handle(String logicalPageName, String nestedComponentId,
-            String eventType, String[] context, String[] activationContext,
-            ComponentActionRequestHandler handler);
+                                   String eventType, String[] context, String[] activationContext,
+                                   ComponentActionRequestHandler handler);
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentActionRequestHandler.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentActionRequestHandler.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentActionRequestHandler.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentActionRequestHandler.java Mon Nov 12 14:22:31 2007
@@ -21,7 +21,7 @@
  * Handler interface for action requests. Action requests <em>do things</em> such as process a
  * form submission or otherwise change state. In the majority of cases, after the action, a redirect
  * response is sent to the client which, in turn, causes a page render.
- * 
+ *
  * @see ActionLink
  * @see Form
  * @see ComponentActionRequestFilter
@@ -31,18 +31,14 @@
     /**
      * Handler for a component action request, which returns a response generator used to send the
      * final response to the client.
-     * 
-     * @param logicalPageName
-     *            the page name containing the component, and the default component to render the
-     *            response
-     * @param nestedComponentId
-     *            the id of the component within the page
-     * @param eventType
-     *            the type of event to trigger on the component
-     * @param context
-     *            context information to provide to the event handler
+     *
+     * @param logicalPageName   the page name containing the component, and the default component to render the
+     *                          response
+     * @param nestedComponentId the id of the component within the page
+     * @param eventType         the type of event to trigger on the component
+     * @param context           context information to provide to the event handler
      * @parram activationContext activation context for the page
      */
     ActionResponseGenerator handle(String logicalPageName, String nestedComponentId,
-            String eventType, String[] context, String[] activationContext);
+                                   String eventType, String[] context, String[] activationContext);
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentClassResolver.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentClassResolver.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentClassResolver.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentClassResolver.java Mon Nov 12 14:22:31 2007
@@ -21,16 +21,16 @@
  * be provided by the application or inside a <em>mapped package</em>. Page names often appear
  * inside URLs, and component types often appear in component template (when specifying the type of
  * an embedded component).
- * <p>
+ * <p/>
  * The service is configured using a collection of {@link LibraryMapping}s. Each mapping maps a
  * prefix, such as "core" to a root package name, such as "org.apache.tapestry.corelib". The root
  * package is expected to have sub-packages: "pages", "components", "mixins" and "base" ("base" is
  * for base classes).
- * <p>
+ * <p/>
  * The resolver performs a search of the classpath (via {@link ClassNameLocator}, to build up a set
  * of case-insensitive maps from logical page name, component type, or mixin type to fully qualified
  * class name.
- * <p>
+ * <p/>
  * Certain ambiguities occur if mapped packages overlap, either in terms of the the prefixes or the
  * package names. Keep things clearly seperate to avoid lookup problems.
  */
@@ -39,21 +39,18 @@
     /**
      * Converts a logical page name (such as might be encoded into a URL) into a fully qualified
      * class name. The case of the page name is irrelevant.
-     * 
-     * @param logicalPageName
-     *            logical page name
+     *
+     * @param logicalPageName logical page name
      * @return fully qualified class name for the page
-     * @throws IllegalArgumentException
-     *             if the name does not match a known page class
+     * @throws IllegalArgumentException if the name does not match a known page class
      */
     String resolvePageNameToClassName(String logicalPageName);
 
     /**
      * For a particular path, determines if the path is a logical page name. The check is case
      * insensitive.
-     * 
-     * @param pageName
-     *            potential logical page name
+     *
+     * @param pageName potential logical page name
      * @return true if the page name is valid
      */
     boolean isPageName(String pageName);
@@ -62,45 +59,38 @@
      * Converts a fully qualified page class name into a logical class name (often, for inclusion as
      * part of the URI). This value may later be passed to
      * {@link #resolvePageNameToClassName(String)}.
-     * 
-     * @param pageClassName
-     *            fully qualified name of a page class
+     *
+     * @param pageClassName fully qualified name of a page class
      * @return equivalent logical page name
-     * @throws IllegalArgumentException
-     *             if the name can not be resolved
+     * @throws IllegalArgumentException if the name can not be resolved
      */
     String resolvePageClassNameToPageName(String pageClassName);
 
     /**
      * Returns the canonical form of a logical page name. The canonical form uses character case
      * matching the underlying class name.
-     * 
-     * @throws IllegalArgumentException
-     *             if the page name does not match a logical page name
+     *
+     * @throws IllegalArgumentException if the page name does not match a logical page name
      */
     String canonicalizePageName(String pageName);
 
     /**
      * Converts a component type (a logical component name such as might be used inside a template
      * or annotation) into a fully qualified class name. Case is ignored in resolving the name.
-     * 
-     * @param componentType
-     *            a logical component type
+     *
+     * @param componentType a logical component type
      * @return fully qualified class name
-     * @throws IllegalArgumentException
-     *             if the component type can not be resolved
+     * @throws IllegalArgumentException if the component type can not be resolved
      */
     String resolveComponentTypeToClassName(String componentType);
 
     /**
      * Converts a logical mixin type (as with component types) into a fully qualified class name.
      * Case is ignored when resolving the name.
-     * 
-     * @param mixinType
-     *            a logical mixin type
+     *
+     * @param mixinType a logical mixin type
      * @return fully qualified class name
-     * @throws IllegalArgumentException
-     *             if the mixin type can not be resolved
+     * @throws IllegalArgumentException if the mixin type can not be resolved
      */
     String resolveMixinTypeToClassName(String mixinType);
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentClassTransformWorker.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentClassTransformWorker.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentClassTransformWorker.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentClassTransformWorker.java Mon Nov 12 14:22:31 2007
@@ -1,27 +1,25 @@
-// Copyright 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.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
+// Copyright 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.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
 package org.apache.tapestry.services;
 
-import org.apache.tapestry.model.MutableComponentModel;
+import org.apache.tapestry.model.MutableComponentModel;
 
 /**
  * Interface for a set of objects that can perform component class transformations. Implementations
  * should be multithreaded, ideally they should be stateless (all necessary state can be stored in
  * the {@link org.apache.tapestry.services.ClassTransformation}).
- * 
- * 
  */
 public interface ComponentClassTransformWorker
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentDefaultProvider.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentDefaultProvider.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentDefaultProvider.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentDefaultProvider.java Mon Nov 12 14:22:31 2007
@@ -27,7 +27,7 @@
 {
     /**
      * Computes the default label for the component (which will generally be a {@link Field}).
-     * 
+     *
      * @param resources
      * @return the label, either extracted from the component's container's message catalog, or
      *         derived from the component's {@link ComponentResourcesCommon#getId()}.
@@ -38,11 +38,9 @@
      * Checks to see if the container of the component (identified by its resources) contains a
      * property matching the component's id. If so, a binding for that property is returned. This is
      * usually the default for a {@link Field}'s value parameter (or equivalent).
-     * 
-     * @param parameterName
-     *            the name of the parameter
-     * @param resources
-     *            the resources of the component for which a binding is needed
+     *
+     * @param parameterName the name of the parameter
+     * @param resources     the resources of the component for which a binding is needed
      * @return the binding, or null if the container does not have a matching property
      */
     Binding defaultBinding(String parameterName, ComponentResources resources);

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentEventResultProcessor.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentEventResultProcessor.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentEventResultProcessor.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentEventResultProcessor.java Mon Nov 12 14:22:31 2007
@@ -18,7 +18,7 @@
 
 /**
  * Responsible for handling the return value provided by a component event handler.
- * 
+ *
  * @param <T>
  */
 public interface ComponentEventResultProcessor<T>
@@ -26,17 +26,13 @@
     /**
      * For a given, non-null return value, provide a corresponding Link object (which will
      * ultimately be transformed into a URL and sent to the client as a redirect).
-     * 
-     * @param value
-     *            the value returned from a method
-     * @param component
-     *            the component on which a method was invoked
-     * @param methodDescripion
-     *            a description of method which provided the value
+     *
+     * @param value            the value returned from a method
+     * @param component        the component on which a method was invoked
+     * @param methodDescripion a description of method which provided the value
      * @return an object that can send the request to the client
-     * @throws RuntimeException
-     *             if the value can not be converted into a link
+     * @throws RuntimeException if the value can not be converted into a link
      */
     ActionResponseGenerator processComponentEvent(T value, Component component,
-            String methodDescripion);
+                                                  String methodDescripion);
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentLayer.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentLayer.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentLayer.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentLayer.java Mon Nov 12 14:22:31 2007
@@ -14,16 +14,15 @@
 
 package org.apache.tapestry.services;
 
-import static java.lang.annotation.ElementType.FIELD;
-import static java.lang.annotation.ElementType.PARAMETER;
-import static java.lang.annotation.RetentionPolicy.RUNTIME;
+import org.apache.tapestry.ioc.services.Builtin;
 
 import java.lang.annotation.Documented;
+import static java.lang.annotation.ElementType.FIELD;
+import static java.lang.annotation.ElementType.PARAMETER;
 import java.lang.annotation.Retention;
+import static java.lang.annotation.RetentionPolicy.RUNTIME;
 import java.lang.annotation.Target;
 
-import org.apache.tapestry.ioc.services.Builtin;
-
 /**
  * Used to identify a service from the component layer that conflicts, in terms of service
  * interface, with a service from elsewhere. In particular, this is used to disambiguate
@@ -31,7 +30,7 @@
  * another with this annotation.
  */
 @Target(
-{ PARAMETER, FIELD })
+        {PARAMETER, FIELD})
 @Retention(RUNTIME)
 @Documented
 public @interface ComponentLayer

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentMessagesSource.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentMessagesSource.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentMessagesSource.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentMessagesSource.java Mon Nov 12 14:22:31 2007
@@ -14,13 +14,15 @@
 
 package org.apache.tapestry.services;
 
-import java.util.Locale;
-
 import org.apache.tapestry.internal.event.InvalidationEventHub;
 import org.apache.tapestry.ioc.Messages;
 import org.apache.tapestry.model.ComponentModel;
 
-/** Used to connect a Tapestry component to its message catalog. */
+import java.util.Locale;
+
+/**
+ * Used to connect a Tapestry component to its message catalog.
+ */
 public interface ComponentMessagesSource extends InvalidationEventHub
 {
     /**
@@ -28,7 +30,7 @@
      * locale. If the component extends from another component, then its localized properties will
      * merge with its parent's properties (with the subclass overriding the super class on any
      * conflicts).
-     * 
+     *
      * @param componentModel
      * @param locale
      * @return the message catalog for the component, in the indicated locale

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentSource.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentSource.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentSource.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ComponentSource.java Mon Nov 12 14:22:31 2007
@@ -27,12 +27,10 @@
 {
     /**
      * Gets a component by its id.
-     * 
-     * @param componentId
-     *            complete component id
+     *
+     * @param componentId complete component id
      * @return the component
-     * @throws IllegalArgumentException
-     *             if the component can not be found
+     * @throws IllegalArgumentException if the component can not be found
      * @see ComponentResourcesCommon#getCompleteId()
      */
     Component getComponent(String componentId);
@@ -40,12 +38,10 @@
     /**
      * Returns the page identified by its logical page name. A logical page name is the short form
      * of a page name often emebedded into URLs.
-     * 
-     * @param pageName
-     *            the logical page name
+     *
+     * @param pageName the logical page name
      * @return the corresponding page's root component
-     * @throws IllegalArgumentException
-     *             if the page can not be found
+     * @throws IllegalArgumentException if the page can not be found
      */
     Component getPage(String pageName);
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/Context.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/Context.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/Context.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/Context.java Mon Nov 12 14:22:31 2007
@@ -26,31 +26,31 @@
     /**
      * Returns a URL to a resource stored within the context. The path should start with a leading
      * slash.
-     * 
+     *
      * @param path
      * @return the URL for the path, or null if the path does not correspond to a file.
      */
     URL getResource(String path);
 
-    /** Returns an initial parameter value defined by servlet. */
+    /**
+     * Returns an initial parameter value defined by servlet.
+     */
     String getInitParameter(String name);
 
     /**
      * Looks for resources within the web application within the supplied path. The list will be
      * recurively expanded, as necessary. The path must start with a leading slash, and usually ends
      * with a slash as well.
-     * 
-     * @param path
-     *            to search for (should start with a leading slash)
+     *
+     * @param path to search for (should start with a leading slash)
      * @return the matches, sorted alphabetically
      */
     List<String> getResourcePaths(String path);
 
     /**
      * Returns an attribute previously stored into the context with the given name.
-     * 
-     * @param name
-     *            used to retrieve the attribute
+     *
+     * @param name used to retrieve the attribute
      * @return the attribute, or null if not found
      */
     Object getAttribute(String name);

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ContextProvider.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ContextProvider.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ContextProvider.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/ContextProvider.java Mon Nov 12 14:22:31 2007
@@ -14,12 +14,11 @@
 
 package org.apache.tapestry.services;
 
+import java.lang.annotation.Documented;
 import static java.lang.annotation.ElementType.FIELD;
 import static java.lang.annotation.ElementType.PARAMETER;
-import static java.lang.annotation.RetentionPolicy.RUNTIME;
-
-import java.lang.annotation.Documented;
 import java.lang.annotation.Retention;
+import static java.lang.annotation.RetentionPolicy.RUNTIME;
 import java.lang.annotation.Target;
 
 /**
@@ -27,7 +26,7 @@
  * assets located in the web application context.
  */
 @Target(
-{ PARAMETER, FIELD })
+        {PARAMETER, FIELD})
 @Retention(RUNTIME)
 @Documented
 public @interface ContextProvider

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/Cookies.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/Cookies.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/Cookies.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/Cookies.java Mon Nov 12 14:22:31 2007
@@ -37,13 +37,10 @@
 
     /**
      * As with {@link #writeCookieValue(String, String)} but an explicit maximum age may be set.
-     * 
-     * @param name
-     *            the name of the cookie
-     * @param value
-     *            the value to be stored in the cookie
-     * @param maxAge
-     *            the maximum age, in seconds, to store the cookie
+     *
+     * @param name   the name of the cookie
+     * @param value  the value to be stored in the cookie
+     * @param maxAge the maximum age, in seconds, to store the cookie
      */
 
     void writeCookieValue(String name, String value, int maxAge);

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/Core.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/Core.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/Core.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/Core.java Mon Nov 12 14:22:31 2007
@@ -14,19 +14,18 @@
 
 package org.apache.tapestry.services;
 
+import java.lang.annotation.Documented;
 import static java.lang.annotation.ElementType.FIELD;
 import static java.lang.annotation.ElementType.PARAMETER;
-import static java.lang.annotation.RetentionPolicy.RUNTIME;
-
-import java.lang.annotation.Documented;
 import java.lang.annotation.Retention;
+import static java.lang.annotation.RetentionPolicy.RUNTIME;
 import java.lang.annotation.Target;
 
 /**
  * Marker for services that are provided by the Tapestry core module.
  */
 @Target(
-{ PARAMETER, FIELD })
+        {PARAMETER, FIELD})
 @Retention(RUNTIME)
 @Documented
 public @interface Core

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/DataTypeAnalyzer.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/DataTypeAnalyzer.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/DataTypeAnalyzer.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/DataTypeAnalyzer.java Mon Nov 12 14:22:31 2007
@@ -24,13 +24,15 @@
  * what kind of interface to use for displaying the value of the property, or what kind of interface
  * to use for editing the value of the property. Common property types are "text", "enum",
  * "checkbox", but the list is extensible.
- * 
+ *
  * @see Grid
  * @see BeanEditForm
  * @see BeanBlockSource
  */
 public interface DataTypeAnalyzer
 {
-    /** Identifies the data type, if known, or returns null if not known. */
+    /**
+     * Identifies the data type, if known, or returns null if not known.
+     */
     String identifyDataType(PropertyAdapter adapter);
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/DefaultPageRenderCommand.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/DefaultPageRenderCommand.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/DefaultPageRenderCommand.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/DefaultPageRenderCommand.java Mon Nov 12 14:22:31 2007
@@ -21,12 +21,16 @@
  */
 public abstract class DefaultPageRenderCommand implements PageRenderCommand
 {
-    /** Does nothing. */
+    /**
+     * Does nothing.
+     */
     public void cleanup(Environment environment, Document document)
     {
     }
 
-    /** Does nothing. */
+    /**
+     * Does nothing.
+     */
     public void setup(Environment environment, Document document)
     {
     }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/Dispatcher.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/Dispatcher.java?rev=594319&r1=594318&r2=594319&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/Dispatcher.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/services/Dispatcher.java Mon Nov 12 14:22:31 2007
@@ -24,7 +24,7 @@
 {
     /**
      * Analyzes the incoming request and performs an appropriate operation for each.
-     * 
+     *
      * @return true if a response was delivered, false if the servlet container should be allowed to
      *         handle the request
      */