Rework exceptions

[libs/gl.git] / source / texture3d.cpp

diff --git a/source/texture3d.cpp b/source/texture3d.cpp
index e716c901b53f408820123130dec62fc3db109eea..4aac49edfaf82f27eaf4c3eef8c7021596d5033e 100644 (file)
--- a/source/texture3d.cpp
+++ b/source/texture3d.cpp
@@ -1,7 +1,7 @@
 #include <cmath>
 #include <msp/graphics/image.h>
 #include "bindable.h"
-#include "except.h"
+#include "error.h"
 #include "extension.h"
 #include "texture3d.h"
 #include "version_1_2.h"
@@ -24,9 +24,9 @@ Texture3D::Texture3D():
 void Texture3D::storage(PixelFormat f, unsigned w, unsigned h, unsigned d)
 {
        if(width>0)
-               throw InvalidState("Textures storage may only be specified once");
+               throw invalid_operation("Texture3D::storage");
        if(w==0 || h==0 || d==0)
-               throw InvalidParameterValue("Invalid texture dimensions");
+               throw invalid_argument("Texture3D::storage");
 
        width = w;
        height = h;
@@ -44,7 +44,8 @@ void Texture3D::allocate(unsigned level)
 
 void Texture3D::image(unsigned level, PixelFormat fmt, DataType type, const void *data)
 {
-       require_storage();
+       if(width==0 || height==0 || depth==0)
+               throw invalid_operation("Texture3D::image");
 
        unsigned w = width;
        unsigned h = height;
@@ -64,7 +65,9 @@ void Texture3D::image(unsigned level, PixelFormat fmt, DataType type, const void
 
 void Texture3D::sub_image(unsigned level, int x, int y, int z, unsigned wd, unsigned ht, unsigned dp, PixelFormat fmt, DataType type, const void *data)
 {
-       require_storage();
+       if(width==0 || height==0 || depth==0)
+               throw invalid_operation("Texture3D::image");
+
        allocate(level);
 
        Bind _bind(this, true);
@@ -83,7 +86,7 @@ void Texture3D::load_image(const string &fn, int dp)
        if(dp==-1)
        {
                if(h%w)
-                       throw IncompatibleData("Image height is not divisible by its width");
+                       throw incompatible_data("Texture3D::load_image");
                d = h/w;
                h = w;
        }
@@ -92,7 +95,7 @@ void Texture3D::load_image(const string &fn, int dp)
                for(d=h; d*d>h; d>>=2) ;
                for(; d*d<h; ++d) ;
                if(d*d!=h)
-                       throw IncompatibleData("Could not find a square root of texture height");
+                       throw incompatible_data("Texture3D::load_image");
                h = d;
        }
        else if(dp>0)
@@ -102,17 +105,11 @@ void Texture3D::load_image(const string &fn, int dp)
        if(width==0)
                storage(fmt, w, h, d);
        else if(w!=width || h!=height || d!=depth)
-               throw IncompatibleData("Image does not match texture storage");
+               throw incompatible_data("Texture3D::load_image");
 
        image(0, fmt, UNSIGNED_INT, img.get_data());
 }
 
-void Texture3D::require_storage()
-{
-       if(width==0 || height==0 || depth==0)
-               throw InvalidState("Texture storage has not been specified");
-}
-
 void Texture3D::get_level_size(unsigned level, unsigned &w, unsigned &h, unsigned &d)
 {
        w >>= level;