You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by gr...@apache.org on 2003/08/18 15:38:52 UTC
cvs commit: incubator-geronimo/modules/core/src/java/org/apache/geronimo/common AbstractContainer.java
gregw 2003/08/18 06:38:52
Modified: modules/core/src/java/org/apache/geronimo/common
AbstractContainer.java
Log:
Containers can contain containers.
That's the whole point of the JSR 77 model (with recursive start etc.)
At the very least I want to use the model for webapps in webcontainers in
geronimo.
Revision Changes Path
1.4 +1 -21 incubator-geronimo/modules/core/src/java/org/apache/geronimo/common/AbstractContainer.java
Index: AbstractContainer.java
===================================================================
RCS file: /home/cvs/incubator-geronimo/modules/core/src/java/org/apache/geronimo/common/AbstractContainer.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -r1.3 -r1.4
--- AbstractContainer.java 16 Aug 2003 23:16:18 -0000 1.3
+++ AbstractContainer.java 18 Aug 2003 13:38:52 -0000 1.4
@@ -90,26 +90,6 @@
}
/**
- * A Container cannot be itself contained in a Container.
- *
- * NB. Do we agree this is the model?
- * @return always null
- */
- public Container getContainer() {
- return null;
- }
-
- /**
- * A Container cannot be itself contained in a Container.
- *
- * NB. Do we agree this is the model?
- * @throws java.lang.UnsupportedOperationException
- */
- public void setContainer() {
- throw new UnsupportedOperationException("Cannot call setContainer on a Container");
- }
-
- /**
* Add a component to the set for a Container.
* Subclasses might like to override this in order
* to check their state before allowing the addition.