01/24/12 15:34:02 (6 years ago)

Added support for using GL_UNPACK_ROW_LENGTH in conjunction with texture's + osg::Image via new RowLength?
parameter in osg::Image. To support this Image::setData(..) now has a new optional rowLength parameter which
defaults to 0, which provides the original behaviour, Image::setRowLength(int) and int Image::getRowLength() are also provided.

With the introduction of RowLength? support in osg::Image it is now possible to create a sub image where
the t size of the image are smaller than the row length, useful for when you have a large image on the CPU
and which to use a small portion of it on the GPU. However, when these sub images are created the data
within the image is no longer contiguous so data access can no longer assume that all the data is in
one block. The new method Image::isDataContiguous() enables the user to check whether the data is contiguous,
and if not one can either access the data row by row using Image::data(column,row,image) accessor, or use the
new Image::DataIterator? for stepping through each block on memory assocatied with the image.

To support the possibility of non contiguous osg::Image usage of image objects has had to be updated to
check DataContiguous? and handle the case or use access via the DataIerator? or by row by row. To achieve
this a relatively large number of files has had to be modified, in particular the texture classes and
image plugins that doing writing.

1 modified


  • OpenSceneGraph/trunk/src/osgPlugins/tga/ReaderWriterTGA.cpp

    r12574 r12912  
    546546        bool saveTGAStream(const osg::Image& image, std::ostream& fout) const 
    547547        { 
     548            if (! return false; 
    548550            // At present, I will only save the image to unmapped RGB format 
    549551            // Other data types can be added soon with different options 
    554556            int numPerPixel = image.computeNumComponents(pixelFormat); 
    555557            int pixelMultiplier = (image.getDataType()==GL_FLOAT ? 255 : 1); 
    556             const unsigned char* data =; 
    557             if ( !data ) return false; 
    559559            // Headers 
    582582            for (int y=0; y<height; ++y) 
    583583            { 
    584                 const unsigned char* ptr = data + y * width * numPerPixel; 
     584                const unsigned char* ptr =,y); 
    585585                for (int x=0; x<width; ++x) 
    586586                {