roster.remove_item not compliant

[roberto]

when using function roster.add_item:

- a string jid - a roster item

may be passed. instead, when using function roster.remove_item:

- only a JID object can be passed.

=> roster.remove_item should comply and accept:

- a string jid or - a roster item.

r.

[jajcus]

add_item() accepts RosterItem?() object as more than a JID is needed to add a full roster entry (roster entry is not just a JID). remove_item() only needs an identifier of an existing roster item to remove. And JID is the natural primary key of the roster. Making remove_item() accept RosterItem?() would be only unnecessary complication of the code. And doing it right (with changing argument name) would break the API too. There is no need for these two methods to be symmetrical.