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 2010/07/11 21:06:21 UTC

svn commit: r963134 - in /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal: services/InternalClassTransformationImpl.java transform/AbstractIncludeAssetWorker.java

Author: hlship
Date: Sun Jul 11 19:06:20 2010
New Revision: 963134

URL: http://svn.apache.org/viewvc?rev=963134&view=rev
Log:
TAP5-1197: Change some created component fields from private to protected, for more efficient access

Modified:
    tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/InternalClassTransformationImpl.java
    tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/transform/AbstractIncludeAssetWorker.java

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/InternalClassTransformationImpl.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/InternalClassTransformationImpl.java?rev=963134&r1=963133&r2=963134&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/InternalClassTransformationImpl.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/InternalClassTransformationImpl.java Sun Jul 11 19:06:20 2010
@@ -1693,7 +1693,7 @@ public final class InternalClassTransfor
         Defense.notNull(type, "type");
         Defense.notNull(provider, "provider");
 
-        TransformField field = createField(Modifier.PRIVATE | Modifier.FINAL, type.getName(), suggestedName);
+        TransformField field = createField(Modifier.PROTECTED | Modifier.FINAL, type.getName(), suggestedName);
 
         String argReference = addConstructorArg(providerType, provider);
 

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/transform/AbstractIncludeAssetWorker.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/transform/AbstractIncludeAssetWorker.java?rev=963134&r1=963133&r2=963134&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/transform/AbstractIncludeAssetWorker.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/transform/AbstractIncludeAssetWorker.java Sun Jul 11 19:06:20 2010
@@ -82,7 +82,7 @@ public abstract class AbstractIncludeAss
 
     private FieldAccess createFieldForAssets(ClassTransformation transformation)
     {
-        TransformField field = transformation.createField(Modifier.PRIVATE, List.class.getName(), "includedAssets");
+        TransformField field = transformation.createField(Modifier.PROTECTED, List.class.getName(), "includedAssets");
 
         return field.getAccess();
     }