Get rid of "ATIClampBug", it's not a bug, it's just lack of full non-pow-2 texture support

This commit is contained in:
Henrik Rydgard 2014-01-02 00:09:01 +01:00
parent 845235e4f6
commit 91531a9792
3 changed files with 6 additions and 6 deletions

View file

@ -1089,7 +1089,7 @@ void FramebufferManager::ReadFramebufferToMemory(VirtualFramebuffer *vfb, bool s
#ifdef USING_GLES2
PackFramebufferSync_(nvfb); // synchronous glReadPixels
#else
if (gl_extensions.PBO_ARB || !gl_extensions.ATIClampBug) {
if (gl_extensions.PBO_ARB && gl_extensions.OES_texture_npot) {
if (!sync) {
PackFramebufferAsync_(nvfb); // asynchronous glReadPixels using PBOs
} else {

View file

@ -527,10 +527,10 @@ void TextureCache::UpdateSamplingParams(TexCacheEntry &entry, bool force) {
entry.magFilt = magFilt;
}
// Workaround for a clamping bug in pre-HD ATI/AMD drivers
// Applies to Tegra too apparently. Maybe the issue is that we can't wrap non-pow2 textures on these
// platforms, if so then we should change the name of this bool.
if (gl_extensions.ATIClampBug && entry.framebuffer)
// Platforms without non-pow-2 extensions can't wrap non-pow-2 textures.
// Only framebuffer textures are non-pow-2 so this check works but excludes some cases
// where we could have enabled wrapping. TODO
if (!gl_extensions.OES_texture_npot && entry.framebuffer)
return;
if (force || entry.sClamp != sClamp) {

2
native

@ -1 +1 @@
Subproject commit d9467bca85933f85e978d22da8c711d072513131
Subproject commit 4cb8c4062a77981567013d05fee6fb6402131e45