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();
}