Merge pull request #311 from Mayflower/develop

Revert "cleanup semicolons, whitespace and mutable default arguments"

1 files changed, 2 insertions, 4 deletions

diff --git a/sleekxmpp/roster/single.py b/sleekxmpp/roster/single.py
index e9ce4f21..f080ae8a 100644
--- a/sleekxmpp/roster/single.py
+++ b/sleekxmpp/roster/single.py
@@ -237,7 +237,8 @@ class RosterNode(object):
         if not self.xmpp.is_component:
             return self.update(jid, subscription='remove')
 
-    def update(self, jid, name=None, subscription=None, groups=None, block=True, timeout=None, callback=None):
+    def update(self, jid, name=None, subscription=None, groups=[],
+                     block=True, timeout=None, callback=None):
         """
         Update a JID's subscription information.
 
@@ -257,9 +258,6 @@ class RosterNode(object):
                             Will be executed when the roster is received.
                             Implies block=False.
         """
-        if not groups:
-            groups = []
-
         self[jid]['name'] = name
         self[jid]['groups'] = groups
         self[jid].save()