Changeset 4257 in orxonox.OLD

Timestamp:

May 22, 2005, 2:39:59 AM (19 years ago)

Author:

bensch

Message:

orxonox/trunk: performance-update when creating a new entity

Location:

orxonox/branches/levelLoader/src

Files:

• lib/lang/base_object.cc (modified) (1 diff)
• lib/lang/base_object.h (modified) (2 diffs)
• world_entities/skybox.cc (modified) (1 diff)
• world_entities/world_entity.cc (modified) (1 diff)

orxonox/branches/levelLoader/src/lib/lang/base_object.cc

@@ -39 +39 @@
 BaseObject::~BaseObject () 
 {
-  delete []this->className;
+  //  delete []this->className;
 }
 
-void BaseObject::setClassName (char* className)
+void BaseObject::setClassName (const char* className)
 {
-  this->className = new char[strlen(className)+1];
-  strcpy(this->className, className);
-}
-
-char* BaseObject::getClassName(void) const
-{
-  return className;
+  this->className = className;
 }
 

orxonox/branches/levelLoader/src/lib/lang/base_object.h

@@ -17 +17 @@
   virtual ~BaseObject ();
 
-  void setClassName (char* className);
-  char* getClassName(void) const;
+  void setClassName (const char* className);
+  inline const char* getClassName(void) const { return this->className;};
   bool isA (char* className);
 
@@ -25 +25 @@
 
  private:
-  char* className;
+  const char* className;
   bool finalized;
 };

orxonox/branches/levelLoader/src/world_entities/skybox.cc

@@ -56 +56 @@
 void SkyBox::loadParams(const TiXmlElement* root)
 {
-  LoadParam<SkyBox>(root, "Materialset", this, &SkyBox::setTexture).describe("Sets the material on the skysphere. The string must be the path relative to the data-dir, and without a trailing .jpg");
+  LoadParam<SkyBox>(root, "Materialset", this, &SkyBox::setTexture).describe("Sets the material on the SkyBox. The string must be the path relative to the data-dir, and without a trailing .jpg");
 }
 

orxonox/branches/levelLoader/src/world_entities/world_entity.cc

@@ -42 +42 @@
 {
   // name setup
-  LoadParam<WorldEntity>(root, "name", this, &WorldEntity::setName);
+  LoadParam<WorldEntity>(root, "name", this, &WorldEntity::setName).describe("the name of the Object at hand");
 
   // Model Loading      
-  LoadParam<WorldEntity>(root, "model", this, &WorldEntity::loadModel);
+  LoadParam<WorldEntity>(root, "model", this, &WorldEntity::loadModel).describe("the fileName of the model, that should be loaded onto this world-entity. (must be relative to the data-dir)") ;
 }