You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@avalon.apache.org by bl...@apache.org on 2003/06/02 16:44:48 UTC
cvs commit: avalon-excalibur/fortress/src/java/org/apache/avalon/fortress/util ContextManager.java
bloritsch 2003/06/02 07:44:48
Modified: datasource build.xml
fortress/src/java/org/apache/avalon/fortress/impl/factory
BCELWrapperGenerator.java
fortress/src/java/org/apache/avalon/fortress/util
ContextManager.java
Log:
Remove a couple TODO items, and make sure i18n is used where needed.
Revision Changes Path
1.56 +1 -0 avalon-excalibur/datasource/build.xml
Index: build.xml
===================================================================
RCS file: /home/cvs/avalon-excalibur/datasource/build.xml,v
retrieving revision 1.55
retrieving revision 1.56
diff -u -r1.55 -r1.56
--- build.xml 2 Jun 2003 14:12:01 -0000 1.55
+++ build.xml 2 Jun 2003 14:44:46 -0000 1.56
@@ -42,6 +42,7 @@
<pathelement location="${excalibur-testcase.jar}"/>
<pathelement location="${excalibur-component.jar}"/>
<pathelement location="${excalibur-logger.jar}"/>
+ <pathelement location="${excalibur-i18n.jar}"/>
<pathelement location="${test.jdbc.driver.jar}"/>
<path refid="project.class.path"/>
</path>
1.8 +2 -2 avalon-excalibur/fortress/src/java/org/apache/avalon/fortress/impl/factory/BCELWrapperGenerator.java
Index: BCELWrapperGenerator.java
===================================================================
RCS file: /home/cvs/avalon-excalibur/fortress/src/java/org/apache/avalon/fortress/impl/factory/BCELWrapperGenerator.java,v
retrieving revision 1.7
retrieving revision 1.8
diff -u -r1.7 -r1.8
--- BCELWrapperGenerator.java 1 Jun 2003 13:32:09 -0000 1.7
+++ BCELWrapperGenerator.java 2 Jun 2003 14:44:46 -0000 1.8
@@ -224,7 +224,7 @@
/**
*/
- public Class createWrapper( final Class classToWrap ) throws Exception
+ public synchronized Class createWrapper( final Class classToWrap ) throws Exception
{
if ( classToWrap == null )
{
@@ -264,7 +264,7 @@
m_classGenerator );
final byte[] byteCode = buildWrapper( javaInterfacesToImplement );
- // TODO: Check synchronization
+
Class generatedClass;
synchronized ( m_bcelClassLoader )
{
1.38 +3 -3 avalon-excalibur/fortress/src/java/org/apache/avalon/fortress/util/ContextManager.java
Index: ContextManager.java
===================================================================
RCS file: /home/cvs/avalon-excalibur/fortress/src/java/org/apache/avalon/fortress/util/ContextManager.java,v
retrieving revision 1.37
retrieving revision 1.38
diff -u -r1.37 -r1.38
--- ContextManager.java 30 May 2003 19:23:12 -0000 1.37
+++ ContextManager.java 2 Jun 2003 14:44:46 -0000 1.38
@@ -846,9 +846,9 @@
}
else
{
- /**
+ /*
* We rely on namespace handing being turned off in DefaultConfiguration
- * builder here. TODO: add code that test
+ * builder here. TODO: add code that tests
* root element for name "configuration" and for the correct Log4J
* configuration namespace (not currently known to me - Anton Tagunov)
* to survive if a namespace-enabled configuration has been passed to us.
---------------------------------------------------------------------
To unsubscribe, e-mail: cvs-unsubscribe@avalon.apache.org
For additional commands, e-mail: cvs-help@avalon.apache.org