From 45626391be7a4f5b8b6d9e7bb8624fcbb9f62b2f Mon Sep 17 00:00:00 2001
From: Danny van Heumen <danny@dannyvanheumen.nl>
Date: Wed, 6 May 2015 22:34:34 +0200
Subject: [PATCH] Updated comments (TODO, FIXME)

---
 .../sip/communicator/impl/protocol/irc/IrcConnection.java  | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/net/java/sip/communicator/impl/protocol/irc/IrcConnection.java b/src/net/java/sip/communicator/impl/protocol/irc/IrcConnection.java
index 94bd1d77a..c0bdb800f 100644
--- a/src/net/java/sip/communicator/impl/protocol/irc/IrcConnection.java
+++ b/src/net/java/sip/communicator/impl/protocol/irc/IrcConnection.java
@@ -218,6 +218,10 @@ private static CapabilityNegotiator determineNegotiator(final String user,
         {
             return new NoopNegotiator();
         }
+        // TODO In time, replace SaslNegotiator with CompositeNegotiator and
+        // SaslCapability together with any supported other capabilities.
+        // 'away-notify' would be an interesting option, so we do not have to
+        // periodically query channel status.
         return new SaslNegotiator(sasl.getUser(), sasl.getPass(),
             sasl.getRole());
     }
@@ -237,9 +241,6 @@ private static IIRCState connectSynchronized(
     {
         final Result<IIRCState, Exception> result =
             new Result<IIRCState, Exception>();
-        // FIXME Decide between SASL authentication and normal 'PASS'-parameter
-        // authentication. You cannot do both as some services will respond with
-        // already-authenticated warning.
         synchronized (result)
         {
             // start connecting to the specified server ...