From a2816b156c92993f2bde6c82cdafbb556cf2c030 Mon Sep 17 00:00:00 2001
From: Danny van Heumen <danny@dannyvanheumen.nl>
Date: Sun, 2 Feb 2014 15:12:25 +0100
Subject: [PATCH] Added TODO and fine tuned trace logging.

---
 .../java/sip/communicator/impl/protocol/irc/IrcStack.java | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/net/java/sip/communicator/impl/protocol/irc/IrcStack.java b/src/net/java/sip/communicator/impl/protocol/irc/IrcStack.java
index 1eff37956..1475e8269 100644
--- a/src/net/java/sip/communicator/impl/protocol/irc/IrcStack.java
+++ b/src/net/java/sip/communicator/impl/protocol/irc/IrcStack.java
@@ -368,6 +368,8 @@ public List<String> getServerChatRoomList()
         // channels, since it isn't available.
         synchronized (this.channellist)
         {
+            // TODO time out channel list cache and refresh it regularly (every
+            // minute?)
             if (this.channellist.instance == null)
             {
                 Result<List<String>, Exception> listSignal =
@@ -393,8 +395,12 @@ public List<String> getServerChatRoomList()
                     }
                 }
                 this.channellist.instance = listSignal.getValue();
+                LOGGER.trace("Finished retrieve server chat room list.");
+            }
+            else
+            {
+                LOGGER.trace("Using cached list of server chat rooms.");
             }
-            LOGGER.trace("Finished retrieve server chat room list.");
             return Collections.unmodifiableList(this.channellist.instance);
         }
     }