X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fgraphics%2Fdevil%2Fdevilloader.cpp;h=e837154ecc3ed4730ba4d469a7023c8ce0724418;hb=b74b0945b2e19c32f05d8bcb016cc9434c99979d;hp=88beb04802d47025c2995d886d227d3d6dc3d78d;hpb=21b0c47700f1b570b6129384ce0b96a0c6b48ffd;p=libs%2Fgui.git

diff --git a/source/graphics/devil/devilloader.cpp b/source/graphics/devil/devilloader.cpp
index 88beb04..e837154 100644
--- a/source/graphics/devil/devilloader.cpp
+++ b/source/graphics/devil/devilloader.cpp
@@ -1,4 +1,5 @@
 #include <algorithm>
+#include <msp/strings/format.h>
 #include <IL/il.h>
 #include "devilloader.h"
 
@@ -53,14 +54,26 @@ int tell(void *handle)
 	return reinterpret_cast<Msp::IO::Seekable *>(handle)->tell();
 }
 
+std::string error_string(ILenum err)
+{
+	switch(err)
+	{
+	case IL_FORMAT_NOT_SUPPORTED: return "Format not supported";
+	case IL_INTERNAL_ERROR: return "DevIL internal error";
+	case IL_INVALID_FILE_HEADER: return "Invalid file header";
+	case IL_FILE_READ_ERROR: return "File read error";
+	case IL_LIB_PNG_ERROR: return "LibPNG error";
+	case IL_LIB_JPEG_ERROR: return "LibJPEG error";
+	default: return Msp::format("Unknown error (%04X)", err);
+	}
+}
+
 } // namespace
 
 
 namespace Msp {
 namespace Graphics {
 
-ImageLoader::Register<DevilLoader> DevilLoader::reg;
-
 DevilLoader::DevilLoader(IO::Seekable &i):
 	io(i)
 {
@@ -92,7 +105,10 @@ void DevilLoader::load(Image::Data &data)
 {
 	ilSetRead(0, 0, eof, get, read, seek, tell);
 	ilBindImage(id);
-	ilLoadF(IL_TYPE_UNKNOWN, &io);
+
+	ilGetError();
+	if(!ilLoadF(IL_TYPE_UNKNOWN, &io))
+		throw bad_image_data(error_string(ilGetError()));
 
 	switch(ilGetInteger(IL_IMAGE_FORMAT))
 	{
@@ -108,10 +124,11 @@ void DevilLoader::load(Image::Data &data)
 
 	data.width = ilGetInteger(IL_IMAGE_WIDTH);
 	data.height = ilGetInteger(IL_IMAGE_HEIGHT);
-	unsigned data_size = data.width*data.height*ilGetInteger(IL_IMAGE_BYTES_PER_PIXEL);
-	data.data = new char[data_size];
+	data.stride = data.width*ilGetInteger(IL_IMAGE_BYTES_PER_PIXEL);
+	unsigned data_size = data.stride*data.height;
+	data.pixels = new char[data_size];
 	ILubyte *il_data = ilGetData();
-	copy(il_data, il_data+data_size, data.data);
+	copy(il_data, il_data+data_size, data.pixels);
 
 	ilBindImage(0);
 	ilResetRead();