aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPaul Gilbert2010-08-22 02:06:18 +0000
committerEugene Sandulenko2010-10-12 23:13:24 +0000
commitb294e88f7e89a0b3d89b5d184e1b491eaddff70f (patch)
tree0b54d8bd083aa9783d023a2255fabc4ac4a3dbcd
parent2c6201683854d3f56c93f3f2ca1f826b1b2f450e (diff)
downloadscummvm-rg350-b294e88f7e89a0b3d89b5d184e1b491eaddff70f.tar.gz
scummvm-rg350-b294e88f7e89a0b3d89b5d184e1b491eaddff70f.tar.bz2
scummvm-rg350-b294e88f7e89a0b3d89b5d184e1b491eaddff70f.zip
SWORD25: Converted FontResource to use ScummVM XML Parser
svn-id: r53277
-rw-r--r--engines/sword25/gfx/fontresource.cpp200
-rw-r--r--engines/sword25/gfx/fontresource.h36
2 files changed, 80 insertions, 156 deletions
diff --git a/engines/sword25/gfx/fontresource.cpp b/engines/sword25/gfx/fontresource.cpp
index 0e5257a872..f51c246769 100644
--- a/engines/sword25/gfx/fontresource.cpp
+++ b/engines/sword25/gfx/fontresource.cpp
@@ -41,7 +41,6 @@
#include "sword25/kernel/kernel.h"
#include "sword25/kernel/string.h"
#include "sword25/package/packagemanager.h"
-#include "sword25/util/tinyxml/tinyxml.h"
#include "sword25/gfx/fontresource.h"
@@ -61,138 +60,63 @@ static const unsigned int DEFAULT_GAPWIDTH = 1;
FontResource::FontResource(Kernel *pKernel, const Common::String &FileName) :
_pKernel(pKernel),
_Valid(false),
- Resource(FileName, Resource::TYPE_FONT) {
- // XML Fontdatei parsen
- TiXmlDocument Doc;
- if (!_ParseXMLDocument(FileName, Doc)) {
- BS_LOG_ERRORLN("The following TinyXML-Error occured while parsing \"%s\": %s", GetFileName().c_str(), Doc.ErrorDesc());
- return;
- }
-
- // Font-Tag finden
- TiXmlElement *pElement = Doc.FirstChildElement("font");
- if (!pElement) {
- BS_LOG_ERRORLN("No <font> tag found in \"%s\".", GetFileName().c_str());
- return;
- }
-
- // Font-Tag parsen
- Common::String BitmapFileName;
- if (!_ParseFontTag(*pElement, BitmapFileName, _LineHeight, _GapWidth)) {
- BS_LOG_ERRORLN("An error occurred while parsing <font> tag in \"%s\".", GetFileName().c_str());
- return;
- }
-
- // Absoluten, eindeutigen Pfad zur Bitmapdatei bestimmen und dabei auf vorhandensein prüfen
- {
- // Pointer auf den Package-Manager bekommen
- BS_ASSERT(_pKernel);
- PackageManager *pPackage = static_cast<PackageManager *>(_pKernel->GetService("package"));
- BS_ASSERT(pPackage);
-
- // Absoluten, eindeutigen Pfad bestimmen
- _BitmapFileName = pPackage->GetAbsolutePath(BitmapFileName);
- if (_BitmapFileName == "") {
- BS_LOG_ERRORLN("Image file \"%s\" was specified in <font> tag of \"%s\" but could not be found.",
- _BitmapFileName.c_str(), GetFileName().c_str());
- return;
- }
-
- // Bitmapdatei cachen
- if (!_pKernel->GetResourceManager()->PrecacheResource(_BitmapFileName)) {
- BS_LOG_ERRORLN("Could not precache \"%s\".", _BitmapFileName.c_str());
- return;
- }
- }
-
- // Das Erste Character-Tag finden
- pElement = pElement->FirstChildElement("character");
- if (!pElement) {
- BS_LOG_ERRORLN("No <character> tag found in \"%s\".", GetFileName().c_str());
- return;
- }
-
- // Alle Character-Tags parsen
- while (pElement) {
- int CharCode;
- Common::Rect CharRect;
-
- // Aktuelles Character-Tag parsen
- if (!_ParseCharacterTag(*pElement, CharCode, CharRect)) {
- BS_LOG_ERRORLN("An error occured while parsing a <character> tag in \"%s\".", GetFileName().c_str());
- return;
- }
-
- // Ausgelesene Daten in das _CharacterRects-Array eintragen
- BS_ASSERT(CharCode < 256);
- _CharacterRects[CharCode] = CharRect;
-
- // Zum nächsten Character-Tag iterieren
- pElement = pElement->NextSiblingElement("character");
- }
-
- // Erfolg signalisieren
- _Valid = true;
-}
+ Resource(FileName, Resource::TYPE_FONT),
+ Common::XMLParser() {
-// -----------------------------------------------------------------------------
-
-bool FontResource::_ParseXMLDocument(const Common::String &FileName, TiXmlDocument &Doc) const {
- // Pointer auf den Package-Manager bekommen
+ // Get a pointer to the package manager
BS_ASSERT(_pKernel);
PackageManager *pPackage = static_cast<PackageManager *>(_pKernel->GetService("package"));
BS_ASSERT(pPackage);
- // Die Daten werden zunächst über den Package-Manager gelesen und dann in einen um ein Byte größeren Buffer kopiert
- // und NULL-Terminiert, da TinyXML NULL-Terminierte Daten benötigt.
- unsigned int FileSize;
- char *LoadBuffer = (char *) pPackage->GetFile(GetFileName(), &FileSize);
- if (!LoadBuffer) {
+ // Load the contents of the file
+ unsigned int fileSize;
+ char *xmlData = pPackage->GetXmlFile(GetFileName(), &fileSize);
+ if (!xmlData) {
BS_LOG_ERRORLN("Could not read \"%s\".", GetFileName().c_str());
- return false;
+ return;
}
- // Daten kopieren und NULL-terminieren
- char *WorkBuffer;
- WorkBuffer = (char *)malloc(FileSize + 1);
- memcpy(&WorkBuffer[0], LoadBuffer, FileSize);
- delete LoadBuffer;
- WorkBuffer[FileSize] = '\0';
-
- // Daten parsen
- Doc.Parse(&WorkBuffer[0]);
-
- free(WorkBuffer);
+ // Parse the contents
+ if (!loadBuffer((const byte *)xmlData, fileSize))
+ return;
- return !Doc.Error();
+ _Valid = parse();
+ close();
}
// -----------------------------------------------------------------------------
-bool FontResource::_ParseFontTag(TiXmlElement &Tag, Common::String &BitmapFileName, int &Lineheight, int &GapWidth) const {
- // Bitmap Attribut auslesen
- const char *BitmapString = Tag.Attribute("bitmap");
- if (!BitmapString) {
- BS_LOG_ERRORLN("<font> tag without bitmap attribute occurred in \"%s\".", GetFileName().c_str());
- return false;
- }
- BitmapFileName = BitmapString;
+bool FontResource::parserCallback_font(ParserNode *node) {
+ // Get the attributes of the font
+ Common::String bitmapFilename = node->values["bitmap"];
- // Lineheight Attribut auslesen
- const char *LineheightString = Tag.Attribute("lineheight");
- if (!LineheightString || !BS_String::ToInt(Common::String(LineheightString), Lineheight) || Lineheight < 0) {
+ if (!parseIntegerKey(node->values["lineheight"].c_str(), 1, &_LineHeight)) {
BS_LOG_WARNINGLN("Illegal or missing lineheight attribute in <font> tag in \"%s\". Assuming default (\"%d\").",
GetFileName().c_str(), DEFAULT_LINEHEIGHT);
- Lineheight = DEFAULT_LINEHEIGHT;
+ _LineHeight = DEFAULT_LINEHEIGHT;
}
-
- // Gap Attribut auslesen
- const char *GapString = Tag.Attribute("gap");
- if (!GapString || !BS_String::ToInt(Common::String(GapString), GapWidth) || GapWidth < 0) {
+ if (!parseIntegerKey(node->values["gap"].c_str(), 1, &_GapWidth)) {
BS_LOG_WARNINGLN("Illegal or missing gap attribute in <font> tag in \"%s\". Assuming default (\"%d\").",
GetFileName().c_str(), DEFAULT_GAPWIDTH);
- GapWidth = DEFAULT_GAPWIDTH;
+ _GapWidth = DEFAULT_GAPWIDTH;
+ }
+
+ // Get a reference to the package manager
+ BS_ASSERT(_pKernel);
+ PackageManager *pPackage = static_cast<PackageManager *>(_pKernel->GetService("package"));
+ BS_ASSERT(pPackage);
+
+ // Get the full path and filename for the bitmap resource
+ _BitmapFileName = pPackage->GetAbsolutePath(bitmapFilename);
+ if (_BitmapFileName == "") {
+ BS_LOG_ERRORLN("Image file \"%s\" was specified in <font> tag of \"%s\" but could not be found.",
+ _BitmapFileName.c_str(), GetFileName().c_str());
+ }
+
+ // Pre-cache the resource
+ if (!_pKernel->GetResourceManager()->PrecacheResource(_BitmapFileName)) {
+ BS_LOG_ERRORLN("Could not precache \"%s\".", _BitmapFileName.c_str());
}
return true;
@@ -200,48 +124,28 @@ bool FontResource::_ParseFontTag(TiXmlElement &Tag, Common::String &BitmapFileNa
// -----------------------------------------------------------------------------
-bool FontResource::_ParseCharacterTag(TiXmlElement &Tag, int &Code, Common::Rect &Rect) const {
- // Code Attribut auslesen
- const char *CodeString = Tag.Attribute("code");
- if (!CodeString || !BS_String::ToInt(Common::String(CodeString), Code) || Code < 0 || Code >= 256) {
- BS_LOG_ERRORLN("Illegal or missing code attribute in <character> tag in \"%s\".", GetFileName().c_str());
- return false;
- }
-
- int tmp;
+bool FontResource::parserCallback_character(ParserNode *node) {
+ // Get the attributes of the character
+ int charCode, top, left, right, bottom;
- // Left Attribut auslesen
- const char *LeftString = Tag.Attribute("left");
- if (!LeftString || !BS_String::ToInt(Common::String(LeftString), tmp) || tmp < 0) {
- BS_LOG_ERRORLN("Illegal or missing left attribute in <character> tag in \"%s\".", GetFileName().c_str());
- return false;
+ if (!parseIntegerKey(node->values["code"].c_str(), 1, &charCode) || (charCode < 0) || (charCode >= 256)) {
+ return parserError("Illegal or missing code attribute in <character> tag in \"%s\".", GetFileName().c_str());
}
- Rect.left = tmp;
- // Right Attribut auslesen
- const char *RightString = Tag.Attribute("right");
- if (!RightString || !BS_String::ToInt(RightString, tmp) || tmp < 0) {
- BS_LOG_ERRORLN("Illegal or missing right attribute in <character> tag in \"%s\".", GetFileName().c_str());
- return false;
+ if (!parseIntegerKey(node->values["top"].c_str(), 1, &top) || (top < 0)) {
+ return parserError("Illegal or missing top attribute in <character> tag in \"%s\".", GetFileName().c_str());
}
- Rect.right = tmp;
-
- // Top Attribut auslesen
- const char *TopString = Tag.Attribute("top");
- if (!TopString || !BS_String::ToInt(TopString, tmp) || tmp < 0) {
- BS_LOG_ERRORLN("Illegal or missing top attribute in <character> tag in \"%s\".", GetFileName().c_str());
- return false;
+ if (!parseIntegerKey(node->values["left"].c_str(), 1, &left) || (left < 0)) {
+ return parserError("Illegal or missing left attribute in <character> tag in \"%s\".", GetFileName().c_str());
}
- Rect.top = tmp;
-
- // Bottom Attribut auslesen
- const char *BottomString = Tag.Attribute("bottom");
- if (!BottomString || !BS_String::ToInt(BottomString, tmp) || tmp < 0) {
- BS_LOG_ERRORLN("Illegal or missing bottom attribute in <character> tag in \"%s\".", GetFileName().c_str());
- return false;
+ if (!parseIntegerKey(node->values["right"].c_str(), 1, &right) || (right < 0)) {
+ return parserError("Illegal or missing right attribute in <character> tag in \"%s\".", GetFileName().c_str());
+ }
+ if (!parseIntegerKey(node->values["bottom"].c_str(), 1, &bottom) || (bottom < 0)) {
+ return parserError("Illegal or missing bottom attribute in <character> tag in \"%s\".", GetFileName().c_str());
}
- Rect.bottom = tmp;
+ this->_CharacterRects[charCode] = Common::Rect(left, top, right, bottom);
return true;
}
diff --git a/engines/sword25/gfx/fontresource.h b/engines/sword25/gfx/fontresource.h
index 786405fe11..cfb0251084 100644
--- a/engines/sword25/gfx/fontresource.h
+++ b/engines/sword25/gfx/fontresource.h
@@ -39,12 +39,11 @@
// Includes
// -----------------------------------------------------------------------------
+#include "common/scummsys.h"
+#include "common/rect.h"
+#include "common/xmlparser.h"
#include "sword25/kernel/common.h"
#include "sword25/kernel/resource.h"
-#include "common/rect.h"
-
-class TiXmlDocument;
-class TiXmlElement;
namespace Sword25 {
@@ -58,7 +57,7 @@ class Kernel;
// Klassendefinition
// -----------------------------------------------------------------------------
-class FontResource : public Resource {
+class FontResource : public Resource, Common::XMLParser {
public:
/**
@brief Erzeugt eine neues Exemplar von BS_FontResource
@@ -120,13 +119,34 @@ private:
int _GapWidth;
Common::Rect _CharacterRects[256];
+ // Parser
+ CUSTOM_XML_PARSER(FontResource) {
+ XML_KEY(font)
+ XML_PROP(bitmap, true)
+ XML_PROP(lineheight, false)
+ XML_PROP(gap, false)
+
+ XML_KEY(character)
+ XML_PROP(code, true)
+ XML_PROP(left, true)
+ XML_PROP(top, true)
+ XML_PROP(right, true)
+ XML_PROP(bottom, true)
+ KEY_END()
+ KEY_END()
+ } PARSER_END()
+
+ // Parser callback methods
+ bool parserCallback_font(ParserNode *node);
+ bool parserCallback_character(ParserNode *node);
+
// -----------------------------------------------------------------------------
// Hilfsmethoden
// -----------------------------------------------------------------------------
- bool _ParseXMLDocument(const Common::String &FileName, TiXmlDocument &Doc) const;
- bool _ParseFontTag(TiXmlElement &Tag, Common::String &BitmapFileName, int &LineHeight, int &GapWidth) const;
- bool _ParseCharacterTag(TiXmlElement &Tag, int &Code, Common::Rect &Rect) const;
+// bool _ParseXMLDocument(const Common::String &FileName, TiXmlDocument &Doc) const;
+// bool _ParseFontTag(TiXmlElement &Tag, Common::String &BitmapFileName, int &LineHeight, int &GapWidth) const;
+// bool _ParseCharacterTag(TiXmlElement &Tag, int &Code, Common::Rect &Rect) const;
};
} // End of namespace Sword25