Changeset 10916 for code/branches/cpp11_v2/src/libraries/core/command/CommandEvaluation.cc

Timestamp:

Dec 2, 2015, 11:22:03 PM (9 years ago)

Author:

landauf

Message:

use actual types instead of 'auto'. only exception is for complicated template types, e.g. when iterating over a map

File:

• code/branches/cpp11_v2/src/libraries/core/command/CommandEvaluation.cc (modified) (7 diffs)

code/branches/cpp11_v2/src/libraries/core/command/CommandEvaluation.cc

@@ -306 +306 @@
                 // the user typed 1-2 arguments, check what he tried to type and print a suitable error
                 std::string groupLC = getLowercase(this->getToken(0));
-                for (const auto & elem : ConsoleCommandManager::getInstance().getCommandsLC())
-                    if (elem.first == groupLC)
+                for (const auto& mapEntry : ConsoleCommandManager::getInstance().getCommandsLC())
+                    if (mapEntry.first == groupLC)
                         return std::string("Error: There is no command in group \"") + this->getToken(0) + "\" starting with \"" + this->getToken(1) + "\".";
 
@@ -328 +328 @@
 
         // iterate through all groups and their commands and calculate the distance to the current command. keep the best.
-        for (const auto & elem : ConsoleCommandManager::getInstance().getCommandsLC())
-        {
-            if (elem.first != "")
-            {
-                for (std::map<std::string, ConsoleCommand*>::const_iterator it_name = elem.second.begin(); it_name != elem.second.end(); ++it_name)
-                {
-                    std::string command = elem.first + " " + it_name->first;
+        for (const auto& mapEntry : ConsoleCommandManager::getInstance().getCommandsLC())
+        {
+            if (mapEntry.first != "")
+            {
+                for (std::map<std::string, ConsoleCommand*>::const_iterator it_name = mapEntry.second.begin(); it_name != mapEntry.second.end(); ++it_name)
+                {
+                    std::string command = mapEntry.first + " " + it_name->first;
                     unsigned int distance = getLevenshteinDistance(command, token0_LC + " " + token1_LC);
                     if (distance < nearestDistance)
@@ -349 +349 @@
         if (it_group !=  ConsoleCommandManager::getInstance().getCommandsLC().end())
         {
-            for (const auto & elem : it_group->second)
-            {
-                std::string command = elem.first;
+            for (const auto& mapEntry : it_group->second)
+            {
+                std::string command = mapEntry.first;
                 unsigned int distance = getLevenshteinDistance(command, token0_LC);
                 if (distance < nearestDistance)
@@ -429 +429 @@
     {
         size_t count = 0;
-        for (const auto & elem : list)
-            if (elem.getComparable() != "")
+        for (const ArgumentCompletionListElement& element : list)
+            if (element.getComparable() != "")
                 ++count;
         return count;
@@ -495 +495 @@
         {
             // only one (non-empty) value in the list - search it and return it
-            for (const auto & elem : list)
-            {
-                if (elem.getComparable() != "")
+            for (const ArgumentCompletionListElement& element : list)
+            {
+                if (element.getComparable() != "")
                 {
                     // arguments that have a separate string to be displayed need a little more care - just return them without modification. add a space character to the others.
-                    if (elem.hasDisplay())
-                        return (elem.getString());
+                    if (element.hasDisplay())
+                        return (element.getString());
                     else
-                        return (elem.getString() + ' ');
+                        return (element.getString() + ' ');
                 }
             }
@@ -517 +517 @@
                 char tempComparable = '\0';
                 char temp = '\0';
-                for (const auto & elem : list)
-                {
-                    const std::string& argumentComparable = elem.getComparable();
-                    const std::string& argument = elem.getString();
+                for (const ArgumentCompletionListElement& element : list)
+                {
+                    const std::string& argumentComparable = element.getComparable();
+                    const std::string& argument = element.getString();
 
                     // ignore empty arguments
@@ -560 +560 @@
     {
         std::string output;
-        for (const auto & elem : list)
-        {
-            output += elem.getDisplay();
+        for (const ArgumentCompletionListElement& element : list)
+        {
+            output += element.getDisplay();
 
             // add a space character between two elements for all non-empty arguments
-            if (elem.getComparable() != "")
+            if (element.getComparable() != "")
                 output += ' ';
         }