Changeset 6940 for code/branches/presentation3/src/modules/questsystem/QuestItem.cc

Timestamp:

May 20, 2010, 7:38:53 PM (14 years ago)

Author:

dafrick

Message:

Resolved issue with the Questsystem causing a segfault when a level with Quests was loaded twice. The issue still persists, but not with the Questsystem anymore and it will be addressed with my next commit.
Also removed restrictions to the number of characters a quest/hint identifier must have.

File:

• code/branches/presentation3/src/modules/questsystem/QuestItem.cc (modified) (3 diffs)

code/branches/presentation3/src/modules/questsystem/QuestItem.cc

@@ -46 +46 @@
     QuestItem::QuestItem(BaseObject* creator) : BaseObject(creator)
     {
+        this->registered_ = false;
+        
         RegisterObject(QuestItem);
     }
@@ -79 +81 @@
     void QuestItem::setId(const std::string & id)
     {
-        if(!isId(id)) //!< Checks whether the id is a valid id.
+        if(id.compare(BLANKSTRING) == 0) //!< Checks whether the id is a valid id.
         {
             COUT(2) << "Invalid id. QuestItem id {" << id << "} could not be set." << std::endl;
@@ -88 +90 @@
     }
 
-    /**
-    @brief
-        Checks whether an input id is of the required form.
-    @param id
-        The id to be checked.
-    @return
-        Returns true if the string is likely to be of the required form.
-    */
-    /*static*/ bool QuestItem::isId(const std::string & id)
-    {
-        return id.size() >= 32;
-    }
-
 }