Changeset 7978 for code/branches/usability/src/modules

Timestamp:

Feb 27, 2011, 3:11:42 AM (13 years ago)

Author:

rgrieder

Message:

Increased code readability by replacing some BLANKSTRING with plain old "".

Location:

code/branches/usability/src/modules

Files:

• objects/triggers/DistanceMultiTrigger.cc (modified) (2 diffs)
• objects/triggers/DistanceTrigger.cc (modified) (1 diff)
• objects/triggers/DistanceTrigger.h (modified) (1 diff)
• pickup/PickupRepresentation.cc (modified) (1 diff)
• questsystem/effects/AddQuestHint.cc (modified) (1 diff)
• questsystem/effects/ChangeQuestStatus.cc (modified) (1 diff)

code/branches/usability/src/modules/objects/triggers/DistanceMultiTrigger.cc

@@ -54 +54 @@
 
         this->distance_ = 100.0f;
-        this->targetName_ = BLANKSTRING;
+        this->targetName_ = "";
         this->singleTargetMode_ = false;
     }
@@ -181 +181 @@
     {
         // If the targetname is no blank string single-target mode is enabled.
-        if(targetname.compare(BLANKSTRING) != 0)
+        if(targetname != "")
             this->singleTargetMode_ = true;
         else

code/branches/usability/src/modules/objects/triggers/DistanceTrigger.cc

@@ -50 +50 @@
     this->distance_ = 100;
     this->targetMask_.exclude(Class(BaseObject));
-    this->targetName_ = BLANKSTRING;
+    this->targetName_ = "";
     this->singleTargetMode_ = false;
   }

code/branches/usability/src/modules/objects/triggers/DistanceTrigger.h

@@ -70 +70 @@
 
       inline void setTargetName(const std::string& targetname)
-        { if(targetname.compare(BLANKSTRING) != 0) this->singleTargetMode_ = true; else this->singleTargetMode_ = false; this->targetName_ = targetname; }
+        { if(targetname != "") this->singleTargetMode_ = true; else this->singleTargetMode_ = false; this->targetName_ = targetname; }
       inline const std::string& getTargetName(void)
         { return this->targetName_; }

code/branches/usability/src/modules/pickup/PickupRepresentation.cc

@@ -156 +156 @@
         {
             COUT(4) << "PickupRepresentation: No spawner representation found." << std::endl;
-            if(this->spawnerTemplate_ == BLANKSTRING)
+            if(this->spawnerTemplate_ == "")
             {
                 COUT(4) << "PickupRepresentation: Spawner template is empty." << std::endl;

code/branches/usability/src/modules/questsystem/effects/AddQuestHint.cc

@@ -86 +86 @@
     bool AddQuestHint::setHintId(const std::string & id)
     {
-        if(id.compare(BLANKSTRING) == 0)
+        if(id == "")
         {
             COUT(2) << "Invalid id. QuestItem id {" << id << "} could not be set." << std::endl;

code/branches/usability/src/modules/questsystem/effects/ChangeQuestStatus.cc

@@ -81 +81 @@
     bool ChangeQuestStatus::setQuestId(const std::string & id)
     {
-        if(id.compare(BLANKSTRING) == 0)
+        if(id == "")
         {
             COUT(2) << "Invalid id. QuestItem id {" << id << "} could not be set." << std::endl;