New iteration of check 'imageSizeInBytes':

Premises:
    See http://www.glprogramming.com/red/chapter08.html

    "If the rectangle in memory is larger than the subrectangle that's being drawn or read,
    you need to specify the actual length (measured in pixels) of the larger rectangle with *ROW_LENGTH. "

    This equals ROW_LENGTH == SubPicture-Width + 'Dirt', ie the offset to go from one line to the next.

    "You also need to specify the number of rows and pixels to skip before starting to copy the data for the subrectangle.
    These numbers are set using the parameters *SKIP_ROWS and *SKIP_PIXELS, as shown in Figure 8-9.
    By default, both parameters are 0, so you start at the lower-left corner."

    This equals to a one time offset _before_ you start copy the data, ie with your DMA engine,
    using ROW_LENGTH and HEIGHT only.

Compared to Mesa3D's implementation (proof):

    http://cgit.freedesktop.org/mesa/mesa/tree/src/glx/x11/pixelstore.c
    Line 78 - 200
      PixelStore State Tracker: Store the values in the PixelStore structure.

    http://cgit.freedesktop.org/mesa/mesa/tree/src/glx/x11/pixel.c
    Line 82 - 155
      Line 210: Start position is set one time using SKIP_* values the same way we do.
      Line 228: Jump to the first pixel in the newxt row, just using ROW_LENGTH
      Line 230: Jump to the first pixel in the next 'image' just using imageSize,

            rowsPerImage = IMAGE_HEIGHT>0?IMAGE_HEIGHT:height;

            groupsPerRow = ROW_LENGTH>0?ROW_LENGTH:width;
            groupSize = elementSize * components;
            rowSize = groupsPerRow * groupSize;

            imageSize = rowSize * rowsPerImage;

changes:
    Removed redundand 'dimension' parameted, which is derived from depth and height,
        1D == depth:1 height:1
        2D == depth:1 height>1
        nD == depth>1 height>1

    Safe fail depth to >= 1
    Safe fail height to >= 1D

    Take either the ROW_LENGTH / IMAGE_HEIGHT PixelStore value if > 0,
    or the given width / height.

0 files changed, 0 insertions, 0 deletions