Temporarily turned off mipmap filter mode 17/254717/1
authorDavid Steele <david.steele@samsung.com>
Mon, 8 Mar 2021 18:54:12 +0000 (18:54 +0000)
committerDavid Steele <david.steele@samsung.com>
Mon, 8 Mar 2021 18:54:12 +0000 (18:54 +0000)
Until mip-map generation is implemented, mipmap filtering should
not be set.

Change-Id: I95f7e2f648c4aecf71cb35b682f4ef5e3fd36e13
Signed-off-by: David Steele <david.steele@samsung.com>
automated-tests/src/dali-graphics/utc-Dali-GraphicsSampler.cpp
dali/internal/graphics/gles-impl/gles-graphics-texture.cpp
dali/internal/graphics/gles-impl/gles-graphics-types.h

index 8f1994e..217fbd6 100644 (file)
@@ -131,7 +131,10 @@ int UtcDaliGraphicsSamplerAllSet(void)
   auto params = glParamTrace.FindLastMatch("TexParameteri", minFilter);
   DALI_TEST_CHECK(params != nullptr);
   auto iter = params->find("param");
-  DALI_TEST_CHECK(*iter == GL_LINEAR_MIPMAP_NEAREST);
+
+  //@todo Change back when mip-map generation is implemented
+  //DALI_TEST_CHECK(*iter == GL_LINEAR_MIPMAP_NEAREST);
+  DALI_TEST_CHECK(*iter == GL_LINEAR);
 
   TraceCallStack::NamedParams magFilter;
   magFilter["pname"] << std::hex << GL_TEXTURE_MAG_FILTER;
@@ -161,5 +164,12 @@ int UtcDaliGraphicsSamplerAllSet(void)
   app.SendNotification();
   app.Render(16); // The above actor will get rendered and drawn once.
 
+  params = glParamTrace.FindLastMatch("TexParameteri", minFilter);
+  DALI_TEST_CHECK(params != nullptr);
+  iter = params->find("param");
+  //@todo Change back when mip-map generation is implemented
+  //DALI_TEST_CHECK(*iter == GL_NEAREST_MIPMAP_LINEAR);
+  DALI_TEST_CHECK(*iter == GL_NEAREST);
+
   END_TEST;
 }
index 442583f..54fab1e 100644 (file)
@@ -197,7 +197,10 @@ void Texture::Bind(const TextureBinding& binding) const
     auto*       sampler           = static_cast<const GLES::Sampler*>(binding.sampler);
     const auto& samplerCreateInfo = sampler->GetCreateInfo();
 
-    gl->TexParameteri(mGlTarget, GL_TEXTURE_MIN_FILTER, GLSamplerFilterAndMipMapMode(samplerCreateInfo.minFilter, samplerCreateInfo.mipMapMode).glFilter);
+    auto mipMapMode = samplerCreateInfo.mipMapMode;
+    mipMapMode      = Graphics::SamplerMipmapMode::NONE; // @todo Remove when mip-map generation is supported
+
+    gl->TexParameteri(mGlTarget, GL_TEXTURE_MIN_FILTER, GLSamplerFilterAndMipMapMode(samplerCreateInfo.minFilter, mipMapMode).glFilter);
     gl->TexParameteri(mGlTarget, GL_TEXTURE_MAG_FILTER, GLSamplerFilter(samplerCreateInfo.magFilter).glFilter);
     gl->TexParameteri(mGlTarget, GL_TEXTURE_WRAP_S, GLAddressMode(samplerCreateInfo.addressModeU).texParameter);
     gl->TexParameteri(mGlTarget, GL_TEXTURE_WRAP_T, GLAddressMode(samplerCreateInfo.addressModeV).texParameter);
index 46946c8..4246adf 100644 (file)
@@ -1073,7 +1073,7 @@ struct GLSamplerFilterAndMipMapMode
           }
           case Graphics::SamplerMipmapMode::LINEAR:
           {
-            glFilter = GL_NEAREST_MIPMAP_NEAREST;
+            glFilter = GL_NEAREST_MIPMAP_LINEAR;
             break;
           }
         }
@@ -1096,10 +1096,11 @@ struct GLSamplerFilterAndMipMapMode
           }
           case Graphics::SamplerMipmapMode::LINEAR:
           {
-            glFilter = GL_LINEAR_MIPMAP_NEAREST;
+            glFilter = GL_LINEAR_MIPMAP_LINEAR;
             break;
           }
         }
+        break;
       }
     }
   }