Misc tweaks and comments.

3 files changed, 109 insertions, 1 deletions

diff --git a/src/net/java/sip/communicator/impl/protocol/irc/ChatRoomIrcImpl.java b/src/net/java/sip/communicator/impl/protocol/irc/ChatRoomIrcImpl.java
index d473fb2..69a7e48 100644
--- a/src/net/java/sip/communicator/impl/protocol/irc/ChatRoomIrcImpl.java
+++ b/src/net/java/sip/communicator/impl/protocol/irc/ChatRoomIrcImpl.java
@@ -646,6 +646,7 @@ public class ChatRoomIrcImpl
      * @throws OperationFailedException if the setting the new nickname changes
      *             for some reason.
      */
+    @Override
     public void setUserNickname(final String nickName)
         throws OperationFailedException
     {
@@ -1382,7 +1383,11 @@ public class ChatRoomIrcImpl
     @Override
     public Contact getPrivateContactByNickname(final String name)
     {
-        LOGGER.debug("Getting private contact for nick name '" + name + "'.");
+        if (LOGGER.isDebugEnabled())
+        {
+            LOGGER.debug("Getting private contact for nick name '" + name
+                + "'.");
+        }
         return this.parentProvider.getPersistentPresence()
             .findOrCreateContactByID(name);
     }
diff --git a/src/net/java/sip/communicator/impl/protocol/irc/Color.java b/src/net/java/sip/communicator/impl/protocol/irc/Color.java
index f42dd8e..afffaef 100644
--- a/src/net/java/sip/communicator/impl/protocol/irc/Color.java
+++ b/src/net/java/sip/communicator/impl/protocol/irc/Color.java
@@ -13,21 +13,84 @@ package net.java.sip.communicator.impl.protocol.irc;
  */
 public enum Color
 {
+    /**
+     * White.
+     */
     WHITE("White"),
+
+    /**
+     * Black.
+     */
     BLACK("Black"),
+
+    /**
+     * Navy.
+     */
     BLUE("Navy"),
+
+    /**
+     * Green.
+     */
     GREEN("Green"),
+
+    /**
+     * Red.
+     */
     RED("Red"),
+
+    /**
+     * Maroon.
+     */
     BROWN("Maroon"),
+
+    /**
+     * Purple.
+     */
     PURPLE("Purple"),
+
+    /**
+     * Orange.
+     */
     ORANGE("Orange"),
+
+    /**
+     * Yellow.
+     */
     YELLOW("Yellow"),
+
+    /**
+     * Lime.
+     */
     LIGHT_GREEN("Lime"),
+
+    /**
+     * Teal.
+     */
     TEAL("Teal"),
+
+    /**
+     * Cyan.
+     */
     LIGHT_CYAN("Cyan"),
+
+    /**
+     * RoyalBlue.
+     */
     LIGHT_BLUE("RoyalBlue"),
+
+    /**
+     * Fuchsia.
+     */
     PINK("Fuchsia"),
+
+    /**
+     * Grey.
+     */
     GREY("Grey"),
+
+    /**
+     * Silver.
+     */
     LIGHT_GREY("Silver");
 
     /**
diff --git a/src/net/java/sip/communicator/impl/protocol/irc/Mode.java b/src/net/java/sip/communicator/impl/protocol/irc/Mode.java
index f7ae746..67a37ec 100644
--- a/src/net/java/sip/communicator/impl/protocol/irc/Mode.java
+++ b/src/net/java/sip/communicator/impl/protocol/irc/Mode.java
@@ -15,15 +15,55 @@ import net.java.sip.communicator.service.protocol.*;
  */
 public enum Mode
 {
+    /**
+     * Instance for unknown mode type.
+     */
     UNKNOWN('?', null),
+
+    /**
+     * Mode 'Owner'. (Not recognized by all IRC servers.)
+     */
     OWNER('O', ChatRoomMemberRole.OWNER),
+
+    /**
+     * Mode 'Operator'.
+     */
     OPERATOR('o', ChatRoomMemberRole.ADMINISTRATOR),
+
+    /**
+     * Mode 'Half-Operator'. (Not recognized by all IRC servers.)
+     */
     HALFOP('h', ChatRoomMemberRole.MODERATOR),
+
+    /**
+     * Mode 'Voice'. For giving voice to an IRC member which comes into effect
+     * in a moderated channel.
+     */
     VOICE('v', ChatRoomMemberRole.MEMBER),
+
+    /**
+     * Mode 'Limit'.
+     */
     LIMIT('l', null),
+
+    /**
+     * Mode 'Private'.
+     */
     PRIVATE('p', null),
+
+    /**
+     * Mode 'Secret'.
+     */
     SECRET('s', null),
+
+    /**
+     * Mode 'Invite'.
+     */
     INVITE('i', null),
+
+    /**
+     * Mode 'Ban'.
+     */
     BAN('b', null);
 
     /**