comments / trivial

kate/kateapp.cpp

@@ -197,11 +197,7 @@ bool KateApp::startupKate()
     KTextEditor::Document *doc = nullptr;
     const QString codec_name = codec ? QString::fromLatin1(codec->name()) : QString();
 
-    //  Bug 397913: Reverse the order here so the new tabs are opened in same order as the files were passed in on the command line
-    // NOTE: apparently the cause of the above bug is not present, fix reversed
-    QString positionalArgument;
-    for (int i = 0; i < m_args.positionalArguments().count(); ++i) {
-        positionalArgument = m_args.positionalArguments().at(i);
+    for (const auto& positionalArgument : m_args.positionalArguments()) {
         UrlInfo info(positionalArgument);
 
         // this file is no local dir, open it, else warn

kate/main.cpp

@@ -459,8 +459,6 @@ int main(int argc, char **argv)
             QStringList tokens;
 
             // open given files...
-            // Bug 397913: Reverse the order here so the new tabs are opened in same order as the files were passed in on the command line
-            // Cause of above bug not present, reversed...
             for (int i = 0; i < urls.size(); ++i) {
                 const QString &url = urls[i];
                 QDBusMessage m = QDBusMessage::createMethodCall(serviceName,