Changeset 301 for trunk/src

Timestamp:

08/07/14 12:11:16 (11 years ago)

Author:

epyon

Message:

• textures are now handled by lightweight handles
• textures now need to be manually released via context
• removed all old texture2d functionality
• unreleased textures will be auto-released
• textures are properly tracked via entity system
• detailed stats and checking now possible

Location:

trunk/src

Files:

• gl/gl_context.cc (modified) (3 diffs)
• gl/gl_device.cc (modified) (3 diffs)
• gl/gl_texture2d.cc (deleted)
• gui/gui_renderer.cc (modified) (6 diffs)

trunk/src/gl/gl_context.cc

@@ -8 +8 @@
 #include "nv/lib/gl.hh"
 #include "nv/lib/sdl.hh"
-#include "nv/gl/gl_texture2d.hh"
+#include "nv/gl/gl_device.hh"
 #include "nv/gl/gl_program.hh"
 #include "nv/gl/gl_vertex_buffer.hh"
@@ -14 +14 @@
 using namespace nv;
 
-void gl_context::bind( texture2d* texture, texture_slot slot )
-{
-        GLuint id = static_cast< gl_texture2d* >( texture )->glid;
-        glActiveTexture( GL_TEXTURE0 + static_cast< GLenum >( slot ) );
-        glBindTexture( GL_TEXTURE_2D, id );
+void gl_context::bind( texture t, texture_slot slot )
+{
+        const gl_texture_info* info = static_cast< const gl_texture_info* >( m_device->get_texture_info( t ) );
+        if ( info )
+        {
+                glActiveTexture( GL_TEXTURE0 + static_cast< GLenum >( slot ) );
+                glBindTexture( GL_TEXTURE_2D, info->glid );
+        }
 }
 
@@ -94 +97 @@
 }
 
-void gl_context::update( texture2d* texture, void* data )
-{
-        GLuint id = static_cast< gl_texture2d* >( texture )->glid;
-        image_format format = texture->get_format();
-        ivec2        size   = texture->get_size();
-
-        glBindTexture( GL_TEXTURE_2D, id );
-        glTexImage2D( GL_TEXTURE_2D, 0, (GLint)nv::image_format_to_enum(format.format), size.x, size.y, 0, nv::image_format_to_enum(format.format), nv::datatype_to_gl_enum(format.type), data );
+void nv::gl_context::update( texture t, void* data )
+{
+        const gl_texture_info* info = static_cast< const gl_texture_info* >( m_device->get_texture_info( t ) );
+        if ( info )
+        {
+                image_format format = info->format;
+                ivec2        size   = info->size;
+
+                glBindTexture( GL_TEXTURE_2D, info->glid );
+                glTexImage2D( GL_TEXTURE_2D, 0, (GLint)nv::image_format_to_enum(format.format), size.x, size.y, 0, nv::image_format_to_enum(format.format), nv::datatype_to_gl_enum(format.type), data );
+        }
 }
 

trunk/src/gl/gl_device.cc

@@ -8 +8 @@
 #include "nv/gl/gl_program.hh"
 #include "nv/gl/gl_vertex_buffer.hh"
-#include "nv/gl/gl_texture2d.hh"
 #include "nv/logging.hh"
 #include "nv/lib/sdl.hh"
 #include "nv/lib/sdl_image.hh"
+#include "nv/gl/gl_enum.hh"
+#include "nv/lib/gl.hh"
 
 using namespace nv;
@@ -88 +89 @@
 }
 
-texture2d* gl_device::create_texture2d( ivec2 size, pixel_format aformat, datatype adatatype, sampler asampler, void* data /*= nullptr */ )
-{
-        return new gl_texture2d( size, aformat, adatatype, asampler, data );
-}
-
 uint32 gl_device::get_ticks()
 {
@@ -105 +101 @@
 gl_device::~gl_device()
 {
+        // TODO: better use release_texture
+        for ( auto& t : m_textures ) glDeleteTextures( 1, &t.glid );
+
         SDL_Quit();
 }
+
+nv::texture nv::gl_device::create_texture( ivec2 size, image_format aformat, sampler asampler, void* data /*= nullptr */ )
+{
+        unsigned glid = 0;
+        glGenTextures( 1, &glid );
+
+        glBindTexture( GL_TEXTURE_2D, glid );
+
+        // Detect if mipmapping was requested
+        if (( asampler.filter_min != sampler::LINEAR && asampler.filter_min != sampler::NEAREST ) ||
+                ( asampler.filter_max != sampler::LINEAR && asampler.filter_max != sampler::NEAREST ))
+        {
+                glTexParameteri(GL_TEXTURE_2D, GL_GENERATE_MIPMAP, GL_TRUE);
+        }
+
+        glTexParameteri( GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, (int)nv::sampler_filter_to_enum( asampler.filter_min ) );
+        glTexParameteri( GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, (int)nv::sampler_filter_to_enum( asampler.filter_max ) );
+        glTexParameteri( GL_TEXTURE_2D, GL_TEXTURE_WRAP_S, (int)nv::sampler_wrap_to_enum( asampler.wrap_s) );
+        glTexParameteri( GL_TEXTURE_2D, GL_TEXTURE_WRAP_T, (int)nv::sampler_wrap_to_enum( asampler.wrap_t) );
+
+        if (data)
+        {
+                glTexImage2D( GL_TEXTURE_2D, 0, (GLint)nv::image_format_to_enum(aformat.format), size.x, size.y, 0, nv::image_format_to_enum(aformat.format), nv::datatype_to_gl_enum(aformat.type), data );
+        }
+
+        glBindTexture( GL_TEXTURE_2D, 0 );
+
+        texture result = m_textures.create();
+        gl_texture_info* info = m_textures.get( result );
+        info->format  = aformat;
+        info->sampler = asampler;
+        info->size    = size;
+        info->glid    = glid;
+        return result;
+}
+
+void nv::gl_device::release_texture( texture t )
+{
+        gl_texture_info* info = m_textures.get( t );
+        if ( info )
+        {
+                glDeleteTextures( 1, &(info->glid) );
+                m_textures.destroy( t );
+        }
+}
+
+const texture_info* nv::gl_device::get_texture_info( texture t )
+{
+        return m_textures.get( t );
+}
+

trunk/src/gui/gui_renderer.cc

@@ -65 +65 @@
 {
 public:
-        screen_render_data( context* ctx, size_t initial_size )
-                : buffer( ctx, nv::DYNAMIC_DRAW, initial_size ), varray( nullptr ), shader(nullptr), texture(nullptr) 
+        screen_render_data( context* actx, size_t initial_size )
+                : buffer( actx, nv::DYNAMIC_DRAW, initial_size ), varray( nullptr ), shader(nullptr)
         {
 
@@ -74 +74 @@
                 delete shader; 
                 delete varray; 
-                delete texture;
         }
 
         nv::sliced_buffer<gui_quad> buffer;
+        nv::texture       tex;
         nv::vertex_array* varray;
         nv::program*      shader;
-        nv::texture2d*    texture;
 };
 
@@ -121 +120 @@
 
         nv::sampler sampler( nv::sampler::LINEAR, nv::sampler::CLAMP_TO_EDGE );
-        sr->texture = m_window->get_device()->create_texture2d( m_atlas.get_size(), nv::RGBA, nv::UBYTE, sampler, nullptr );
+        sr->tex = m_window->get_device()->create_texture( m_atlas.get_size(), image_format( nv::RGBA, nv::UBYTE ), sampler, nullptr );
 
         m_render_state.depth_test.enabled = false;
@@ -262 +261 @@
         if ( m_reupload )
         {
-                m_context->update( sr->texture, (void*)m_atlas.get_data() );
+                m_context->update( sr->tex, (void*)m_atlas.get_data() );
                 m_reupload = false;
         }
@@ -273 +272 @@
                 sr->varray->update_vertex_buffer( nv::slot::COLOR,    vb, false );
         }
-        m_context->bind( sr->texture, TEX_DIFFUSE );
+        m_context->bind( sr->tex, TEX_DIFFUSE );
         m_context->draw( TRIANGLES, m_render_state, m_scene_state, sr->shader, sr->varray, sr->buffer.get_size() * 6 );
 }
@@ -283 +282 @@
                 delete p;
         }
-        delete m_render_data;
-}
+        if ( m_render_data )
+        {
+                m_context->get_device()->release_texture( ((screen_render_data*)m_render_data)->tex );
+                delete m_render_data;
+        }
+}