Changeset 8316 in orxonox.OLD for trunk/src

Timestamp:

Jun 11, 2006, 1:57:27 PM (19 years ago)

Author:

bensch

Message:

trunk: fixed most -Wall warnings… but there are still many missing :/

Location:

trunk/src

Files:

• lib/collision_detection/cd_engine.cc (modified) (1 diff)
• lib/collision_detection/obb_tree.cc (modified) (1 diff)
• lib/collision_detection/obb_tree_node.cc (modified) (4 diffs)
• lib/coord/p_node.cc (modified) (5 diffs)
• lib/event/event_handler.cc (modified) (1 diff)
• lib/graphics/effects/fog_effect.cc (modified) (6 diffs)
• lib/graphics/effects/graphics_effect.cc (modified) (1 diff)
• lib/graphics/effects/lense_flare.cc (modified) (4 diffs)
• lib/graphics/effects/lightening_effect.cc (modified) (9 diffs)
• lib/graphics/effects/rain_effect.cc (modified) (6 diffs)
• lib/graphics/effects/snow_effect.cc (modified) (7 diffs)
• lib/graphics/effects/volfog_effect.cc (modified) (6 diffs)
• lib/graphics/effects/weather_effect.cc (modified) (1 diff)
• lib/graphics/graphics_engine.cc (modified) (10 diffs)
• lib/graphics/graphics_engine.h (modified) (2 diffs)
• lib/graphics/importer/height_map.cc (modified) (2 diffs)
• lib/graphics/importer/material.cc (modified) (4 diffs)
• lib/graphics/importer/material.h (modified) (1 diff)
• lib/graphics/importer/md2Model.cc (modified) (3 diffs)
• lib/graphics/importer/media_container.cc (modified) (1 diff)
• lib/graphics/importer/model.cc (modified) (1 diff)
• lib/graphics/importer/primitive_model.cc (modified) (6 diffs)
• lib/graphics/importer/static_model.cc (modified) (15 diffs)
• lib/graphics/importer/tc.cc (modified) (6 diffs)
• lib/graphics/importer/texture.cc (modified) (6 diffs)
• lib/graphics/render2D/element_2d.cc (modified) (3 diffs)
• lib/graphics/shader.cc (modified) (1 diff)
• lib/graphics/text_engine/font.cc (modified) (11 diffs)
• lib/gui/gl/glmenu/glmenu_imagescreen.cc (modified) (1 diff)
• lib/gui/gl/signal_connector.cc (modified) (1 diff)
• lib/gui/qt/gui_control.cc (modified) (1 diff)
• lib/lang/class_list.cc (modified) (1 diff)
• lib/parser/cmdline_parser/cmdline_parser.cc (modified) (8 diffs)
• lib/parser/cmdline_parser/cmdline_parser.h (modified) (2 diffs)
• lib/parser/preferences/cmd_line_prefs_reader.cc (modified) (10 diffs)
• lib/sound/ogg_player.cc (modified) (2 diffs)
• lib/util/loading/dynamic_loader.cc (modified) (3 diffs)
• lib/util/loading/load_param.cc (modified) (1 diff)
• lib/util/loading/load_param_description.cc (modified) (1 diff)
• lib/util/loading/resource_manager.cc (modified) (5 diffs)
• lib/util/multi_type.cc (modified) (3 diffs)
• orxonox.cc (modified) (4 diffs)
• world_entities/playable.cc (modified) (2 diffs)
• world_entities/player.cc (modified) (2 diffs)
• world_entities/projectiles/bomb.cc (modified) (1 diff)
• world_entities/projectiles/guided_missile.cc (modified) (1 diff)
• world_entities/projectiles/hyperblast.cc (modified) (1 diff)
• world_entities/projectiles/rocket.cc (modified) (1 diff)
• world_entities/projectiles/test_bullet.cc (modified) (1 diff)
• world_entities/space_ships/hover.cc (modified) (3 diffs)
• world_entities/space_ships/turbine_hover.cc (modified) (2 diffs)
• world_entities/terrain.cc (modified) (1 diff)
• world_entities/test_entity.cc (modified) (1 diff)
• world_entities/weapons/weapon.cc (modified) (7 diffs)
• world_entities/world_entity.cc (modified) (2 diffs)

trunk/src/lib/collision_detection/cd_engine.cc

@@ -99 +99 @@
   if( likely( this->terrain != NULL))
   {
-    Quadtree* q = dynamic_cast<Terrain*>(this->terrain)->ssp->getQuadtree();
+//    Quadtree* q = dynamic_cast<Terrain*>(this->terrain)->ssp->getQuadtree();
 //    QuadtreeNode* n = q->getQuadtreeFromPosition(this->player->getAbsCoor());
   }
-  
+
   if( likely( this->bspManager != NULL))
   {

trunk/src/lib/collision_detection/obb_tree.cc

@@ -74 +74 @@
   /* triangles indexes created */
   int* triangleIndexes = new int[modelInf.numTriangles];
-  for( int i = 0; i < modelInf.numTriangles; ++i)
+  for(unsigned int i = 0; i < modelInf.numTriangles; ++i)
     triangleIndexes[i] = i;
 

trunk/src/lib/collision_detection/obb_tree_node.cc

@@ -152 +152 @@
   Vector    p, q, r;                                 //!< holder of the polygon data, much more conveniant to work with Vector than sVec3d
   Vector    t1, t2;                                  //!< temporary values
-  float     covariance[3][3] = {0,0,0, 0,0,0, 0,0,0};//!< the covariance matrix
-  const float*   tmpVec = NULL;                           //!< a temp saving place for sVec3Ds
+  float     covariance[3][3] = {{0,0,0}, {0,0,0}, {0,0,0}};//!< the covariance matrix
 
   /* fist compute all the convex hull face/facelets and centroids */
@@ -275 +274 @@
   PRINTF(4)("Calculate Box Axis\n");
   /* now get the axis length */
-  float               halfLength[3];                         //!< half length of the axis
   float               tmpLength;                             //!< tmp save point for the length
   Plane               p0(box.axis[0], box.center);           //!< the axis planes
@@ -371 +369 @@
   PRINTF(4)("Separating along the longest axis\n");
   /* get the closest vertex near the center */
-  float               dist = 999999.0f;                    //!< the smallest distance to each vertex
   float               tmpDist;                             //!< variable to save diverse distances temporarily
-  int                 vertexIndex;                         //!< index of the vertex near the center
   Plane               middlePlane(box.axis[longestAxisIndex], box.center); //!< the middle plane
-  const sVec3D*       tmpVec;                              //!< temp simple 3D vector
 
 
@@ -662 +657 @@
         glBegin(GL_POINTS);
         glColor3f(0.3, 0.8, 0.54);
-        for( int i = 0; i < this->bvElement->modelInf->numVertices*3; i+=3)
+        for(unsigned int i = 0; i < this->bvElement->modelInf->numVertices*3; i+=3)
           glVertex3f(this->bvElement->modelInf->pVertices[i],
                      this->bvElement->modelInf->pVertices[i+1],

trunk/src/lib/coord/p_node.cc

@@ -36 +36 @@
  */
 PNode::PNode (PNode* parent, long nodeFlags)
-    : Synchronizeable(), BaseObject()
+  : BaseObject(), Synchronizeable()
 {
   this->setClassID(CL_PARENT_NODE, "PNode");
@@ -798 +798 @@
         {
           this->relDirection = rotQuat;
-          this->bRelDirChanged;
+          this->bRelDirChanged = true;
         }
         else
@@ -805 +805 @@
           this->toDirection = NULL;
           PRINTF(5)("SmoothRotate of %s finished\n", this->getName());
-          this->bRelDirChanged;
+          this->bRelDirChanged = true;
         }
       }
@@ -1058 +1058 @@
   else if (parentingMode == PNODE_ROTATE_AND_MOVE)
     return "rotate-and-move";
+  else
+    return "all";
 }
 
@@ -1077 +1079 @@
   else  if (parentingMode == "rotate-and-move")
     return (PNODE_ROTATE_AND_MOVE);
+  else
+    return PNODE_ALL;
 }
 

trunk/src/lib/event/event_handler.cc

@@ -369 +369 @@
   SDL_Event event;
   Event ev;
-  EventListener* listener = NULL;
   while( SDL_PollEvent (&event))
   {

trunk/src/lib/graphics/effects/fog_effect.cc

@@ -72 +72 @@
         this->fogFadeDuration = 0;
         this->localTimer = 0;
-        
+
         this->fogMode = GL_EXP2;
         this->fogDensity = 0.001;
@@ -79 +79 @@
         this->colorVector = Vector(0.7, 0.7, 0.7);
 
+        return true;
 }
 
@@ -87 +88 @@
 
         this->fogActivate = true;
-        
+
         GLfloat fogColor[4] = { colorVector.x, colorVector.y, colorVector.z, 1.0};
         glFogi(GL_FOG_MODE, this->fogMode);
@@ -95 +96 @@
         glFogf(GL_FOG_START, this->fogStart);
         glFogf(GL_FOG_END, this->fogEnd);
-        
+
         glEnable(GL_FOG);
         // glClearColor(0.5, 0.5, 0.5, 1.0);
+        return true;
 }
 
@@ -106 +108 @@
 
         this->fogActivate = false;
-        
+
         glDisable(GL_FOG);
+
+        return true;
 }
 
@@ -122 +126 @@
         if (!this->fogActivate)
                 return;
-                
+
 
         if (this->fogFadeDuration != 0 && this->localTimer < this->fogFadeDuration) {

trunk/src/lib/graphics/effects/graphics_effect.cc

@@ -61 +61 @@
  */
 bool GraphicsEffect::init()
-{}
+{
+  return true;
+}
 
 

trunk/src/lib/graphics/effects/lense_flare.cc

@@ -104 +104 @@
  */
 bool LenseFlare::init()
-{}
+{
+  return true;
+}
 
 
@@ -113 +115 @@
 {
   this->bActivated = true;
+
+  return true;
 }
 
@@ -122 +126 @@
 {
   this->bActivated = false;
+
+  return true;
 }
 
@@ -130 +136 @@
  */
 GLint LenseFlare::stringToFogMode(const std::string& mode)
-{}
+{
+  PRINTF(1)("NOT IMPLEMENTED");
+  return 0;
+}
 
 

trunk/src/lib/graphics/effects/lightening_effect.cc

@@ -36 +36 @@
 {
         this->setClassID(CL_LIGHTENING_EFFECT, "LighteningEffect");
-  
+
         this->init();
 
@@ -54 +54 @@
 {
         WeatherEffect::loadParams(root);
-  
+
   LoadParam(root, "coord", this, LighteningEffect, coord);
   LoadParam(root, "option", this, LighteningEffect, setLighteningOption);
@@ -68 +68 @@
 {
   lighteningActivate = false;
-  
+
   this->time = 0.0;
   this->flashFrequency = 1.4f;
@@ -117 +117 @@
     ResourceManager::getInstance()->unload(this->thunderBuffer);
   this->thunderBuffer = (OrxSound::SoundBuffer*)ResourceManager::getInstance()->load("sound/thunder.wav", WAV);*/
+
+  return true;
 }
 
@@ -149 +151 @@
   this->time = 0;
   lighteningActivate = true;
+
+  return true;
 }
 
@@ -160 +164 @@
   this->billboard[3]->setVisibiliy(false);
   lighteningActivate = false;
+
+
+  return true;
 }
 
@@ -166 +173 @@
   if(!lighteningActivate)
     return;
-  
+
   this->time += dt;
 
@@ -175 +182 @@
     else
       this->flashLight->setDiffuseColor(100,100,100);
-    
+
     //PRINTF(0)("100*time: %f %i\n", 100*time, (int)(100*time));
 }*/
-  
+
   if( this->time > this->flashFrequency)
   {
     this->billboard[0]->setVisibiliy(true);
     this->time = 0.0f;
-    
+
     this->flashLight = new Light();
     this->flashLight->setAbsCoor(this->billboard[0]->getAbsCoor().x, this->billboard[0]->getAbsCoor().y, this->billboard[0]->getAbsCoor().z);
     this->flashLight->setDiffuseColor(100,100,100);
-    
+
     //this->soundSource.play(this->thunderBuffer);
   }
@@ -195 +202 @@
     this->time = 0.0f;
     this->bNewCoordinate = true;
-    
+
     if(flashLight != NULL)
     {

trunk/src/lib/graphics/effects/rain_effect.cc

@@ -117 +117 @@
         lightMan = LightManager::getInstance();
         this->rainAmbient = lightMan->getAmbientColor();
+
+        return true;
 }
 
@@ -150 +152 @@
 
         this->emitter->setSpread(this->rainWindForce / 50, 0.2);
-        
+
         this->soundSource.loop(this->rainBuffer, this->soundRainVolume);
         if (this->rainWindForce != 0) this->soundSource.loop(this->windBuffer, 0.1f * this->rainWindForce);
 
         lightMan->setAmbientColor(.1,.1,.1);
+
+        return true;
 }
 
@@ -161 +165 @@
 {
         PRINTF(0)("Deactivating RainEffect\n");
-        
+
         this->rainActivate = false;
         this->emitter->setSystem(NULL);
@@ -170 +174 @@
         // Restore Light Ambient
         lightMan->setAmbientColor(this->rainAmbient, this->rainAmbient, this->rainAmbient);
+
+        return true;
 }
 
@@ -176 +182 @@
         if (!this->rainActivate)
                 return;
-                
+
         if (this->rainMove) {
                 this->rainCoord = State::getCameraNode()->getAbsCoor();
@@ -202 +208 @@
                 // this->soundSource.fadein(this->rainBuffer, 10);
         }
-        
+
 }
 

trunk/src/lib/graphics/effects/snow_effect.cc

@@ -79 +79 @@
         LoadParam(root, "size", this, SnowEffect, size);
         LoadParam(root, "coord", this, SnowEffect, coord);
-  
+
         LOAD_PARAM_START_CYCLE(root, element);
         {
@@ -111 +111 @@
         this->snowCoord = Vector(100,450,400);
         this->snowWindForce = 1;
+
+        return true;
 }
 
@@ -143 +145 @@
         if (this->snowWindForce > 1)
                 this->soundSource.loop(this->windBuffer);
+
+        return true;
 }
 
@@ -156 +160 @@
                 ResourceManager::getInstance()->unload(this->windBuffer);
 
+        return true;
 }
 
@@ -171 +176 @@
         /*
         float distance = (State::getCameraNode()->getAbsCoor() - Vector(snowCoord.x, State::getCameraNode()->getAbsCoor().y, snowCoord.z)).len();
-        
+
         if(activated)
         {
@@ -182 +187 @@
         else if(distance > 0.1*snowSize.x || distance > 0.1*snowSize.y)
         this->alpha = 0.4;
-        
+
         SnowEffect::snowParticles->setColor(0,1, 1, 1, alpha);
         SnowEffect::snowParticles->setColor(.5, .6, .6, .6, alpha/2);
@@ -197 +202 @@
         else if( distance < 0.1*snowSize.x || distance < 0.1*snowSize.y )
         this->alpha = 0.5;
-        
+
         SnowEffect::snowParticles->setColor(0,1, 1, 1, alpha);
         SnowEffect::snowParticles->setColor(.5, .6, .6, .6, alpha/2);

trunk/src/lib/graphics/effects/volfog_effect.cc

@@ -29 +29 @@
 
 //#include <GL/glext.h> //OpenGL Extensions
-//#include <GL/glxext.h> // GLX Extensions 
+//#include <GL/glxext.h> // GLX Extensions
 
 #ifndef GL_EXT_fog_coord
@@ -92 +92 @@
         // Set fog color
         float fogColor[4] = {0.6f,0.58f,0.79f,0.0f};
-        
+
 
         glFogCoordfEXT       = (PFNGLFOGCOORDFEXTPROC)glXGetProcAddressARB((const GLubyte*)"glFogCoordfEXT");
@@ -132 +132 @@
 {
         PRINTF(0)("Activating VolFogEffect\n");
+
+        return true;
 }
 
@@ -139 +141 @@
 
         glDisable(GL_FOG);
+
+        return true;
 }
 
@@ -160 +164 @@
         //glEnable(GL_BLEND);
         //glBlendFunc(GL_SRC_ALPHA, GL_DST_ALPHA);
-        
+
         int i;
         glLoadIdentity();
 
-        
+
         glBegin( GL_LINES );
                 glNormal3f(0,1,0);
@@ -210 +214 @@
 * ticks the effect if there is any time dependancy
 */
-void VolFogEffect::tick(float dt)       
-{
-}
+void VolFogEffect::tick(float dt)
+{
+}

trunk/src/lib/graphics/effects/weather_effect.cc

@@ -56 +56 @@
  */
 bool WeatherEffect::init()
-{}
+{
+  return true;
+}
 
 

trunk/src/lib/graphics/graphics_engine.cc

@@ -145 +145 @@
     return -1;
   this->initVideo(640, 480, 16);
+  return 1;
 }
 
@@ -161 +162 @@
   // looking if we are in fullscreen-mode
   MultiType textures = Preferences::getInstance()->getString(CONFIG_SECTION_VIDEO_ADVANCED, CONFIG_NAME_TEXTURES, "1");
-    Texture::setTextureEnableState(textures.getBool());
+  Texture::setTextureEnableState(textures.getBool());
 
   // searching for a usefull resolution
@@ -167 +168 @@
   //resolution.debug();
   MultiType x = resolution.getString(0), y = resolution.getString(1);
-  this->initVideo(x.getInt(), y.getInt(), 16);
+  return this->initVideo(x.getInt(), y.getInt(), 16);
 
   //   GraphicsEffect* fe = new FogEffect(NULL);
@@ -203 +204 @@
   {
     PRINTF(1)("could not initialize SDL Video\n");
-    //      return -1;
+    return -1;
   }
   // initialize SDL_GL-settings
@@ -238 +239 @@
 
   this->isInit = true;
+  return 1;
 }
 
@@ -263 +265 @@
  * @brief Sets the GL-attributes
  */
-int GraphicsEngine::setGLattribs()
+void GraphicsEngine::setGLattribs()
 {
   // Set video mode
@@ -358 +360 @@
     //    SDL_Quit();
     //    return -1;
+    return -1;
   }
   glViewport(0, 0, width, height);                     // Reset The Current Viewport
@@ -379 +382 @@
   }
 #endif /* __WIN32__ */
+  return 1;
 }
 
@@ -410 +414 @@
  * @param resizeInfo SDL information about the size of the new screen size
  */
-int GraphicsEngine::resolutionChanged(const SDL_ResizeEvent& resizeInfo)
+void GraphicsEngine::resolutionChanged(const SDL_ResizeEvent& resizeInfo)
 {
   this->setResolution(resizeInfo.w, resizeInfo.h, this->bitsPerPixel);
@@ -655 +659 @@
   switch (event.type)
   {
-  case EV_VIDEO_RESIZE:
-    this->resolutionChanged(event.resize);
-    break;
-  }
-}
+    case EV_VIDEO_RESIZE:
+      this->resolutionChanged(event.resize);
+      break;
+  }
+}

trunk/src/lib/graphics/graphics_engine.h

@@ -54 +54 @@
     inline int getbbp() const { return this->bitsPerPixel; };
 
-    int resolutionChanged(const SDL_ResizeEvent& resizeInfo);
+    void resolutionChanged(const SDL_ResizeEvent& resizeInfo);
 
     static void enter2DMode();
@@ -88 +88 @@
     GraphicsEngine();
     int initVideo(unsigned int resX, unsigned int resY, unsigned int bbp);
-    int setGLattribs();
+    void setGLattribs();
     void grabHardwareSettings();
 

trunk/src/lib/graphics/importer/height_map.cc

@@ -108 +108 @@
 
   float height = 0;
-  int offset = 0;
 
   float r = 0.0;
@@ -474 +473 @@
   Vector v = State::getCameraNode()->getAbsCoor();
 
-  int i_min = 0;
   int i_max = (heightMap->h )/ HM_TILE_SIZE;
-  int j_min = 0;
   int j_max= (heightMap->w  ) / HM_TILE_SIZE;
 

trunk/src/lib/graphics/importer/material.cc

@@ -169 +169 @@
   }
   Material::selectedMaterial = this;
+
+  return true;
 }
 
@@ -411 +413 @@
 void Material::setAmbientMap(const std::string& aMap, GLenum target)
 {
-  SDL_Surface* ambientMap;
+  /// FIXME SDL_Surface* ambientMap;
 
 }
@@ -422 +424 @@
 void Material::setSpecularMap(const std::string& sMap, GLenum target)
 {
-  SDL_Surface* specularMap;
+  /// FIXME SDL_Surface* specularMap;
 
 }
@@ -437 +439 @@
 
 
-int Material::getMaxTextureUnits()
+unsigned int Material::getMaxTextureUnits()
 {
   int maxTexUnits = 0;

trunk/src/lib/graphics/importer/material.h

@@ -69 +69 @@
     static const GLenum glTextureArbs[];  //!< The Texture ARB's
 
-    static int getMaxTextureUnits();
+    static unsigned int getMaxTextureUnits();
 
   private:

trunk/src/lib/graphics/importer/md2Model.cc

@@ -290 +290 @@
 {
   //static sVec3D this->verticesList[MD2_MAX_VERTICES]; /* performance: created only once in a lifetime */
-  int* pCommands = this->data->pGLCommands;
+//  int* pCommands = this->data->pGLCommands;
   /* some face culling stuff */
 //   glPushAttrib(GL_POLYGON_BIT);
@@ -541 +541 @@
   delete [] buffer;
   fclose(pFile);
+  return true;
 }
 
@@ -563 +564 @@
   this->material.setIllum(3);
   this->material.setAmbient(1.0, 1.0, 1.0);
-}
+  return true;
+}

trunk/src/lib/graphics/importer/media_container.cc

@@ -182 +182 @@
 
   // get all the frames and save them in the sequence
-  while(this->addFrame(this->getNextFrame()) != NULL);
+  while(this->addFrame(this->getNextFrame()) != 0);
 }
 

trunk/src/lib/graphics/importer/model.cc

@@ -67 +67 @@
 
   glBegin(GL_TRIANGLES);
-  for( int i = 0; i < this->pModelInfo.numTriangles; ++i)
+  for(unsigned int i = 0; i < this->pModelInfo.numTriangles; ++i)
     {
       //printf("int i = %i\n", i);

trunk/src/lib/graphics/importer/primitive_model.cc

@@ -105 +105 @@
   // defining the binding Faces.
   int v1, v2, v3, v4;
-  for (int i = 0; i <= detail * 2 -1; i++)
-  {
-    for (int j = 0; j <= detail; j++)
+  for (unsigned int i = 0; i <= detail * 2 -1; i++)
+  {
+    for (unsigned int j = 0; j <= detail; j++)
     {
 
@@ -147 +147 @@
   if (detail <= 3)
     detail = 3;
-  int count = 0;
+  unsigned int count = 0;
   // defining Points of the Cylinder.
   for (float phi = 0.0; phi < 2.0*PI; phi += 2.0*PI/(float)detail)
@@ -163 +163 @@
   }
   // adding Faces
-  for (int i = 0; i < detail-1; i++)
+  for (unsigned int i = 0; i < detail-1; i++)
   {
     int p1, p2, p3, p4;
@@ -204 +204 @@
 
   //defining Faces
-  for (int i = 0; i < detail; i++)
+  for (unsigned int i = 0; i < detail; i++)
   {
     unsigned int v1, v2;
     v1 = i+2;
-    if (i == detail -1)
+    if (i+1 == detail)
       v2 = 2;
     else
@@ -225 +225 @@
 {
   //defining vertices
-  for (int i = 0; i < detail; i++)
-    for (int j = 0; j < detail; j++)
+  for (unsigned int i = 0; i < detail; i++)
+    for (unsigned int j = 0; j < detail; j++)
     {
       this->addVertex(((float)i/(float)(detail-1) -.5)*size,
@@ -236 +236 @@
   //defining Faces
   int v1, v2, v3, v4;
-  for (int i = 0; i < detail-1; i++)
-    for (int j = 0; j < detail-1; j++)
+  for (unsigned int i = 0; i < detail-1; i++)
+    for (unsigned int j = 0; j < detail-1; j++)
     {
       v1 = i*detail + j;

trunk/src/lib/graphics/importer/static_model.cc

@@ -386 +386 @@
   }
   ++this->groupCount;
+  return true;
 }
 
@@ -558 +559 @@
   this->currentGroup->faceCount += this->currentGroup->currentFace->vertexCount -2;
   this->faceCount += this->currentGroup->currentFace->vertexCount -2;
+  return true;
 }
 
@@ -591 +593 @@
   this->currentGroup->faceCount += this->currentGroup->currentFace->vertexCount - 2;
   this->faceCount += this->currentGroup->currentFace->vertexCount -2;
+  return true;
 }
 
@@ -606 +609 @@
   if (this->currentGroup->faceCount == 0)
     this->currentGroup->faceCount++;
+  return true;
 }
 
@@ -621 +625 @@
   if (this->currentGroup->faceCount == 0)
     this->currentGroup->faceCount++;
+  return true;
 }
 
@@ -637 +642 @@
 
   Vector* normArray = new Vector [vertices.size()/3];
-  for (int i=0; i<vertices.size()/3;i++)
+  for (unsigned int i=0; i<vertices.size()/3;i++)
     normArray[i] = Vector(.0,.0,.0);
 
-  int firstTouch;
-  int secondTouch;
   Vector prevV;
   Vector nextV;
@@ -668 +671 @@
 
         curElem = firstElem;
-        for (int j=0; j<tmpFace->vertexCount; j++)
+        for (unsigned int j = 0; j < tmpFace->vertexCount; j++)
         {
           if (!(nextElem = curElem->next))
@@ -696 +699 @@
   }
 
-  for (int i=0; i < this->vertices.size()/3;i++)
+  for (unsigned int i=0; i < this->vertices.size()/3;i++)
   {
     normArray[i].normalize();
@@ -705 +708 @@
   }
   delete[] normArray;
+  return true;
 }
 
@@ -741 +745 @@
           glEnd();
         this->currentGroup->faceMode = 0;
-        Material* tmpMat;
         if (tmpFace->material != NULL)
         {
@@ -797 +800 @@
     this->currentGroup = this->currentGroup->next;
   }
+  return true;
 }
 
@@ -933 +937 @@
   PRINTF(5)("importing grafical Element to openGL.\n");
 
-  if (elem->texCoordNumber != -1)
-  {
-    if (likely(elem->texCoordNumber < this->pModelInfo.numTexCoor))
+  if (elem->texCoordNumber > -1)
+  {
+    if (likely((unsigned int)elem->texCoordNumber < this->pModelInfo.numTexCoor))
       glTexCoord2fv(&this->vTexture[0] + elem->texCoordNumber * 2);
     else
@@ -941 +945 @@
                 elem->texCoordNumber, this->pModelInfo.numTexCoor);
   }
-  if (elem->normalNumber != -1)
-  {
-    if (likely(elem->normalNumber < this->pModelInfo.numNormals))
+  if (elem->normalNumber > -1)
+  {
+    if (likely((unsigned int)elem->normalNumber < this->pModelInfo.numNormals))
       glNormal3fv(&this->normals[0] + elem->normalNumber * 3);
     else
@@ -949 +953 @@
                 elem->normalNumber, this->pModelInfo.numNormals);
   }
-  if (elem->vertexNumber != -1)
-  {
-    if (likely(elem->vertexNumber < this->pModelInfo.numVertices))
+  if (elem->vertexNumber > -1)
+  {
+    if (likely((unsigned int)elem->vertexNumber < this->pModelInfo.numVertices))
       glVertex3fv(&this->vertices[0]+ elem->vertexNumber * 3);
     else
@@ -958 +962 @@
   }
 
+  return true;
 }
 

trunk/src/lib/graphics/importer/tc.cc

@@ -707 +707 @@
     if(tc->UsingStaticVerts) {
         double edgeTotal;
-        for(i = 0; i < tc->VertRange; i++) {
+        for(i = 0; i < (int)tc->VertRange; i++) {
             v = &tc->StaticVerts[i];
             if(v->Count > 0) {
@@ -830 +830 @@
 {
     int i;
-    int size;
+    int size = 1;
     ACTCVertex *v;
 
@@ -838 +838 @@
     if(tc->UsingStaticVerts) {
         size = tc->VertRange * sizeof(ACTCVertex);
-        for(i = 0; i < tc->VertRange; i++) {
+        for(i = 0; i < (int)tc->VertRange; i++) {
             v = &tc->StaticVerts[i];
             if(v->Count > 0)
@@ -915 +915 @@
 
         case ACTC_IN_MAX_VERT:
-            if(value < tc->MinInputVert) {
+          if(value < (int)tc->MinInputVert) {
                 ACTC_DEBUG(fprintf(stderr, "actcParami : tried to set "
                     "MAX_INPUT_VERT to %d, less than MIN_INPUT_VERT (%d)\n",
@@ -925 +925 @@
 
         case ACTC_IN_MIN_VERT:
-            if(value > tc->MaxInputVert) {
+          if(value > (int)tc->MaxInputVert) {
                 ACTC_DEBUG(fprintf(stderr, "actcParami : tried to set "
                     "MIN_INPUT_VERT to %d, greater than MAX_INPUT_VERT (%d)\n",
@@ -1065 +1065 @@
 static int mapVertexEdge(ACTCData *tc, ACTCVertex *v1, ACTCVertex *v2, ACTCEdge **edge)
 {
-    unsigned int i;
+    int i;
     ACTCEdge tmp;
     void *r;

trunk/src/lib/graphics/importer/texture.cc

@@ -33 +33 @@
 /*
  * On the BIG_ENDIAN architecture, the 24 and 32bit bitmaps have
- * different masks. If you don't do this distinction properly, 
+ * different masks. If you don't do this distinction properly,
  * you will get weird-looking textures.
  */
@@ -161 +161 @@
   unsigned int* pixels = new unsigned int[width * height * channels];
   memset(pixels, 0, width * height * channels * sizeof(unsigned int));
-  
-  
+
+
   glTexParameterf(GL_TEXTURE_2D,GL_TEXTURE_MIN_FILTER,GL_LINEAR);
   glTexParameterf(GL_TEXTURE_2D,GL_TEXTURE_MAG_FILTER,GL_LINEAR);
@@ -169 +169 @@
 
   glTexImage2D(target, 0, channels, width, height, 0, type, GL_UNSIGNED_INT, pixels);
-  
-  
-  
+
+
+
   delete[] pixels;
 
@@ -309 +309 @@
 
         Uint32* mask = opaqueMask;
-        
+
   /* Save the alpha blending attributes */
   saved_flags = surface->flags&(SDL_SRCALPHA | SDL_RLEACCELOK);
@@ -320 +320 @@
                 mask = alphaMask;
   }
-                
+
   retSurface = SDL_CreateRGBSurface(SDL_HWSURFACE,
                                     surface->w, surface->h,
@@ -360 +360 @@
   int      errorCode = 0;           //!< the error code for the texture loading functions
   GLuint   texture = 0;             //!< the OpenGL texture handle
-  int      mipmapLevel = 0;         //!< the maximum mipmap level for this texture
-  int      mipmapWidth = 0;         //!< the width of the mipmap
-  int      mipmapHight = 0;         //!< the height of the mipmap
+  //int      mipmapLevel = 0;         //!< the maximum mipmap level for this texture
+  //int      mipmapWidth = 0;         //!< the width of the mipmap
+  //int      mipmapHight = 0;         //!< the height of the mipmap
   GLenum   format = GL_RGB;
   if (surface->format->BitsPerPixel == 32)

trunk/src/lib/graphics/render2D/element_2d.cc

@@ -961 +961 @@
         /* update the current absCoordinate */
         this->prevRelCoordinate = this->relCoordinate;
-        float sine = sin(this->parent->getAbsDir2D());
-        float cose = cos(this->parent->getAbsDir2D());
+//         float sine = sin(this->parent->getAbsDir2D());
+//         float cose = cos(this->parent->getAbsDir2D());
         //        this->absCoordinate.x = this->relCoordinate.x*cose - this->relCoordinate.y*sine + this->parent->getRelCoor2D().x*(1-cose) +this->parent->getRelCoor2D().y*sine;
         //        this->absCoordinate.y = this->relCoordinate.x*sine + this->relCoordinate.y*cose + this->parent->getRelCoor2D().y*(1-cose) +this->parent->getRelCoor2D().x*sine;
@@ -1167 +1167 @@
   else if (parentingMode == E2D_PARENT_ROTATE_AND_MOVE)
     return "rotate-and-move";
+  else return "all";
 }
 
@@ -1186 +1187 @@
   else  if (parentingMode == "rotate-and-move")
     return (E2D_PARENT_ROTATE_AND_MOVE);
+  else return E2D_PARENT_ALL;
 }
 

trunk/src/lib/graphics/shader.cc

@@ -153 +153 @@
       glAttachObjectARB(this->shaderProgram, shader);
   }
+  return true;
 }
 

trunk/src/lib/graphics/text_engine/font.cc

@@ -220 +220 @@
 
 /**
- *  sets a specific renderStyle
+ * @brief sets a specific renderStyle
  * @param renderStyle the Style to render: a string (char-array) containing:
  *   i: italic, b: bold, u, underline
@@ -228 +228 @@
   this->renderStyle = TTF_STYLE_NORMAL;
 
-  for (int i = 0; i < renderStyle.size(); i++)
+  for (unsigned int i = 0; i < renderStyle.size(); i++)
   {
     if (renderStyle[i] == 'b')
@@ -246 +246 @@
 
 /**
- * creates and exports an Image, that has all the characters
+ * @brief creates and exports an Image, that has all the characters
  * stored in a Array (as an image)
  * @param fileName the File to write the image into.
@@ -257 +257 @@
 
   //
-  SDL_Color tmpColor = {0, 0, 0};
   // Surface definition.
   SDL_Rect tmpRect; // this represents a Rectangle for blitting.
@@ -277 +276 @@
   tmpRect.x = 0; tmpRect.y = 0; tmpRect.w = tmpSurf->w; tmpRect.h = tmpSurf->h;
   SDL_SetClipRect(tmpSurf, &tmpRect);
-  int maxLineHeight = 0;
 
   int posX, posY;
@@ -339 +337 @@
 /**
  * @returns the maximum ascent of the Font, if the font was initialized, 0 otherwise
-
-   the ascent is the pixels of the font above the baseline
+ *
+ * the ascent is the pixels of the font above the baseline
  */
 int Font::getMaxAscent() const
@@ -352 +350 @@
 /**
  * @returns the maximum descent of the Font, if the font was initialized, 0 otherwise
-
-   the descent is the pixels of the font below the baseline
+ *
+ * the descent is the pixels of the font below the baseline
  */
 int Font::getMaxDescent() const
@@ -404 +402 @@
 
 /**
- * creates a Fast-Texture of this Font
+ * @brief creates a Fast-Texture of this Font
  */
 bool Font::createFastTexture()
@@ -497 +495 @@
   if (this->setSurface(tmpSurf))
     (this->setTexture(Texture::loadTexToGL(tmpSurf)));
+  return true;
 }
 
@@ -547 +546 @@
     return 0;
 
-  int i;
-  int x,y; // the counters
+  unsigned int i;
+  unsigned int x,y; // the counters
   int maxLineHeight = this->getMaxHeight();
   unsigned int size = 32;  // starting Value, we have to start somewhere 32 seems reasonable. (take any small enough 2^i number)
@@ -587 +586 @@
 
 /**
- *  a simple function to get some interesting information about this class
+ * @brief a simple function to get some interesting information about this class
  */
 void Font::debug()
 {
   // print the loaded font's style
-  int style;
+  int style = TTF_STYLE_NORMAL;
   if (likely(this->fontTTF != NULL))
     style = TTF_GetFontStyle(this->fontTTF);

trunk/src/lib/gui/gl/glmenu/glmenu_imagescreen.cc

@@ -269 +269 @@
     glEnd();
 
-    /* draw black border
     glBegin(GL_QUADS);
     glColor3f(0.0, 0.0, 0.0);

trunk/src/lib/gui/gl/signal_connector.cc

@@ -72 +72 @@
     this->object = signalConnector.object;
     this->exec = (signalConnector.exec == NULL) ? NULL : signalConnector.exec->clone();
+
+    return *this;
   }
 

trunk/src/lib/gui/qt/gui_control.cc

@@ -283 +283 @@
         ///case Qt::Key_Power: return SDLK_POWER;
         ///case Qt::Key_Euro: return SDLK_EURO;
+      default:
         return -1;
     }

trunk/src/lib/lang/class_list.cc

@@ -367 +367 @@
       while (std::pow((float)10, (int)lenCount) <= (*cl).objectList.size())
         ++lenCount;
-      for (int i=0; i < 30-(*cl).className.size() - lenCount; i++)
+      for (unsigned int i=0; i < 30-(*cl).className.size() - lenCount; i++)
         (niceString[i]) = ' ';
       niceString[30-(*cl).className.size() - lenCount] = '\0';

trunk/src/lib/parser/cmdline_parser/cmdline_parser.cc

@@ -52 +52 @@
   else
     argTable.push_front( entry );
+  return true;
 }
 
@@ -70 +71 @@
       if ( s.length() > 2 && s[0] == '-' && s[1] != '-' )
       {
-        for ( int j = 1; j < s.length(); j++ )
+        for (unsigned int j = 1; j < s.length(); j++ )
         {
           std::string t = "-";
@@ -95 +96 @@
   }
 
-  int i = 0;
+  unsigned int i = 0;
 
   ArgTable::iterator it;
@@ -110 +111 @@
         found = true;
 
-        int posArgs = 1;
+        unsigned int posArgs = 1;
 
         while ( i + posArgs < args.size() )
@@ -130 +131 @@
         std::vector<MultiType> argArgs;
 
-        for ( int j = 1; j <= it->numArgs; j++ )
+        for (unsigned int j = 1; j <= it->numArgs; j++ )
           argArgs.push_back( args[i+j] );
 
@@ -219 +220 @@
     assert( it->numArgs == substr.size() );
 
-    for ( int i = 0; i<it->numArgs; i++ )
+    for (unsigned int i = 0; i<it->numArgs; i++ )
     {
       args += " [" + substr[i] + "]";
@@ -251 +252 @@
   output.reverse();
 
-  int maxShort = 0;
-  int maxLong = 0;
+  unsigned int maxShort = 0;
+  unsigned int maxLong = 0;
 
   std::list<std::vector<std::string> >::const_iterator it;
@@ -269 +270 @@
     printf("%s ", (*it)[0].c_str());
 
-    for ( int i = 0; i<maxShort-(*it)[0].length(); i++ )
+    for (unsigned  int i = 0; i<maxShort-(*it)[0].length(); i++ )
       printf(" ");
 
     printf("%s ", (*it)[1].c_str());
 
-    for ( int i = 0; i<maxLong-(*it)[1].length(); i++ )
+    for (unsigned int i = 0; i<maxLong-(*it)[1].length(); i++ )
       printf(" ");
 

trunk/src/lib/parser/cmdline_parser/cmdline_parser.h

@@ -17 +17 @@
 struct ArgTableEntry
 {
-  int         id;
-  std::string longOption;
-  char        shortOption;
-  int         numArgs;
-  std::string argNames;
-  std::string help;
+  unsigned int    id;
+  std::string     longOption;
+  char            shortOption;
+  unsigned int    numArgs;
+  std::string     argNames;
+  std::string     help;
 };
 
@@ -36 +36 @@
     CmdLineParser();
     virtual ~CmdLineParser();
-    
+
     bool add( int id, const std::string& longOption, char shortOption, int numArgs, const std::string & argNames, const std::string& help, bool back=false );
 
     bool parse( ArgParserCallback cb, void* data, int argc, char** argv );
-    
+
     void showHelp();
-    
+
   private:
     ArgTable argTable;
     std::string exeName;
-    
+
     inline bool matches( ArgTableEntry entry, std::string arg, bool & finish );
 

trunk/src/lib/parser/preferences/cmd_line_prefs_reader.cc

@@ -43 +43 @@
 {
   CallbackData * cbd = (CallbackData *)data;
-  
+
   switch ( entry.id )
   {
@@ -54 +54 @@
       key.erase( 0, key.find(".")+1 );
       //PRINTF(0)("SECTION '%s', KEY '%s'\n", section.c_str(), key.c_str());
-      
+
       if ( key == "" || section == "" || argArgs.size() != 1 )
       {
@@ -60 +60 @@
         return false;
       }
-      
+
       //Preferences::getInstance()->setMultiType( section, key, argArgs[0], true );
       cbd->iniEntries.push_back( IniEntry() );
@@ -72 +72 @@
       return true;
   }
-  
-  if ( entry.id >= ID_PREFS && entry.id - ID_PREFS < regArgs.size() )
+
+  if ( entry.id >= ID_PREFS && entry.id  < regArgs.size() + ID_PREFS)
   {
     if ( regArgs[entry.id - ID_PREFS].value == "%arg%" )
     {
       assert( argArgs.size() == 1 );
-      
+
       cbd->iniEntries.push_back( IniEntry() );
       cbd->iniEntries.back().section = regArgs[entry.id - ID_PREFS].section;
@@ -96 +96 @@
     assert(false);
   }
-  
+
   return true;
 }
@@ -104 +104 @@
 {
   CmdLineParser parser;
-  
+
   parser.add( ID_HELP, "help", 'h', 0, "", "Shows this help and exits");
-  
-  for ( int i = 0; i<regArgs.size(); i++ )
+
+  for (unsigned int i = 0; i<regArgs.size(); i++ )
   {
     if ( regArgs[i].value == "%arg%" )
@@ -118 +118 @@
     }
   }
-  
+
   parser.add( ID_SET_INI, "set-%", '\0', 1, "value", "Override a configuration element." );
-  
+
   CallbackData cbd;
-  
+
   cbd.parser = &parser;
-  
+
   if ( parser.parse( &callBack, &cbd, argc, argv ) )
   {
@@ -137 +137 @@
     exit(EXIT_FAILURE);
   }
-  
+
   return true;
 }
@@ -144 +144 @@
 {
   RegistredArgument arg;
-  
+
   arg.longOption = longOption;
   while ( arg.longOption.find("_") != std::string::npos )
@@ -158 +158 @@
   arg.section = section;
   arg.key = key;
-  
+
   regArgs.push_back( arg );
+  return true;
 }
 

trunk/src/lib/sound/ogg_player.cc

@@ -163 +163 @@
   bool OggPlayer::play()
   {
-    /*if (!(this->state & OggPlayer::FileOpened))
+    if (!(this->state & OggPlayer::FileOpened))
       return false;
 
@@ -173 +173 @@
     if (this->musicThreadID == NULL)
       return ((this->musicThreadID = SDL_CreateThread(OggPlayer::musicThread, (void*)this)) != NULL);
-    return true;*/
+    return true;
   }
 

trunk/src/lib/util/loading/dynamic_loader.cc

@@ -60 +60 @@
   }
   void *mkr = dlsym( this->handle, "maker");
+  return (mkr != NULL);
 }
 
@@ -72 +73 @@
   }
 //  void *mkr = dlsym("maker");
-
+  return true;
 }
 
@@ -78 +79 @@
 BaseObject* DynamicLoader::fabricateObject(const TiXmlElement* root) const
 {
+  /// FIXME
+  return NULL;
 }
 #endif

trunk/src/lib/util/loading/load_param.cc

@@ -29 +29 @@
  */
 CLoadParam::CLoadParam(const TiXmlElement* root, const std::string& paramName, BaseObject* object, Executor* executor, bool inLoadCycle)
-  :  paramName(paramName), object(object)
+  :  object(object), paramName(paramName)
 {
   this->object = object;

trunk/src/lib/util/loading/load_param_description.cc

@@ -275 +275 @@
 
   return retVal;*/
-}
+  std::list<std::string> a;
+  return a;
+}

trunk/src/lib/util/loading/resource_manager.cc

@@ -557 +557 @@
 bool ResourceManager::unloadAllByPriority(ResourcePriority prio)
 {
+  bool removedAll = true;
   unsigned int removeCount;
   for (unsigned int round = 0; round < 3; round++)
@@ -571 +572 @@
         {
           if (round == 3)
+          {
             PRINTF(2)("unable to unload %s because there are still %d references to it\n",
                       this->resourceList[index]->name.c_str(), this->resourceList[index]->count);
+            removedAll = false;
+          }
           removeCount++;
         }
@@ -580 +584 @@
     if (removeCount == 0) break;
   }
+  return removedAll;
 }
 
@@ -711 +716 @@
   }
   return tmp;*/
+  return "";
 }
 
@@ -733 +739 @@
     return this->cache(splits[1], ResourceManager::stringToResourceType(splits[0]),
                 RP_LEVEL, splits[2], splits[3], splits[4]);*/
+  return false;
 }
 

trunk/src/lib/util/multi_type.cc

@@ -160 +160 @@
     case MT_STRING:
       return (this->storedString == mt.storedString);
+    default:
+      return false;
   }
 }
@@ -190 +192 @@
       this->setString(this->getString());
       break;
+    default:
+      this->type = type;
   }
 }
@@ -477 +481 @@
     case MT_STRING:
       return MultiType::typeNames[5];
-  }
-  return MultiType::typeNames[0];
+    default:
+      return MultiType::typeNames[0];
+  }
 }
 

trunk/src/orxonox.cc

@@ -203 +203 @@
 
   PRINTF(3)("Parsed Config File: '%s'\n", this->configFileName.c_str());
+  return this->configFileName;
 }
 
@@ -459 +460 @@
 
   return startOrxonox(argc, argv, "", -1);
-  return 0;
 }
 
@@ -497 +497 @@
     startOrxonox(argc, argv, "", port);
   }
+  return 1;
 }
 
@@ -546 +547 @@
   delete orx;
   File("~/.orxonox/orxonox.lock").remove();
-}
+  return 1;
+}

trunk/src/world_entities/playable.cc

@@ -79 +79 @@
   // this->setPlayer(NULL);
   // IN ITS DESTRUCTOR.
-  
+
   assert(this->currentPlayer == NULL);
 }
@@ -118 +118 @@
         this->increaseHealthMax(ppu->getValue());
         return true;
+      default:
+        /// EVERYTHING THAT IS NOT HANDLED
+        /// FIXME
+        return false;
     }
   }

trunk/src/world_entities/player.cc

@@ -81 +81 @@
       return true;
   }
-  
+
   if ( playable == NULL )
     this->playable = NULL;
@@ -90 +90 @@
 bool Player::eject()
  {
-   this->setPlayable(NULL);
+   return this->setPlayable(NULL);
  }
 

trunk/src/world_entities/projectiles/bomb.cc

@@ -37 +37 @@
     this->loadParams(root);
 
-  float modelSize = 1.0;
   this->loadModel("models/projectiles/RadioActiveBomb.obj", 1.0);
 

trunk/src/world_entities/projectiles/guided_missile.cc

@@ -35 +35 @@
   this->setClassID(CL_GUIDED_MISSILE, "GuidedMissile");
 
-  float modelSize = .3;
   this->loadModel("models/projectiles/orx-rocket.obj", .3);
   this->loadExplosionSound("sound/explosions/explosion_4.wav");

trunk/src/world_entities/projectiles/hyperblast.cc

@@ -36 +36 @@
   this->setClassID(CL_HYPERBLAST, "Hyperblast");
 
-  float modelSize = .3;
   this->loadModel("models/projectiles/hyperblast.obj", 5);
 

trunk/src/world_entities/projectiles/rocket.cc

@@ -36 +36 @@
   this->setClassID(CL_ROCKET, "Rocket");
 
-  float modelSize = .3;
   this->loadModel("models/projectiles/orx-rocket.obj", .3);
 

trunk/src/world_entities/projectiles/test_bullet.cc

@@ -35 +35 @@
   this->setClassID(CL_TEST_BULLET, "TestBullet");
 
-  float modelSize = .3;
   this->loadModel("models/projectiles/orx-rocket.obj", .3);
 

trunk/src/world_entities/space_ships/hover.cc

@@ -175 +175 @@
   this->getWeaponManager().getFixedTarget()->setParent(&this->cameraNode);
   this->getWeaponManager().getFixedTarget()->setRelCoor(1000,0,0);
-  
+
   // NETWORK THINGS
-  
+
   registerVar( new SynchronizeableBool( &bForward, &bForward, "bForward", PERMISSION_OWNER ) );
   registerVar( new SynchronizeableBool( &bBackward, &bBackward, "bBackward", PERMISSION_OWNER ) );
@@ -184 +184 @@
   registerVar( new SynchronizeableBool( &bAscend, &bAscend, "bAscend", PERMISSION_OWNER ) );
   registerVar( new SynchronizeableBool( &bDescend, &bDescend, "bDescend", PERMISSION_OWNER ) );
-  
+
   registerVar( new SynchronizeableFloat( &rotation, &rotation, "rotation", PERMISSION_OWNER ) );
 }
@@ -262 +262 @@
 {
   Vector accel(0.0, 0.0, 0.0);
-  float rotSpeed = .3;
 
   if( this->bForward )

trunk/src/world_entities/space_ships/turbine_hover.cc

@@ -297 +297 @@
 {
   Vector accel(0.0, 0.0, 0.0);
-  float rotSpeed = .3;
 
   if( this->bForward )
@@ -416 +415 @@
     }
     break;
+    default:
+      PRINTF(2)("Playmode %s Not Implemented\n", Playable::playmodeToString(this->getPlaymode()).c_str());
   }
 }

trunk/src/world_entities/terrain.cc

@@ -63 +63 @@
   this->init();
 
-  if (fileName.rfind(".obj" ) != -1 || fileName.rfind(".OBJ") != -1 )
+  if (fileName.rfind(".obj" ) != std::string::npos || fileName.rfind(".OBJ") != std::string::npos )
   {
     this->loadModel(fileName);

trunk/src/world_entities/test_entity.cc

@@ -122 +122 @@
 
   this->bDeath = true;
-  float anim;
   int randi = (int)(5.0f * (float)rand()/(float)RAND_MAX);
 

trunk/src/world_entities/weapons/weapon.cc

@@ -86 +86 @@
     }
   }
+  return NULL;
 }
 
@@ -421 +422 @@
       return this->activateW();
       break;
+    default:
+      PRINTF(2)("Action %s Not Implemented yet \n", Weapon::actionToChar(action));
+      return false;
   }
 }
@@ -442 +446 @@
     this->enterState(WS_ACTIVATING);
   }
+  return true;
 }
 
@@ -460 +465 @@
     this->enterState(WS_DEACTIVATING);
   }
+
+  return true;
 }
 
@@ -483 +490 @@
     this->requestAction(WA_RELOAD);
   }
+  return true;
 }
 
@@ -509 +517 @@
     this->execute();
   }
+  return true;
 }
 
@@ -539 +548 @@
   this->reload();
   this->enterState(WS_RELOADING);
+
+  return true;
 }
 

trunk/src/world_entities/world_entity.cc

@@ -401 +401 @@
     if( this->collisionHandles[i] != NULL)
       this->collisionHandles[i]->registerCollisionEvent(c);
+  return true;
 }
 
@@ -424 +425 @@
     if( this->collisionHandles[i] != NULL)
       this->collisionHandles[i]->registerCollisionEvent(c);
+  return true;
 }