You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@avalon.apache.org by co...@apache.org on 2001/12/01 07:28:03 UTC
cvs commit: jakarta-avalon-cornerstone/apps/overlord/src/java/org/apache/avalon/overlord Overlord.java
colus 01/11/30 22:28:03
Modified: apps/overlord/src/java/org/apache/avalon/overlord
Overlord.java
Log:
Added message for Collector adding and removing.
Revision Changes Path
1.4 +10 -9 jakarta-avalon-cornerstone/apps/overlord/src/java/org/apache/avalon/overlord/Overlord.java
Index: Overlord.java
===================================================================
RCS file: /home/cvs/jakarta-avalon-cornerstone/apps/overlord/src/java/org/apache/avalon/overlord/Overlord.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -r1.3 -r1.4
--- Overlord.java 2001/11/28 07:43:42 1.3
+++ Overlord.java 2001/12/01 06:28:03 1.4
@@ -96,8 +96,6 @@
m_context, m_componentManager, conf );
addCollector( name, collector );
-
- getLogger().info( REZ.getString( "overlord.notice.collector.add", name, clazz ) );
}
}
@@ -164,24 +162,27 @@
private void addCollector( final String name, final Collector collector )
{
- if ( null == name || name.length() == 0 )
- {
- final String message = "Invalid collector name";
- throw new IllegalArgumentException( message );
- }
-
if ( m_collectors.containsKey( name ) )
{
- final String message = "Collector named " + name + " already exists";
+ final String message =
+ REZ.getString( "overlord.error.collector.exists", name );
throw new IllegalArgumentException( message );
}
m_collectors.put( name, collector );
+
+ final String message =
+ REZ.getString( "overlord.notice.collector.add", name );
+ getLogger().info( message );
}
private void removeCollector( final String name )
{
m_collectors.remove( name );
+
+ final String message =
+ REZ.getString( "overlord.notice.collector.remove", name );
+ getLogger().info( message );
}
/**
--
To unsubscribe, e-mail: <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>