Re: MulticastSocketSyncChannelFactory should implement SyncChannelFactory?

[Alan D. Cabrera]

Oi.  I simply copied it from another factory but I cannot find it now.  
I'll fix this.

Regards,
Alan

Bill Dudney wrote:
Hi All,

I'm playing around with activeio and I found that 
MulticastSocketSyncChannelFactory does not implement 
SyncChannelFactory. It should right?

Figured this little patch was not worth a JIRA, be glad to file one if 
you'd prefer.

in case the license grant is important I grant you full rights to do 
whatever you wish with this code

TTFN,

Bill Dudney
MyFaces - http://myfaces.apache.org
Cayenne - http://incubator.apache.org/projects/cayenne.html


Index: 
/Users/bdudney/Development/activemq/activeio/activeio-core/src/main/java/org/apache/activeio/packet/sync/multicast/MulticastSocketSyncChannelFactory.java 

===================================================================
--- 
/Users/bdudney/Development/activemq/activeio/activeio-core/src/main/java/org/apache/activeio/packet/sync/multicast/MulticastSocketSyncChannelFactory.java    
(revision 397379)
+++ 
/Users/bdudney/Development/activemq/activeio/activeio-core/src/main/java/org/apache/activeio/packet/sync/multicast/MulticastSocketSyncChannelFactory.java    
(working copy)
@@ -22,13 +22,14 @@
import java.net.URI;
import org.apache.activeio.packet.sync.SyncChannel;
+import org.apache.activeio.packet.sync.SyncChannelFactory;
import org.apache.activeio.packet.sync.SyncChannelServer;
/**
  * @version $Revision: $ $Date: $
  */
-public class MulticastSocketSyncChannelFactory {
+public class MulticastSocketSyncChannelFactory implements 
SyncChannelFactory {
     public SyncChannel openSyncChannel(URI groupURI) throws 
IOException {
         if (groupURI == null) throw new 
IllegalArgumentException("group URI cannot be null");