A bit of cleanup and TODOs.

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

@@ -1117,9 +1117,9 @@ protected void setSubjectFromServer(String subject)
 
     /**
      * Determines whether this chat room should be stored in the configuration
-     * file or not. If the chat room is persistent it still will be shown after a
-     * restart in the chat room list. A non-persistent chat room will be only in
-     * the chat room list until the the program is running.
+     * file or not. If the chat room is persistent it still will be shown after
+     * a restart in the chat room list. A non-persistent chat room will be only
+     * in the chat room list until the the program is running.
      *
      * @return true if this chat room is persistent, false otherwise
      */
@@ -1217,9 +1217,11 @@ public void revokeAdmin(String address)
     }
 
     /**
-     * Revokes the membership role for the participant given by <tt>address</tt>.
+     * Revokes the membership role for the participant given by <tt>address</tt>
+     * .
+     * 
      * @param address the address of the participant to revoke membership role
-     * for
+     *            for
      */
     public void revokeMembership(String address)
     {

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

@@ -126,8 +126,8 @@ public String done()
 
     /**
      * Return the formatted string in its current state. (This means that if
-     * {@link #done()} was not yet called, it will print an intermediate state of
-     * the formatted text.)
+     * {@link #done()} was not yet called, it will print an intermediate state
+     * of the formatted text.)
      */
     public String toString()
     {

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

@@ -79,7 +79,8 @@ private void parse(String modestring)
                 }
                 catch (ArrayIndexOutOfBoundsException e)
                 {
-                    throw new IllegalArgumentException("invalid mode string provided: parameter missing", e);
+                    throw new IllegalArgumentException("invalid mode string "
+                        + "provided: parameter missing", e);
                 }
                 break;
             }

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

@@ -106,7 +106,8 @@ public List<ChatRoom> getCurrentlyJoinedChatRooms()
      * @return a list of the chat rooms that <tt>chatRoomMember</tt> has joined
      * and is currently active in.
      */
-    public List<String> getCurrentlyJoinedChatRooms(ChatRoomMember chatRoomMember)
+    public List<String> getCurrentlyJoinedChatRooms(
+        ChatRoomMember chatRoomMember)
     {
         //TODO: Implement "who is" for the IRC stack.
         return null;
@@ -198,7 +199,6 @@ public void rejectInvitation(ChatRoomInvitation invitation, String reason)
      */
     public boolean isMultiChatSupportedByContact(Contact contact)
     {
-        //TODO: Implement isMultiChatSupportedByContact.
         return true;
     }
 

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

@@ -89,13 +89,14 @@ protected void initialize(String userID, AccountID accountID)
             //Initialize the multi user chat support
             multiUserChat = new OperationSetMultiUserChatIrcImpl(this);
             
-            // TODO Implement OperationSetPersistentPresence in order to support
-            // private messaging.
-
             addSupportedOperationSet(
                 OperationSetMultiUserChat.class,
                 multiUserChat);
             
+            // TODO Implement OperationSetPersistentPresence and
+            // OperationSetBasicInstantMessaging in order to support private
+            // messaging.
+
             // TODO Implement OperationSetServerStoredAccountInfo so we can
             // suggest a display name to use when adding new chat rooms?
 

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

@@ -10,6 +10,8 @@
 
 /**
  * Represents the informations we get from a user when we use WHOIS
+ * 
+ * TODO Class is unused at the moment.
  *
  * @author Stephane Remy
  */

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

@@ -141,7 +141,7 @@ private static Color parseBackgroundColor(String text)
                 return Color.values()[color];
             }
             throw new IllegalArgumentException(
-                "no color separator present, hence no background color present");
+               "no color separator present, hence no background color present");
         }
         catch (StringIndexOutOfBoundsException e)
         {
@@ -180,8 +180,8 @@ private static Color parseForegroundColor(String text)
         catch (StringIndexOutOfBoundsException e)
         {
             // Invalid control code, since text has ended.
-            LOGGER
-                .trace("ArrayIndexOutOfBounds during foreground color control code parsing.");
+            LOGGER.trace("ArrayIndexOutOfBounds during foreground "
+                + "color control code parsing.");
             throw new IllegalArgumentException("missing foreground color code");
         }
         catch (NumberFormatException e)