Clean up and preparation work.

src/net/java/sip/communicator/impl/protocol/irc/IrcStack.java

@@ -686,8 +686,7 @@ public void onFailure(Exception e)
             }
             catch (InterruptedException e)
             {
-                // TODO what should we do with this? Maybe store in
-                // joinSignal if there's nothing else?
+                LOGGER.error("Wait for join operation was interrupted.", e);
                 throw new OperationFailedException(e.getMessage(),
                     OperationFailedException.INTERNAL_ERROR, e);
             }
@@ -728,10 +727,20 @@ private void leave(String chatRoomName)
         }
     }
 
+    /**
+     * Ban chat room member.
+     * 
+     * @param chatroom chat room to ban from
+     * @param member member to ban
+     * @param reason reason for banning
+     * @throws OperationFailedException
+     */
     public void banParticipant(ChatRoomIrcImpl chatroom, ChatRoomMember member,
-        String reason)
+        String reason) throws OperationFailedException
     {
-        // TODO Implement this.
+        // TODO Implement banParticipant.
+        throw new OperationFailedException("Not implemented yet.",
+            OperationFailedException.NOT_SUPPORTED_OPERATION);
     }
 
     /**

src/net/java/sip/communicator/impl/protocol/irc/OperationSetMultiUserChatIrcImpl.java

@@ -113,6 +113,33 @@ public List<String> getCurrentlyJoinedChatRooms(
         ChatRoomMember chatRoomMember)
     {
         //TODO: Implement "who is" for the IRC stack.
+        /*
+         * According to the RFC:
+         * 
+         * 311 RPL_WHOISUSER "<nick> <user> <host> * :<real name>"
+         * 
+         * 312 RPL_WHOISSERVER "<nick> <server> :<server info>"
+         * 
+         * 313 RPL_WHOISOPERATOR "<nick> :is an IRC operator"
+         * 
+         * 317 RPL_WHOISIDLE "<nick> <integer> :seconds idle"
+         * 
+         * 318 RPL_ENDOFWHOIS "<nick> :End of /WHOIS list"
+         * 
+         * 319 RPL_WHOISCHANNELS "<nick> :{[@|+]<channel><space>}"
+         * 
+         * - Replies 311 - 313, 317 - 319 are all replies generated in response
+         * to a WHOIS message. Given that there are enough parameters present,
+         * the answering server must either formulate a reply out of the above
+         * numerics (if the query nick is found) or return an error reply. The
+         * '*' in RPL_WHOISUSER is there as the literal character and not as a
+         * wild card. For each reply set, only RPL_WHOISCHANNELS may appear more
+         * than once (for long lists of channel names). The '@' and '+'
+         * characters next to the channel name indicate whether a client is a
+         * channel operator or has been granted permission to speak on a
+         * moderated channel. The RPL_ENDOFWHOIS reply is used to mark the end
+         * of processing a WHOIS message.
+         */
         return null;
     }