From 5a84b129047e96f6fc73b56c73dd6d2d876c9ad9 Mon Sep 17 00:00:00 2001
From: Matt Tucker <matt@jivesoftware.com>
Date: Fri, 1 Aug 2003 21:14:51 +0000
Subject: [PATCH] Minor changes.

git-svn-id: http://svn.igniterealtime.org/svn/repos/smack/trunk@1995 b35dd754-fafc-0310-a699-88a17e54d16e
---
 .../smack/provider/ProviderManager.java       | 21 ++++++++++++-------
 1 file changed, 14 insertions(+), 7 deletions(-)

diff --git a/source/org/jivesoftware/smack/provider/ProviderManager.java b/source/org/jivesoftware/smack/provider/ProviderManager.java
index 5f8300aa0..43594ac4d 100644
--- a/source/org/jivesoftware/smack/provider/ProviderManager.java
+++ b/source/org/jivesoftware/smack/provider/ProviderManager.java
@@ -261,22 +261,29 @@ public class ProviderManager {
      * @param namespace the XML namespace.
      * @return the IQ provider.
      */
-    public synchronized static Object getIQProvider(String elementName, String namespace) {
+    public static Object getIQProvider(String elementName, String namespace) {
         String key = getProviderKey(elementName, namespace);
         return iqProviders.get(key);
     }
 
     /**
-     * Adds an IQ provider with the specified element name and name space. The provider
-     * will override any providers loaded through the classpath.
+     * Adds an IQ provider (must be an instance of IQProvider or Class object that is an IQ)
+     * with the specified element name and name space. The provider will override any providers
+     * loaded through the classpath.
      *
      * @param elementName the XML element name.
      * @param namespace the XML namespace.
      * @param provider the IQ provider.
      */
-    public synchronized static void addIQProvider(String elementName, String namespace,
-            IQProvider provider)
+    public static void addIQProvider(String elementName, String namespace,
+            Object provider)
     {
+        if (!(provider instanceof IQProvider || (provider instanceof Class &&
+                IQ.class.isAssignableFrom((Class)provider))))
+        {
+            throw new IllegalArgumentException("Provider must be an IQProvider " +
+                    "or a Class instance.");
+        }
         String key = getProviderKey(elementName, namespace);
         iqProviders.put(key, provider);
     }
@@ -300,7 +307,7 @@ public class ProviderManager {
      * @param namespace
      * @return the extenion provider.
      */
-    public synchronized static Object getExtensionProvider(String elementName, String namespace) {
+    public static Object getExtensionProvider(String elementName, String namespace) {
         String key = getProviderKey(elementName, namespace);
         return extensionProviders.get(key);
     }
@@ -314,7 +321,7 @@ public class ProviderManager {
      * @param namespace the XML namespace.
      * @param provider the extension provider.
      */
-    public synchronized static void addExtensionProvider(String elementName, String namespace,
+    public static void addExtensionProvider(String elementName, String namespace,
             Object provider)
     {
         if (!(provider instanceof PacketExtensionProvider || provider instanceof Class)) {