You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tapestry.apache.org by hl...@apache.org on 2008/03/02 18:21:32 UTC

svn commit: r632791 [5/8] - in /tapestry/tapestry5/trunk: tapestry-core/src/main/java/org/apache/tapestry/ tapestry-core/src/main/java/org/apache/tapestry/annotations/ tapestry-core/src/main/java/org/apache/tapestry/corelib/components/ tapestry-core/sr...

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/NoOpComponentInvocationMap.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/NoOpComponentInvocationMap.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/NoOpComponentInvocationMap.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/NoOpComponentInvocationMap.java Sun Mar  2 09:21:16 2008
@@ -18,9 +18,8 @@
 import org.apache.tapestry.dom.Element;
 
 /**
- * The production implementation for
- * {@link org.apache.tapestry.internal.services.ComponentInvocationMap}. It does absolutely nothing
- * because it is not needed in production.
+ * The production implementation for {@link org.apache.tapestry.internal.services.ComponentInvocationMap}. It does
+ * absolutely nothing because it is not needed in production.
  */
 public class NoOpComponentInvocationMap implements ComponentInvocationMap
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/PageContentTypeAnalyzer.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/PageContentTypeAnalyzer.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/PageContentTypeAnalyzer.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/PageContentTypeAnalyzer.java Sun Mar  2 09:21:16 2008
@@ -20,8 +20,8 @@
 public interface PageContentTypeAnalyzer
 {
     /**
-     * Analyzes the meta-data for the page and identifies the correct
-     * {@link org.apache.tapestry.ContentType} (including encoding).
+     * Analyzes the meta-data for the page and identifies the correct {@link org.apache.tapestry.ContentType} (including
+     * encoding).
      *
      * @param page to be rendered
      * @return the content type

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/PageLoader.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/PageLoader.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/PageLoader.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/PageLoader.java Sun Mar  2 09:21:16 2008
@@ -21,11 +21,11 @@
 import java.util.Locale;
 
 /**
- * Instantiates a fully loaded, configured instance of a Tapestry page. This is a recursive process,
- * since part of loading a page is to load the page elements for the page, many of which are
- * components. Further, in some cases, the full component tree is not identified until after each
- * component's template is loaded. Because this is an expensive process, loaded pages will be used
- * for many requests (on behalf of many different users) and will be pooled between requests.
+ * Instantiates a fully loaded, configured instance of a Tapestry page. This is a recursive process, since part of
+ * loading a page is to load the page elements for the page, many of which are components. Further, in some cases, the
+ * full component tree is not identified until after each component's template is loaded. Because this is an expensive
+ * process, loaded pages will be used for many requests (on behalf of many different users) and will be pooled between
+ * requests.
  *
  * @see PagePool
  * @see RequestPageCache
@@ -35,13 +35,12 @@
     /**
      * Loads the page in the given locale.
      *
-     * @param logicalPageName the <em>canonicalized</em> logical name of the page, which will be made
-     *                        available via {@link Page#getLogicalName()} and
-     *                        {@link ComponentResourcesCommon#getPageName()} (for any component within the
-     *                        page).
-     * @param locale          the locale to load the page and its components , which will be made available via
-     *                        {@link Page#getLocale()} and {@link ComponentResourcesCommon#getLocale()} (for any
-     *                        component within the page)
+     * @param logicalPageName the <em>canonicalized</em> logical name of the page, which will be made available via
+     *                        {@link Page#getLogicalName()} and {@link ComponentResourcesCommon#getPageName()} (for any
+     *                        component within the page).
+     * @param locale          the locale to load the page and its components , which will be made available via {@link
+     *                        Page#getLocale()} and {@link ComponentResourcesCommon#getLocale()} (for any component
+     *                        within the page)
      * @see Page#getLocale()
      */
     Page loadPage(String logicalPageName, Locale locale);

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/PageTemplateLocator.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/PageTemplateLocator.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/PageTemplateLocator.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/PageTemplateLocator.java Sun Mar  2 09:21:16 2008
@@ -25,8 +25,8 @@
 public interface PageTemplateLocator
 {
     /**
-     * Given model, determines if the model is for a page (rather than a component) and if so, sees
-     * if there is a localized template for the page in the web application context.
+     * Given model, determines if the model is for a page (rather than a component) and if so, sees if there is a
+     * localized template for the page in the web application context.
      *
      * @param model  the component model defining the page to search for
      * @param locale the desired localization of the template

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/PropertyConduitSourceImpl.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/PropertyConduitSourceImpl.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/PropertyConduitSourceImpl.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/PropertyConduitSourceImpl.java Sun Mar  2 09:21:16 2008
@@ -90,10 +90,10 @@
     private final Map<MultiKey, PropertyConduit> _cache = newConcurrentMap();
 
     private static final MethodSignature GET_SIGNATURE = new MethodSignature(Object.class, "get",
-                                                                             new Class[]{Object.class}, null);
+                                                                             new Class[] { Object.class }, null);
 
     private static final MethodSignature SET_SIGNATURE = new MethodSignature(void.class, "set",
-                                                                             new Class[]{Object.class, Object.class},
+                                                                             new Class[] { Object.class, Object.class },
                                                                              null);
 
     private final Pattern SPLIT_AT_DOTS = Pattern.compile("\\.");
@@ -166,7 +166,8 @@
 
         ClassFab classFab = _classFactory.newClass(name, BasePropertyConduit.class);
 
-        classFab.addConstructor(new Class[]{Class.class, AnnotationProvider.class, String.class}, null, "super($$);");
+        classFab.addConstructor(new Class[] { Class.class, AnnotationProvider.class, String.class }, null,
+                                "super($$);");
 
         String[] terms = SPLIT_AT_DOTS.split(expression);
 

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/RequestEncodingInitializer.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/RequestEncodingInitializer.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/RequestEncodingInitializer.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/RequestEncodingInitializer.java Sun Mar  2 09:21:16 2008
@@ -15,8 +15,8 @@
 package org.apache.tapestry.internal.services;
 
 /**
- * Determines the reuest encoding for the given page and applies that to the request, so that
- * parameters may be properly decoded.
+ * Determines the reuest encoding for the given page and applies that to the request, so that parameters may be properly
+ * decoded.
  */
 public interface RequestEncodingInitializer
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/RequestPageCache.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/RequestPageCache.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/RequestPageCache.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/RequestPageCache.java Sun Mar  2 09:21:16 2008
@@ -17,17 +17,17 @@
 import org.apache.tapestry.internal.structure.Page;
 
 /**
- * Per-thread service that caches page instances for the duration of the request, and is also
- * responsible for tracking the active page (the page which will ultimately render the response).
+ * Per-thread service that caches page instances for the duration of the request, and is also responsible for tracking
+ * the active page (the page which will ultimately render the response).
  */
 public interface RequestPageCache
 {
     /**
-     * Gets the page via its page name, in the current locale. The page name is resolved to a class
-     * name, which is used to obtain the page (from the page pool).
+     * Gets the page via its page name, in the current locale. The page name is resolved to a class name, which is used
+     * to obtain the page (from the page pool).
      *
-     * @param logicalPageName the name of the page to retrieve (this is the logical page name, not the fully
-     *                        qualified class name)
+     * @param logicalPageName the name of the page to retrieve (this is the logical page name, not the fully qualified
+     *                        class name)
      * @return a page instance reserved for this request
      * @throws IllegalArgumentException if the name can not be resolved to a page instance
      */

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/RequestPageCacheImpl.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/RequestPageCacheImpl.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/RequestPageCacheImpl.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/RequestPageCacheImpl.java Sun Mar  2 09:21:16 2008
@@ -51,8 +51,8 @@
     }
 
     /**
-     * At the end of the request, when the thread cleanup event occurs, release any pages attached
-     * to the request back to the page pool.
+     * At the end of the request, when the thread cleanup event occurs, release any pages attached to the request back
+     * to the page pool.
      */
     public void threadDidCleanup()
     {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ResourceCache.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ResourceCache.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ResourceCache.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ResourceCache.java Sun Mar  2 09:21:16 2008
@@ -19,15 +19,15 @@
 import org.apache.tapestry.services.ResourceDigestGenerator;
 
 /**
- * Caches information about resources on the classpath. In addition, acts as an invalidation hub for
- * any resources for which information is obtained (when any of the resources are changed,
- * invalidation listeners are notified so they can clear their caches).
+ * Caches information about resources on the classpath. In addition, acts as an invalidation hub for any resources for
+ * which information is obtained (when any of the resources are changed, invalidation listeners are notified so they can
+ * clear their caches).
  */
 public interface ResourceCache extends InvalidationEventHub
 {
     /**
-     * Returns true if the path requires that the client URL for the resource include a digest to
-     * validate that the client is authorized to access the resource.
+     * Returns true if the path requires that the client URL for the resource include a digest to validate that the
+     * client is authorized to access the resource.
      *
      * @param resource
      * @return true if digest is required for the resource
@@ -47,8 +47,7 @@
      * Returns the time modified for the resource.
      *
      * @param resource
-     * @return the date time modified for the path, or a negative value if the resource does not
-     *         exist
+     * @return the date time modified for the path, or a negative value if the resource does not exist
      */
     long getTimeModified(Resource resource);
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ResourceCacheImpl.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ResourceCacheImpl.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ResourceCacheImpl.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ResourceCacheImpl.java Sun Mar  2 09:21:16 2008
@@ -25,7 +25,7 @@
 import java.util.Map;
 
 public class ResourceCacheImpl extends InvalidationEventHubImpl implements ResourceCache,
-                                                                           UpdateListener
+        UpdateListener
 {
     private final URLChangeTracker _tracker;
 

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ResourceStreamer.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ResourceStreamer.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ResourceStreamer.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ResourceStreamer.java Sun Mar  2 09:21:16 2008
@@ -20,8 +20,8 @@
 import java.io.IOException;
 
 /**
- * Responsible for streaming the contents of a resource to the client. The {@link Resource} to
- * stream is almost always a {@link ClasspathResource}.
+ * Responsible for streaming the contents of a resource to the client. The {@link Resource} to stream is almost always a
+ * {@link ClasspathResource}.
  */
 public interface ResourceStreamer
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/RootPathDispatcher.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/RootPathDispatcher.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/RootPathDispatcher.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/RootPathDispatcher.java Sun Mar  2 09:21:16 2008
@@ -21,8 +21,8 @@
 import java.io.IOException;
 
 /**
- * Recognizes a request for the application root (i.e., "/") and handles this the same as a render
- * request for the "Start" page.
+ * Recognizes a request for the application root (i.e., "/") and handles this the same as a render request for the
+ * "Start" page.
  */
 public class RootPathDispatcher implements Dispatcher
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ServiceAnnotationObjectProvider.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ServiceAnnotationObjectProvider.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ServiceAnnotationObjectProvider.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ServiceAnnotationObjectProvider.java Sun Mar  2 09:21:16 2008
@@ -20,9 +20,8 @@
 import org.apache.tapestry.ioc.ObjectProvider;
 
 /**
- * Adds support for the {@link Service} annotation (which can be applied to fields or parameters),
- * which is used to disambiguate injection when multiple services implement the same service
- * interface.
+ * Adds support for the {@link Service} annotation (which can be applied to fields or parameters), which is used to
+ * disambiguate injection when multiple services implement the same service interface.
  */
 public class ServiceAnnotationObjectProvider implements ObjectProvider
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ServiceInjectionProvider.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ServiceInjectionProvider.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ServiceInjectionProvider.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ServiceInjectionProvider.java Sun Mar  2 09:21:16 2008
@@ -21,8 +21,8 @@
 import org.apache.tapestry.services.InjectionProvider;
 
 /**
- * A very late worker related to the {@link Inject} annotation that, when all other forms of
- * injection have failed, matches the field type to a service interface.
+ * A very late worker related to the {@link Inject} annotation that, when all other forms of injection have failed,
+ * matches the field type to a service interface.
  */
 public class ServiceInjectionProvider implements InjectionProvider
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/SessionApplicationStatePersistenceStrategy.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/SessionApplicationStatePersistenceStrategy.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/SessionApplicationStatePersistenceStrategy.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/SessionApplicationStatePersistenceStrategy.java Sun Mar  2 09:21:16 2008
@@ -22,11 +22,11 @@
 /**
  * Stores ASOs in the {@link Session}, which will be created as necessary.
  * <p/>
- * TODO: Re-storing the object back into the session at the end of the request. That's going to
- * require some kind of end-of-request notification.
+ * TODO: Re-storing the object back into the session at the end of the request. That's going to require some kind of
+ * end-of-request notification.
  */
 public class SessionApplicationStatePersistenceStrategy implements
-                                                        ApplicationStatePersistenceStrategy
+        ApplicationStatePersistenceStrategy
 {
     static final String PREFIX = "aso:";
 

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/SessionPersistentFieldStrategy.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/SessionPersistentFieldStrategy.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/SessionPersistentFieldStrategy.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/SessionPersistentFieldStrategy.java Sun Mar  2 09:21:16 2008
@@ -20,15 +20,13 @@
 /**
  * A strategy for storing persistent page properties into the {@link Session session}.
  * <p/>
- * Builds attribute names as:
- * <code>state:<em>page-name</em>:<em>component-id</em>:<em>field-name</em></code>
+ * Builds attribute names as: <code>state:<em>page-name</em>:<em>component-id</em>:<em>field-name</em></code>
  */
 
 public class SessionPersistentFieldStrategy extends AbstractSessionPersistentFieldStrategy
 {
     /**
-     * Prefix used to identify keys stored in the session that are being used to store persistent
-     * field data.
+     * Prefix used to identify keys stored in the session that are being used to store persistent field data.
      */
     static final String PREFIX = "state:";
 

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/StaticFilesFilter.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/StaticFilesFilter.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/StaticFilesFilter.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/StaticFilesFilter.java Sun Mar  2 09:21:16 2008
@@ -22,8 +22,8 @@
 import java.net.URL;
 
 /**
- * Identifies requests that are for actual resource files in the context. For those, Tapestry allows
- * the servlet container to process the request.
+ * Identifies requests that are for actual resource files in the context. For those, Tapestry allows the servlet
+ * container to process the request.
  */
 public class StaticFilesFilter implements RequestFilter
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/StringProvider.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/StringProvider.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/StringProvider.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/StringProvider.java Sun Mar  2 09:21:16 2008
@@ -20,8 +20,8 @@
 public interface StringProvider
 {
     /**
-     * Ask the object to provide the desired string. Often this involves computing the string value
-     * dynamically, or aggregating together multiple StringProviders.
+     * Ask the object to provide the desired string. Often this involves computing the string value dynamically, or
+     * aggregating together multiple StringProviders.
      */
     String provideString();
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/TransformationException.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/TransformationException.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/TransformationException.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/TransformationException.java Sun Mar  2 09:21:16 2008
@@ -18,9 +18,8 @@
 import org.apache.tapestry.services.ClassTransformation;
 
 /**
- * Exception thrown when there is a failure transforming a class, or instantiating a transformed
- * class. The cause may be an Error. The goal is to get the {@link ClassTransformation} into the
- * exception report page, properly formatted.
+ * Exception thrown when there is a failure transforming a class, or instantiating a transformed class. The cause may be
+ * an Error. The goal is to get the {@link ClassTransformation} into the exception report page, properly formatted.
  */
 public class TransformationException extends TapestryException
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/UpdateListenerHub.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/UpdateListenerHub.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/UpdateListenerHub.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/UpdateListenerHub.java Sun Mar  2 09:21:16 2008
@@ -1,29 +1,28 @@
-// Copyright 2006, 2007 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, 2007 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.internal.services;
 
 import org.apache.tapestry.internal.events.UpdateListener;
 
 /**
- * Manages a set of {@link org.apache.tapestry.internal.events.UpdateListener}s. Periodically (say, every
- * request during development, or every minute or so during production), request processing is
- * locked down so that only a single thread is active, and the active thread invokes
- * {@link #fireUpdateEvent()}. Various services that are dependent on external resource files (such
- * as classes or template files) can check to see if any file they've used has changed. If so, the
- * service can invalidate its internal cache, or notify other services (typically via
- * {@link org.apache.tapestry.internal.events.InvalidationListener} that they should do the same.
+ * Manages a set of {@link org.apache.tapestry.internal.events.UpdateListener}s. Periodically (say, every request during
+ * development, or every minute or so during production), request processing is locked down so that only a single thread
+ * is active, and the active thread invokes {@link #fireUpdateEvent()}. Various services that are dependent on external
+ * resource files (such as classes or template files) can check to see if any file they've used has changed. If so, the
+ * service can invalidate its internal cache, or notify other services (typically via {@link
+ * org.apache.tapestry.internal.events.InvalidationListener} that they should do the same.
  *
  * @see org.apache.tapestry.internal.util.URLChangeTracker
  */
@@ -32,10 +31,9 @@
     void addUpdateListener(UpdateListener listener);
 
     /**
-     * Invoked periodically to allow services to check if underlying state has changed. For example,
-     * a template file may have changed. Listeners will typically notify applicable listeners of
-     * their own (they usually implement
-     * {@link org.apache.tapestry.internal.event.InvalidationEventHub}) when such a change occurs.
+     * Invoked periodically to allow services to check if underlying state has changed. For example, a template file may
+     * have changed. Listeners will typically notify applicable listeners of their own (they usually implement {@link
+     * org.apache.tapestry.internal.event.InvalidationEventHub}) when such a change occurs.
      */
     void fireUpdateEvent();
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ValidationMessagesSourceImpl.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ValidationMessagesSourceImpl.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ValidationMessagesSourceImpl.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ValidationMessagesSourceImpl.java Sun Mar  2 09:21:16 2008
@@ -64,8 +64,8 @@
     }
 
     /**
-     * Delegates to a {@link Messages} instance obtained from the {@link MessagesSource}. This
-     * ensures that changes to the underlying properties files will be reflected.
+     * Delegates to a {@link Messages} instance obtained from the {@link MessagesSource}. This ensures that changes to
+     * the underlying properties files will be reflected.
      */
     private class ValidationMessages implements Messages
     {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ValueEncoderSourceImpl.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ValueEncoderSourceImpl.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ValueEncoderSourceImpl.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/services/ValueEncoderSourceImpl.java Sun Mar  2 09:21:16 2008
@@ -35,7 +35,7 @@
         _registry = StrategyRegistry.newInstance(ValueEncoderFactory.class, configuration);
     }
 
-    @SuppressWarnings({"unchecked"})
+    @SuppressWarnings({ "unchecked" })
     public <T> ValueEncoder<T> getValueEncoder(Class<T> type)
     {
         Defense.notNull(type, "type");

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/structure/BodyPageElement.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/structure/BodyPageElement.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/structure/BodyPageElement.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/structure/BodyPageElement.java Sun Mar  2 09:21:16 2008
@@ -17,15 +17,15 @@
 import org.apache.tapestry.internal.services.PageLoader;
 
 /**
- * A type of {@link PageElement} that has a body that can be added to. This is part of the
- * constuction phase that is faciliated by the {@link PageLoader}.
+ * A type of {@link PageElement} that has a body that can be added to. This is part of the constuction phase that is
+ * faciliated by the {@link PageLoader}.
  */
 public interface BodyPageElement
 {
     /**
-     * Used during the construction of the page. Adds a page element as part of the body of the
-     * component. The body of a component is defined as the portion of the container's template
-     * directly enclosed by component's start and end elements.
+     * Used during the construction of the page. Adds a page element as part of the body of the component. The body of a
+     * component is defined as the portion of the container's template directly enclosed by component's start and end
+     * elements.
      */
     void addToBody(PageElement element);
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/structure/PageElement.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/structure/PageElement.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/structure/PageElement.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/structure/PageElement.java Sun Mar  2 09:21:16 2008
@@ -1,24 +1,24 @@
-// 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.internal.structure;
 
-import org.apache.tapestry.runtime.RenderCommand;
+import org.apache.tapestry.runtime.RenderCommand;
 
 /**
- * An element within a page. Page elements are placeholders that delegate much of their behavior to
- * application-specific component classes.
+ * An element within a page. Page elements are placeholders that delegate much of their behavior to application-specific
+ * component classes.
  */
 public interface PageElement extends RenderCommand
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/test/CodeEq.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/test/CodeEq.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/test/CodeEq.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/test/CodeEq.java Sun Mar  2 09:21:16 2008
@@ -1,27 +1,26 @@
-// 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.internal.test;
 
 import static org.easymock.EasyMock.reportMatcher;
 import org.easymock.IArgumentMatcher;
 
 /**
- * Special version of string equality used to compare two snippets of code. This is somewhat
- * simpleminded (it certainly doesn't understand about literal strings in quotes). It works by
- * eliminating unecessary whitespace around curly braces, then reducing all whitespace to a single
- * space.
+ * Special version of string equality used to compare two snippets of code. This is somewhat simpleminded (it certainly
+ * doesn't understand about literal strings in quotes). It works by eliminating unecessary whitespace around curly
+ * braces, then reducing all whitespace to a single space.
  */
 public class CodeEq implements IArgumentMatcher
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/test/PageTesterComponentInvocationMap.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/test/PageTesterComponentInvocationMap.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/test/PageTesterComponentInvocationMap.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/test/PageTesterComponentInvocationMap.java Sun Mar  2 09:21:16 2008
@@ -25,8 +25,8 @@
 import java.util.Map;
 
 /**
- * This is the real implementation, used by {@link PageTester}. The typical implementation,
- * {@link NoOpComponentInvocationMap}, is used in production as a place holder.
+ * This is the real implementation, used by {@link PageTester}. The typical implementation, {@link
+ * NoOpComponentInvocationMap}, is used in production as a place holder.
  */
 public class PageTesterComponentInvocationMap implements ComponentInvocationMap
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/test/TestableMarkupWriterFactory.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/test/TestableMarkupWriterFactory.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/test/TestableMarkupWriterFactory.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/test/TestableMarkupWriterFactory.java Sun Mar  2 09:21:16 2008
@@ -18,8 +18,8 @@
 import org.apache.tapestry.services.MarkupWriterFactory;
 
 /**
- * Extension of {@link MarkupWriterFactory} that tracks the most recently created markup writer so
- * that it can be accessed after the page has rendered.
+ * Extension of {@link MarkupWriterFactory} that tracks the most recently created markup writer so that it can be
+ * accessed after the page has rendered.
  */
 public interface TestableMarkupWriterFactory extends MarkupWriterFactory
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/test/TestableMarkupWriterFactoryImpl.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/test/TestableMarkupWriterFactoryImpl.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/test/TestableMarkupWriterFactoryImpl.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/test/TestableMarkupWriterFactoryImpl.java Sun Mar  2 09:21:16 2008
@@ -29,8 +29,7 @@
     private MarkupWriter _lastCreated;
 
     /**
-     * Using Core annotation to reference to framework-provided version, which this implementation wraps
-     * around.
+     * Using Core annotation to reference to framework-provided version, which this implementation wraps around.
      */
     public TestableMarkupWriterFactoryImpl(@Core MarkupWriterFactory delegate)
     {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/test/TestableRequest.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/test/TestableRequest.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/test/TestableRequest.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/test/TestableRequest.java Sun Mar  2 09:21:16 2008
@@ -20,9 +20,8 @@
 import java.util.Map;
 
 /**
- * An extended version of {@link Request} that allows the {@link PageTester} to control and override
- * behavior, effectively simulating the portions of {@link Request} that are provided normally by a
- * servlet container.
+ * An extended version of {@link Request} that allows the {@link PageTester} to control and override behavior,
+ * effectively simulating the portions of {@link Request} that are provided normally by a servlet container.
  */
 public interface TestableRequest extends Request
 {

Added: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/transform/GenerateAccessorsWorker.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/transform/GenerateAccessorsWorker.java?rev=632791&view=auto
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/transform/GenerateAccessorsWorker.java (added)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/transform/GenerateAccessorsWorker.java Sun Mar  2 09:21:16 2008
@@ -0,0 +1,66 @@
+// Copyright 2008 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.internal.transform;
+
+import org.apache.tapestry.annotations.GenerateAccessors;
+import org.apache.tapestry.ioc.internal.util.InternalUtils;
+import org.apache.tapestry.model.MutableComponentModel;
+import org.apache.tapestry.services.ClassTransformation;
+import org.apache.tapestry.services.ComponentClassTransformWorker;
+import org.apache.tapestry.services.TransformMethodSignature;
+
+import java.lang.reflect.Modifier;
+
+/**
+ * Provides the getter and setter methods. The methods are added as "existing", meaning that field access to them will
+ * be transformed as necessary by other annotations. This worker needs to be scheduled before any worker that might
+ * delete a field.
+ */
+public class GenerateAccessorsWorker implements ComponentClassTransformWorker
+{
+    public void transform(ClassTransformation transformation, MutableComponentModel model)
+    {
+        for (String fieldName : transformation.findAllFieldsWithAnnotation(GenerateAccessors.class))
+        {
+            GenerateAccessors annotation = transformation.getFieldAnnotation(fieldName, GenerateAccessors.class);
+
+            String propertyName = InternalUtils.capitalize(InternalUtils.stripMemberPrefix(fieldName));
+
+            String fieldType = transformation.getFieldType(fieldName);
+
+            if (annotation.read())
+            {
+                TransformMethodSignature getter
+                        = new TransformMethodSignature(Modifier.PUBLIC | Modifier.FINAL, fieldType,
+                                                       "get" + propertyName,
+                                                       null, null);
+
+                transformation.addTransformedMethod(getter, "return " + fieldName + ";");
+            }
+
+            if (annotation.write())
+            {
+                TransformMethodSignature setter
+                        = new TransformMethodSignature(Modifier.PUBLIC | Modifier.FINAL, "void", "set" + propertyName,
+                                                       new String[] { fieldType }, null);
+
+                transformation.addTransformedMethod(setter, fieldName + " = $1;");
+            }
+
+            // The field is NOT claimed, because we want annotations for the fields to operate.
+        }
+
+    }
+}

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/Base64InputStream.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/Base64InputStream.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/Base64InputStream.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/Base64InputStream.java Sun Mar  2 09:21:16 2008
@@ -1,17 +1,17 @@
-// 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.internal.util;
 
 import org.apache.commons.codec.binary.Base64;
@@ -19,8 +19,8 @@
 import java.io.ByteArrayInputStream;
 
 /**
- * An extension of {@link ByteArrayInputStream} that is initialized from a Base64 input stream
- * (rather than from a byte array).
+ * An extension of {@link ByteArrayInputStream} that is initialized from a Base64 input stream (rather than from a byte
+ * array).
  */
 public class Base64InputStream extends ByteArrayInputStream
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/Base64ObjectInputStream.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/Base64ObjectInputStream.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/Base64ObjectInputStream.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/Base64ObjectInputStream.java Sun Mar  2 09:21:16 2008
@@ -1,27 +1,27 @@
-// 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.internal.util;
 
-import java.io.BufferedInputStream;
-import java.io.IOException;
-import java.io.ObjectInputStream;
-import java.util.zip.GZIPInputStream;
+import java.io.BufferedInputStream;
+import java.io.IOException;
+import java.io.ObjectInputStream;
+import java.util.zip.GZIPInputStream;
 
 /**
- * A convienience for building a {@link ObjectInputStream} around a Base64 encoding (that originated
- * in {@link Base64ObjectOutputStream}).
+ * A convienience for building a {@link ObjectInputStream} around a Base64 encoding (that originated in {@link
+ * Base64ObjectOutputStream}).
  */
 public class Base64ObjectInputStream extends ObjectInputStream
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/Base64ObjectOutputStream.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/Base64ObjectOutputStream.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/Base64ObjectOutputStream.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/Base64ObjectOutputStream.java Sun Mar  2 09:21:16 2008
@@ -1,17 +1,17 @@
-// 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.internal.util;
 
 import java.io.BufferedOutputStream;
@@ -20,9 +20,8 @@
 import java.util.zip.GZIPOutputStream;
 
 /**
- * Wraps a {@link Base64OutputStream} with a {@link GZIPOutputStream} as an
- * {@link ObjectOutputStream}. This allows an object (or objects) to be encoded into a Base64
- * string (accessed via {@link #toBase64()}).
+ * Wraps a {@link Base64OutputStream} with a {@link GZIPOutputStream} as an {@link ObjectOutputStream}. This allows an
+ * object (or objects) to be encoded into a Base64 string (accessed via {@link #toBase64()}).
  *
  * @see Base64ObjectInputStream
  */

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/Base64OutputStream.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/Base64OutputStream.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/Base64OutputStream.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/Base64OutputStream.java Sun Mar  2 09:21:16 2008
@@ -1,17 +1,17 @@
-// 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.internal.util;
 
 import org.apache.commons.codec.binary.Base64;
@@ -19,8 +19,7 @@
 import java.io.ByteArrayOutputStream;
 
 /**
- * An extension of {@link ByteArrayOutputStream} that allows the final byte array to be converted to
- * a Base64 string.
+ * An extension of {@link ByteArrayOutputStream} that allows the final byte array to be converted to a Base64 string.
  */
 public final class Base64OutputStream extends ByteArrayOutputStream
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/Holder.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/Holder.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/Holder.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/Holder.java Sun Mar  2 09:21:16 2008
@@ -17,9 +17,9 @@
 import java.util.concurrent.atomic.AtomicReference;
 
 /**
- * An object that holds some type of other object. This is useful for communicating information from
- * an inner class (used as a closure) to the containing method. This is similar to
- * {@link AtomicReference}, except that it is simpler but <strong>not</strong> thread safe.
+ * An object that holds some type of other object. This is useful for communicating information from an inner class
+ * (used as a closure) to the containing method. This is similar to {@link AtomicReference}, except that it is simpler
+ * but <strong>not</strong> thread safe.
  *
  * @param <T>
  */

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/IntegerRange.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/IntegerRange.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/IntegerRange.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/IntegerRange.java Sun Mar  2 09:21:16 2008
@@ -1,24 +1,24 @@
-// 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.internal.util;
 
 import java.util.Iterator;
 
 /**
- * Represents a sequence of integer values, either ascending or descending. The sequence is always
- * inclusive (of the finish value).
+ * Represents a sequence of integer values, either ascending or descending. The sequence is always inclusive (of the
+ * finish value).
  */
 public final class IntegerRange implements Iterable<Integer>
 {
@@ -87,8 +87,8 @@
     }
 
     /**
-     * The main puprose of a range object is to produce an Iterator. Since IntegerRange is iterable,
-     * it is useful with the Tapestry Loop component, but also with the Java for loop!
+     * The main puprose of a range object is to produce an Iterator. Since IntegerRange is iterable, it is useful with
+     * the Tapestry Loop component, but also with the Java for loop!
      */
     public Iterator<Integer> iterator()
     {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/MethodInvocationBuilder.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/MethodInvocationBuilder.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/MethodInvocationBuilder.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/MethodInvocationBuilder.java Sun Mar  2 09:21:16 2008
@@ -22,8 +22,8 @@
 import java.util.Map;
 
 /**
- * A utility class for building part of a method body to invoke a method. Analyzes the method and
- * matches parameter types to ParameterBuilders.
+ * A utility class for building part of a method body to invoke a method. Analyzes the method and matches parameter
+ * types to ParameterBuilders.
  */
 public final class MethodInvocationBuilder
 {
@@ -50,9 +50,9 @@
     }
 
     /**
-     * Builds the method invocation. Analyzes the type of each parameter to the method, and uses a
-     * {@link ParameterBuilder} to provide the expression. Supplies a default value (usually null)
-     * for any parameters that do not have parameter builders.
+     * Builds the method invocation. Analyzes the type of each parameter to the method, and uses a {@link
+     * ParameterBuilder} to provide the expression. Supplies a default value (usually null) for any parameters that do
+     * not have parameter builders.
      *
      * @param signature      of the method to invoke
      * @param transformation

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/MultiKey.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/MultiKey.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/MultiKey.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/MultiKey.java Sun Mar  2 09:21:16 2008
@@ -17,8 +17,8 @@
 import java.util.Arrays;
 
 /**
- * Combines multiple values to form a single composite key. MultiKey can often be used as an
- * alternative to nested maps.
+ * Combines multiple values to form a single composite key. MultiKey can often be used as an alternative to nested
+ * maps.
  */
 public final class MultiKey
 {
@@ -29,9 +29,8 @@
     private final int _hashCode;
 
     /**
-     * Creates a new instance from the provided values. It is assumed that the values provided are
-     * good map keys themselves -- immutable, with proper implementations of equals() and
-     * hashCode().
+     * Creates a new instance from the provided values. It is assumed that the values provided are good map keys
+     * themselves -- immutable, with proper implementations of equals() and hashCode().
      *
      * @param values
      */

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/ParameterBuilder.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/ParameterBuilder.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/ParameterBuilder.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/ParameterBuilder.java Sun Mar  2 09:21:16 2008
@@ -1,24 +1,23 @@
-// 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.internal.util;
 
 import org.apache.tapestry.services.ClassTransformation;
 
 /**
- * Builds single parameter value to pass into a method being invoked by a
- * {@link org.apache.tapestry.internal.util.MethodInvocationBuilder}.
+ * Builds single parameter value to pass into a method being invoked by a {@link org.apache.tapestry.internal.util.MethodInvocationBuilder}.
  */
 public interface ParameterBuilder
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/StringParameterBuilder.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/StringParameterBuilder.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/StringParameterBuilder.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/util/StringParameterBuilder.java Sun Mar  2 09:21:16 2008
@@ -1,17 +1,17 @@
-// 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.internal.util;
 
 import org.apache.tapestry.services.ClassTransformation;
@@ -19,8 +19,8 @@
 import static java.lang.String.format;
 
 /**
- * Implementation of {@link org.apache.tapestry.internal.util.ParameterBuilder} that simply provides a static
- * string value for the parameter expression.
+ * Implementation of {@link org.apache.tapestry.internal.util.ParameterBuilder} that simply provides a static string
+ * value for the parameter expression.
  */
 public final class StringParameterBuilder implements ParameterBuilder
 {