identd: Turn !canReadLine into an early return
authorJan Alexander Steffens (heftig) <jan.steffens@gmail.com>
Sat, 19 May 2018 10:29:42 +0000 (12:29 +0200)
committerManuel Nickschas <sputnick@quassel-irc.org>
Mon, 18 Jun 2018 19:25:50 +0000 (21:25 +0200)
Reduces rightward drift.

src/core/identserver.cpp

index ffb2f15..13b32fa 100644 (file)
@@ -96,31 +96,33 @@ void IdentServer::respond() {
 
     qint64 transactionId = _socketId;
 
 
     qint64 transactionId = _socketId;
 
-    if (socket->canReadLine()) {
-        QByteArray query = socket->readLine();
-        if (query.endsWith("\r\n"))
-            query.chop(2);
-        else if (query.endsWith("\n"))
-            query.chop(1);
+    if (!socket->canReadLine()) {
+        return;
+    }
 
 
-        QList<QByteArray> split = query.split(',');
+    QByteArray query = socket->readLine();
+    if (query.endsWith("\r\n"))
+        query.chop(2);
+    else if (query.endsWith("\n"))
+        query.chop(1);
 
 
-        bool success = false;
+    QList<QByteArray> split = query.split(',');
 
 
-        quint16 localPort;
-        if (!split.empty()) {
-            localPort = split[0].trimmed().toUShort(&success, 10);
-        }
+    bool success = false;
 
 
-        Request request{socket, localPort, query, transactionId, _requestId++};
-        if (!success) {
+    quint16 localPort;
+    if (!split.empty()) {
+        localPort = split[0].trimmed().toUShort(&success, 10);
+    }
+
+    Request request{socket, localPort, query, transactionId, _requestId++};
+    if (!success) {
+        responseUnavailable(request);
+    } else if (!responseAvailable(request)) {
+        if (hasSocketsBelowId(transactionId)) {
+            _requestQueue.emplace_back(request);
+        } else {
             responseUnavailable(request);
             responseUnavailable(request);
-        } else if (!responseAvailable(request)) {
-            if (hasSocketsBelowId(transactionId)) {
-                _requestQueue.emplace_back(request);
-            } else {
-                responseUnavailable(request);
-            }
         }
     }
 }
         }
     }
 }