Changeset 5780 in orxonox.OLD for trunk/src/lib/shell/shell_completion.cc

Timestamp:

Nov 26, 2005, 3:47:51 PM (18 years ago)

Author:

bensch

Message:

orxonox/trunk: no more segfault in the Shell

File:

• trunk/src/lib/shell/shell_completion.cc (modified) (9 diffs)

trunk/src/lib/shell/shell_completion.cc

@@ -25 +25 @@
 #include "base_object.h"
 #include "class_list.h"
-#include "list.h"
 #include "debug.h"
 
@@ -37 +36 @@
 ShellCompletion::ShellCompletion(ShellInput* input)
 {
-  this->completionList = NULL;
   this->input = input;
 }
@@ -47 +45 @@
 ShellCompletion::~ShellCompletion ()
 {
-  // delete what has to be deleted here
-  if (this->completionList)
-  {
-    this->emptyCompletionList();
-    delete this->completionList;
-  }
 }
 
@@ -239 +231 @@
 bool ShellCompletion::generalComplete(const char* begin, const char* displayAs, const char* addBack, const char* addFront)
 {
-  if (completionList == NULL || this->input == NULL )
-    return false;
-  if (completionList->size() == 0)
-    return false;
-
-  ShellC_Element* addElem = completionList->front();
-  const char* addString = addElem->name;
+  if (this->input == NULL )
+    return false;
+  if (completionList.size() == 0)
+    return false;
+
+  ShellC_Element addElem = completionList.front();
+  const char* addString = addElem.name;
   unsigned int addLength = 0;
   unsigned int inputLenght = strlen(begin);
@@ -254 +246 @@
 
   SHELLC_TYPE changeType = SHELLC_NONE;
-  list<ShellC_Element*>::iterator charIT;
-  for (charIT = completionList->begin(); charIT != completionList->end(); charIT++)
-  {
-    if ((*charIT)->type != changeType)
+  list<ShellC_Element>::iterator charIT;
+  for (charIT = completionList.begin(); charIT != completionList.end(); charIT++)
+  {
+    if ((*charIT).type != changeType)
     {
       if (changeType != SHELLC_NONE)
         PRINT(0)("\n");
-      PRINT(0)("%s: ", ShellCompletion::typeToString((*charIT)->type));
-      changeType = (*charIT)->type;
+      PRINT(0)("%s: ", ShellCompletion::typeToString((*charIT).type));
+      changeType = (*charIT).type;
     }
-    PRINTF(0)("%s ", (*charIT)->name);
+    PRINTF(0)("%s ", (*charIT).name);
     for (unsigned int i = inputLenght; i < addLength; i++)
-      if (addString[i] != (*charIT)->name[i])
+      if (addString[i] != (*charIT).name[i])
       {
        addLength = i;
@@ -285 +277 @@
      this->input->addCharacters(adder);
 
-      if (completionList->size() == 1)
+      if (completionList.size() == 1)
       {
         if ( addBack != NULL )
@@ -315 +307 @@
         !strncasecmp(*string, completionBegin, searchLength))
     {
-      printf("%s\n", *string);
-      ShellC_Element* newElem = new ShellC_Element;
-      newElem->name = *string;
-      newElem->type = type;
-      this->completionList->push_back(newElem);
+      ShellC_Element newElem;
+      newElem.name = *string;
+      newElem.type = type;
+      this->completionList.push_back(newElem);
     }
   }
@@ -344 +335 @@
         !strncasecmp((*bo)->getName(), completionBegin, searchLength))
     {
-      ShellC_Element* newElem = new ShellC_Element;
-      newElem->name = (*bo)->getName();
-      newElem->type = type;
-      this->completionList->push_back(newElem);
+      ShellC_Element newElem;
+      newElem.name = (*bo)->getName();
+      newElem.type = type;
+      this->completionList.push_back(newElem);
     }
   }
@@ -361 +352 @@
 void ShellCompletion::emptyCompletionList()
 {
-  if (this->completionList != NULL)
-  {
-    while (this->completionList->size() > 0)
-    {
-      delete this->completionList->front();
-      this->completionList->pop_front();
-    }
-  }
+  this->completionList.erase(this->completionList.begin(),
+                             this->completionList.end());
 }