Changeset 7455 in orxonox.OLD for trunk/src/lib/graphics

Timestamp:

Apr 29, 2006, 10:38:12 PM (18 years ago)

Author:

bensch

Message:

orxonox/trunk: fixed a SegFault (that this worked before is some wonder

Location:

trunk/src/lib/graphics/text_engine

Files:

• multi_line_text.cc (modified) (2 diffs)
• text.cc (modified) (3 diffs)

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

@@ -91 +91 @@
   for (unsigned int i = 0; i < this->getText().size(); ++i)
   {
-    if (unlikely(this->getText()[i] == '\n' || i == this->lineEnds[lineNumber]))
+    if (unlikely(!this->lineEnds.empty() && i == this->lineEnds[lineNumber]))
     {
       // go to the next Line.
@@ -140 +140 @@
       width += this->getFont()->getGlyphArray()[this->getText()[i]]->advance;
   }
-  this->setSizeY2D(this->lineEnds.size() * (this->lineSpacing+this->getFont()->getMaxHeight()));
+  this->setSizeY2D(this->lineEnds.size() * (this->lineSpacing + this->getFont()->getMaxHeight()));
 }

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

@@ -35 +35 @@
   // initialize this Text
   this->font = NULL;
-  this->text = "";
-  this->setAlignment(TEXT_DEFAULT_ALIGNMENT);
+
+  this->setFont(fontFile, FONT_DEFAULT_RENDER_SIZE);
+
   this->blending = TEXT_DEFAULT_BLENDING;
   this->color = TEXT_DEFAULT_COLOR;
-
-  if (!fontFile.empty())
-    this->setFont(fontFile, FONT_DEFAULT_RENDER_SIZE);
-
+  this->setAlignment(TEXT_DEFAULT_ALIGNMENT);
   this->setSize(TEXT_DEFAULT_SIZE);
 }
@@ -63 +61 @@
 void Text::setFont(const std::string& fontFile, unsigned int fontSize)
 {
-  Font* newFont;
+  Font* newFont = NULL;
   Font* oldFont = this->font;
 
@@ -76 +74 @@
     }
   }
-  else
+  if (newFont == NULL)
     newFont = Font::getDefaultFont();
+  assert(newFont != NULL);
 
   // unloading the Font if we alrady have one loaded.