Check the flat qualifier from the correct member

[libs/gl.git] / source / core / sampler.cpp

diff --git a/source/core/sampler.cpp b/source/core/sampler.cpp
index bb37b0747e7d12c732ac4e57234c6b8bc3836344..4da13833330dac12eace04b145f9ffacd4a0510b 100644 (file)
--- a/source/core/sampler.cpp
+++ b/source/core/sampler.cpp
@@ -1,73 +1,31 @@
-#include <msp/gl/extensions/arb_direct_state_access.h>
-#include <msp/gl/extensions/arb_sampler_objects.h>
-#include <msp/gl/extensions/arb_shadow.h>
-#include <msp/gl/extensions/ext_texture_filter_anisotropic.h>
-#include <msp/gl/extensions/ext_texture3d.h>
-#include <msp/gl/extensions/khr_debug.h>
 #include <msp/strings/format.h>
+#include "device.h"
 #include "error.h"
 #include "sampler.h"
-#include "texture.h"
-#include "texunit.h"
 
 using namespace std;
 
 namespace Msp {
 namespace GL {
 
-Sampler::Sampler():
-       min_filter(NEAREST_MIPMAP_LINEAR),
-       mag_filter(LINEAR),
-       max_anisotropy(1.0f),
-       wrap_s(REPEAT),
-       wrap_t(REPEAT),
-       wrap_r(REPEAT),
-       border_color(Color(0.0f, 0.0f, 0.0f, 0.0f)),
-       compare(false),
-       cmp_func(LEQUAL),
-       dirty_params(0)
+void Sampler::update() const
 {
-       Require _req(ARB_sampler_objects);
-       if(ARB_direct_state_access)
-               glCreateSamplers(1, &id);
-       else
-               glGenSamplers(1, &id);
-}
-
-void Sampler::update_parameter(int mask) const
-{
-       if(mask&MIN_FILTER)
-               glSamplerParameteri(id, GL_TEXTURE_MIN_FILTER, min_filter);
-       if(mask&MAG_FILTER)
-               glSamplerParameteri(id, GL_TEXTURE_MAG_FILTER, mag_filter);
-       if(mask&MAX_ANISOTROPY)
-               glSamplerParameterf(id, GL_TEXTURE_MAX_ANISOTROPY_EXT, max_anisotropy);
-       if(mask&WRAP_S)
-               glSamplerParameteri(id, GL_TEXTURE_WRAP_S, wrap_s);
-       if(mask&WRAP_T)
-               glSamplerParameteri(id, GL_TEXTURE_WRAP_T, wrap_t);
-       if(mask&WRAP_R)
-               glSamplerParameteri(id, GL_TEXTURE_WRAP_R, wrap_r);
-       if(mask&BORDER_COLOR)
-               glSamplerParameterfv(id, GL_TEXTURE_BORDER_COLOR, &border_color.r);
-       if(mask&COMPARE)
-       {
-               glSamplerParameteri(id, GL_TEXTURE_COMPARE_MODE, (compare ? GL_COMPARE_R_TO_TEXTURE : GL_NONE));
-               if(compare)
-                       glSamplerParameteri(id, GL_TEXTURE_COMPARE_FUNC, cmp_func);
-       }
+       SamplerBackend::update(dirty_params);
+       dirty_params = 0;
 }
 
 void Sampler::set_min_filter(TextureFilter f)
 {
        min_filter = f;
-       update_parameter(MIN_FILTER);
+       dirty_params |= MIN_FILTER;
 }
 
 void Sampler::set_mag_filter(TextureFilter f)
 {
+       if(is_mipmapped(f))
+               throw invalid_argument("Sampler::set_mag_filter");
        mag_filter = f;
-       update_parameter(MAG_FILTER);
+       dirty_params |= MAG_FILTER;
 }
 
 void Sampler::set_filter(TextureFilter f)
@@ -80,98 +38,58 @@ void Sampler::set_max_anisotropy(float a)
 {
        if(a<1.0f)
                throw invalid_argument("Sampler::set_max_anisotropy");
-       else if(a>1.0f)
-               static Require _req(EXT_texture_filter_anisotropic);
+       if(a>Device::get_current().get_info().limits.max_anisotropy)
+               throw out_of_range("Sampler::set_max_anisotropy");
+       bool supported = check_anisotropic(a>1.0f);
        max_anisotropy = a;
-       if(EXT_texture_filter_anisotropic)
-               update_parameter(MAX_ANISOTROPY);
-}
-
-void Sampler::set_wrap(TextureWrap w)
-{
-       set_wrap_s(w);
-       set_wrap_t(w);
-       if(EXT_texture3D)
-               set_wrap_r(w);
+       if(supported)
+               dirty_params |= MAX_ANISOTROPY;
 }
 
 void Sampler::set_wrap_s(TextureWrap w)
 {
        wrap_s = w;
-       update_parameter(WRAP_S);
+       dirty_params |= WRAP_S;
 }
 
 void Sampler::set_wrap_t(TextureWrap w)
 {
        wrap_t = w;
-       update_parameter(WRAP_T);
+       dirty_params |= WRAP_T;
 }
 
 void Sampler::set_wrap_r(TextureWrap w)
 {
-       static Require _req(EXT_texture3D);
        wrap_r = w;
-       update_parameter(WRAP_R);
+       dirty_params |= WRAP_R;
+}
+
+void Sampler::set_wrap(TextureWrap w)
+{
+       set_wrap_s(w);
+       set_wrap_t(w);
+       set_wrap_r(w);
 }
 
 void Sampler::set_border_color(const Color &c)
 {
        border_color = c;
-       update_parameter(BORDER_COLOR);
+       dirty_params |= BORDER_COLOR;
 }
 
 void Sampler::disable_compare()
 {
        compare = false;
-       update_parameter(COMPARE);
+       dirty_params |= COMPARE;
 }
 
 void Sampler::set_compare(Predicate f)
 {
-       static Require _req(ARB_shadow);
        compare = true;
        cmp_func = f;
        dirty_params |= COMPARE;
 }
 
-void Sampler::bind_to(unsigned i) const
-{
-       TexUnit &unit = TexUnit::get_unit(i);
-
-       if(unit.set_sampler(this))
-       {
-               glBindSampler(i, id);
-
-               if(dirty_params)
-               {
-                       update_parameter(dirty_params);
-                       dirty_params = 0;
-               }
-       }
-}
-
-const Sampler *Sampler::current(unsigned i)
-{
-       return TexUnit::get_unit(i).get_sampler();
-}
-
-void Sampler::unbind_from(unsigned i)
-{
-       TexUnit &unit = TexUnit::get_unit(i);
-       if(unit.set_sampler(0))
-               glBindSampler(i, 0);
-}
-
-void Sampler::set_debug_name(const string &name)
-{
-#ifdef DEBUG
-       if(id && KHR_debug)
-               glObjectLabel(GL_SAMPLER, id, name.size(), name.c_str());
-#else
-       (void)name;
-#endif
-}
-
 
 Sampler::Loader::Loader(Sampler &s):
        DataFile::ObjectLoader<Sampler>(s)