Changeset 90 for trunk/src

Timestamp:

06/02/13 22:33:59 (12 years ago)

Author:

epyon

Message:

• common - windows SCL warnings proper suppression
• common - NV_DEPRECATED added
• interface/image_data added
• image constructible from image_data by ownership transfer
• device - create_image_data added (temporary)
• gl_device - create_image_data implementation (temporary)

Location:

trunk/src

Files:

• gfx/image.cc (modified) (4 diffs)
• gl/gl_device.cc (modified) (2 diffs)

trunk/src/gfx/image.cc

@@ -5 +5 @@
 #include "nv/gfx/image.hh"
 
-#include <cstring>
+#include <algorithm>
 
 using namespace nv;
@@ -15 +15 @@
 }
 
+image::image( image_data* data )
+        : m_size( data->get_size() ), m_depth( data->get_depth() ), m_data( data->release_data() )
+{
+        NV_ASSERT( m_data, "image created from empty image_data!" );
+}
+
+
 image::image( glm::ivec2 size, size_t depth, const uint8 * data, bool reversed )
         : m_size( size ), m_depth( depth ), m_data( nullptr )
 {
+        std::size_t bsize = m_size.x * m_size.y * m_depth;
         m_data = new uint8[ m_size.x * m_size.y * m_depth ];
 
         if ( reversed )
         {
-                for( int i = 0; i < size.y; ++i )
+                std::size_t bline = m_size.x * m_depth;
+                for( int i = 0; i < m_size.y; ++i )
                 {
-                        memcpy( m_data + size.x * ( size.y - i - 1) * m_depth, data + i * size.x * m_depth, m_size.x * m_depth );
+                        std::copy( data + i * bline, data + (i + 1) * bline, m_data + bsize - ( i + 1 ) * bline );
                 }
 
@@ -30 +39 @@
         else 
         {
-                memcpy( m_data, data, m_size.x * m_size.y * m_depth );
+                std::copy( data, data + bsize, m_data );
         }
 }
@@ -36 +45 @@
 void image::fill( uint8 value )
 {
-        memset( m_data, value, m_size.x * m_size.y * m_depth );
+        std::fill( m_data, m_data + m_size.x * m_size.y * m_depth, value );
 }
 
 void image::set_region( region r, const uint8 * data, size_t stride )
 {
-        if ( stride == 0 ) stride = r.size.x;
+        if ( stride == 0 ) stride = r.size.x * m_depth;
+        
+        std::size_t bpos  = (r.pos.y*m_size.x + r.pos.x ) * m_depth;
+        std::size_t bline = m_size.x*m_depth;
 
         for( int i = 0; i < r.size.y; ++i )
         {
-                memcpy( m_data+((r.pos.y+i)*m_size.x + r.pos.x ) * m_depth, 
-                        data + (i*stride), r.size.x * m_depth );
+// TODO: test if same as old:
+//              memcpy( m_data+((r.pos.y+i)*m_size.x + r.pos.x ) * m_depth, 
+//                      data + (i*stride), r.size.x * m_depth );
+                std::copy( data + i*stride, data + (i+1)*stride, m_data + bpos + bline * i );
         }
 }

trunk/src/gl/gl_device.cc

@@ -11 +11 @@
 #include "nv/logging.hh"
 #include "nv/lib/sdl12.hh"
+#include "nv/lib/sdl_image.hh"
 
 using namespace nv;
@@ -67 +68 @@
 }
 
+// this is a temporary function that will be removed once we find a way to 
+// pass binary file data around
+image_data* nv::gl_device::create_image_data( const std::string& filename )
+{
+        load_sdl_image_library();
+        SDL_Surface* image = IMG_Load( filename.c_str() );
+        image_data* data = new image_data( glm::ivec2( image->w, image->h ), image->format->BytesPerPixel, (nv::uint8*)image->pixels );
+        return data;
+}
+
 texture2d* gl_device::create_texture2d( ivec2 size, image_format aformat, datatype adatatype, sampler asampler, void* data /*= nullptr */ )
 {