Commit 87d3fe04 by Geoff Lang

Revert "In TextureD3D_2D, don't call Image::redefine directly."

Warnings in 64-bit. This reverts commit a0633a7e. Change-Id: I5b3f31b261b947cd295e9f4a9abbbdbc00740183 Reviewed-on: https://chromium-review.googlesource.com/293841Reviewed-by: 's avatarGeoff Lang <geofflang@chromium.org> Tested-by: 's avatarGeoff Lang <geofflang@chromium.org>
parent a0633a7e
...@@ -707,7 +707,7 @@ gl::Error TextureD3D_2D::setImage(GLenum target, ...@@ -707,7 +707,7 @@ gl::Error TextureD3D_2D::setImage(GLenum target,
bool fastUnpacked = false; bool fastUnpacked = false;
GLint level = static_cast<GLint>(imageLevel); GLint level = static_cast<GLint>(imageLevel);
redefineImage(level, sizedInternalFormat, size, false); redefineImage(level, sizedInternalFormat, size);
gl::ImageIndex index = gl::ImageIndex::Make2D(level); gl::ImageIndex index = gl::ImageIndex::Make2D(level);
...@@ -797,7 +797,7 @@ gl::Error TextureD3D_2D::setCompressedImage(GLenum target, ...@@ -797,7 +797,7 @@ gl::Error TextureD3D_2D::setCompressedImage(GLenum target,
GLint level = static_cast<GLint>(imageLevel); GLint level = static_cast<GLint>(imageLevel);
// compressed formats don't have separate sized internal formats-- we can just use the compressed format directly // compressed formats don't have separate sized internal formats-- we can just use the compressed format directly
redefineImage(level, internalFormat, size, false); redefineImage(level, internalFormat, size);
return TextureD3D::setCompressedImage(gl::ImageIndex::Make2D(level), unpack, pixels, 0); return TextureD3D::setCompressedImage(gl::ImageIndex::Make2D(level), unpack, pixels, 0);
} }
...@@ -827,8 +827,7 @@ gl::Error TextureD3D_2D::copyImage(GLenum target, ...@@ -827,8 +827,7 @@ gl::Error TextureD3D_2D::copyImage(GLenum target,
GLint level = static_cast<GLint>(imageLevel); GLint level = static_cast<GLint>(imageLevel);
GLenum sizedInternalFormat = gl::GetSizedInternalFormat(internalFormat, GL_UNSIGNED_BYTE); GLenum sizedInternalFormat = gl::GetSizedInternalFormat(internalFormat, GL_UNSIGNED_BYTE);
redefineImage(level, sizedInternalFormat, gl::Extents(sourceArea.width, sourceArea.height, 1), redefineImage(level, sizedInternalFormat, gl::Extents(sourceArea.width, sourceArea.height, 1));
false);
gl::ImageIndex index = gl::ImageIndex::Make2D(level); gl::ImageIndex index = gl::ImageIndex::Make2D(level);
gl::Offset destOffset(0, 0, 0); gl::Offset destOffset(0, 0, 0);
...@@ -932,12 +931,12 @@ gl::Error TextureD3D_2D::setStorage(GLenum target, size_t levels, GLenum interna ...@@ -932,12 +931,12 @@ gl::Error TextureD3D_2D::setStorage(GLenum target, size_t levels, GLenum interna
gl::Extents levelSize(std::max(1, size.width >> level), gl::Extents levelSize(std::max(1, size.width >> level),
std::max(1, size.height >> level), std::max(1, size.height >> level),
1); 1);
redefineImage(level, internalFormat, levelSize, true); mImageArray[level]->redefine(GL_TEXTURE_2D, internalFormat, levelSize, true);
} }
for (size_t level = levels; level < gl::IMPLEMENTATION_MAX_TEXTURE_LEVELS; level++) for (size_t level = levels; level < gl::IMPLEMENTATION_MAX_TEXTURE_LEVELS; level++)
{ {
redefineImage(level, GL_NONE, gl::Extents(0, 0, 1), true); mImageArray[level]->redefine(GL_TEXTURE_2D, GL_NONE, gl::Extents(0, 0, 0), true);
} }
// TODO(geofflang): Verify storage creation had no errors // TODO(geofflang): Verify storage creation had no errors
...@@ -969,7 +968,7 @@ void TextureD3D_2D::bindTexImage(egl::Surface *surface) ...@@ -969,7 +968,7 @@ void TextureD3D_2D::bindTexImage(egl::Surface *surface)
GLenum internalformat = surface->getConfig()->renderTargetFormat; GLenum internalformat = surface->getConfig()->renderTargetFormat;
gl::Extents size(surface->getWidth(), surface->getHeight(), 1); gl::Extents size(surface->getWidth(), surface->getHeight(), 1);
redefineImage(0, internalformat, size, true); mImageArray[0]->redefine(GL_TEXTURE_2D, internalformat, size, true);
if (mTexStorage) if (mTexStorage)
{ {
...@@ -1013,7 +1012,7 @@ void TextureD3D_2D::initMipmapsImages() ...@@ -1013,7 +1012,7 @@ void TextureD3D_2D::initMipmapsImages()
std::max(getBaseLevelHeight() >> level, 1), std::max(getBaseLevelHeight() >> level, 1),
1); 1);
redefineImage(level, getBaseLevelInternalFormat(), levelSize, false); redefineImage(level, getBaseLevelInternalFormat(), levelSize);
} }
} }
...@@ -1223,10 +1222,7 @@ gl::Error TextureD3D_2D::updateStorageLevel(int level) ...@@ -1223,10 +1222,7 @@ gl::Error TextureD3D_2D::updateStorageLevel(int level)
return gl::Error(GL_NO_ERROR); return gl::Error(GL_NO_ERROR);
} }
void TextureD3D_2D::redefineImage(GLint level, void TextureD3D_2D::redefineImage(GLint level, GLenum internalformat, const gl::Extents &size)
GLenum internalformat,
const gl::Extents &size,
bool forceRelease)
{ {
ASSERT(size.depth == 1); ASSERT(size.depth == 1);
...@@ -1235,7 +1231,7 @@ void TextureD3D_2D::redefineImage(GLint level, ...@@ -1235,7 +1231,7 @@ void TextureD3D_2D::redefineImage(GLint level,
const int storageHeight = std::max(1, getBaseLevelHeight() >> level); const int storageHeight = std::max(1, getBaseLevelHeight() >> level);
const GLenum storageFormat = getBaseLevelInternalFormat(); const GLenum storageFormat = getBaseLevelInternalFormat();
mImageArray[level]->redefine(GL_TEXTURE_2D, internalformat, size, forceRelease); mImageArray[level]->redefine(GL_TEXTURE_2D, internalformat, size, false);
if (mTexStorage) if (mTexStorage)
{ {
......
...@@ -169,10 +169,7 @@ class TextureD3D_2D : public TextureD3D ...@@ -169,10 +169,7 @@ class TextureD3D_2D : public TextureD3D
gl::Error updateStorageLevel(int level); gl::Error updateStorageLevel(int level);
void redefineImage(GLint level, void redefineImage(GLint level, GLenum internalformat, const gl::Extents &size);
GLenum internalformat,
const gl::Extents &size,
bool forceRelease);
ImageD3D *mImageArray[gl::IMPLEMENTATION_MAX_TEXTURE_LEVELS]; ImageD3D *mImageArray[gl::IMPLEMENTATION_MAX_TEXTURE_LEVELS];
}; };
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment