Skip to content

Commit

Permalink
Unify GLES support in gui scaling filter
Browse files Browse the repository at this point in the history
  • Loading branch information
sfan5 committed Aug 4, 2019
1 parent ca363d3 commit 6c2ded4
Show file tree
Hide file tree
Showing 3 changed files with 44 additions and 38 deletions.
24 changes: 13 additions & 11 deletions src/client/guiscalingfilter.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ with this program; if not, write to the Free Software Foundation, Inc.,
#include "util/numeric.h"
#include <cstdio>
#include "client/renderingengine.h"
#include "client/tile.h" // hasNPotSupport()

/* Maintain a static cache to store the images that correspond to textures
* in a format that's manipulable by code. Some platforms exhibit issues
Expand Down Expand Up @@ -113,17 +114,18 @@ video::ITexture *guiScalingResizeCached(video::IVideoDriver *driver,
(u32)destrect.getHeight()));
imageScaleNNAA(srcimg, srcrect, destimg);

#ifdef __ANDROID__
// Android is very picky about textures being powers of 2, so expand
// the image dimensions to the next power of 2, if necessary, for
// that platform.
video::IImage *po2img = driver->createImage(src->getColorFormat(),
core::dimension2d<u32>(npot2((u32)destrect.getWidth()),
npot2((u32)destrect.getHeight())));
po2img->fill(video::SColor(0, 0, 0, 0));
destimg->copyTo(po2img);
destimg->drop();
destimg = po2img;
#if ENABLE_GLES
// Some platforms are picky about textures being powers of 2, so expand
// the image dimensions to the next power of 2, if necessary.
if (!hasNPotSupport()) {
video::IImage *po2img = driver->createImage(src->getColorFormat(),
core::dimension2d<u32>(npot2((u32)destrect.getWidth()),
npot2((u32)destrect.getHeight())));
po2img->fill(video::SColor(0, 0, 0, 0));
destimg->copyTo(po2img);
destimg->drop();
destimg = po2img;
}
#endif

// Convert the scaled image back into a texture.
Expand Down
57 changes: 30 additions & 27 deletions src/client/tile.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -995,60 +995,63 @@ video::IImage* TextureSource::generateImage(const std::string &name)
}

#if ENABLE_GLES
/**
* Check and align image to npot2 if required by hardware
* @param image image to check for npot2 alignment
* @param driver driver to use for image operations
* @return image or copy of image aligned to npot2
*/


static inline u16 get_GL_major_version()
{
const GLubyte *gl_version = glGetString(GL_VERSION);
return (u16) (gl_version[0] - '0');
}

video::IImage * Align2Npot2(video::IImage * image,
video::IVideoDriver* driver)
{
if (image == NULL) {
return image;
}

core::dimension2d<u32> dim = image->getDimension();
/**
* Check if hardware requires npot2 aligned textures
* @return true if alignment NOT(!) requires, false otherwise
*/

bool hasNPotSupport()
{
// Only GLES2 is trusted to correctly report npot support
// Note: we cache the boolean result. GL context will never change on Android.
static const bool hasNPotSupport = get_GL_major_version() > 1 &&
// Note: we cache the boolean result, the GL context will never change.
static const bool supported = get_GL_major_version() > 1 &&
glGetString(GL_EXTENSIONS) &&
strstr((char *)glGetString(GL_EXTENSIONS), "GL_OES_texture_npot");
return supported;
}

if (hasNPotSupport)
/**
* Check and align image to npot2 if required by hardware
* @param image image to check for npot2 alignment
* @param driver driver to use for image operations
* @return image or copy of image aligned to npot2
*/

video::IImage * Align2Npot2(video::IImage * image,
video::IVideoDriver* driver)
{
if (image == NULL)
return image;

if (hasNPotSupport())
return image;

core::dimension2d<u32> dim = image->getDimension();
unsigned int height = npot2(dim.Height);
unsigned int width = npot2(dim.Width);

if ((dim.Height == height) &&
(dim.Width == width)) {
if (dim.Height == height && dim.Width == width)
return image;
}

if (dim.Height > height) {
if (dim.Height > height)
height *= 2;
}

if (dim.Width > width) {
if (dim.Width > width)
width *= 2;
}

video::IImage *targetimage =
driver->createImage(video::ECF_A8R8G8B8,
core::dimension2d<u32>(width, height));

if (targetimage != NULL) {
if (targetimage != NULL)
image->copyToScaling(targetimage);
}
image->drop();
return targetimage;
}
Expand Down
1 change: 1 addition & 0 deletions src/client/tile.h
Original file line number Diff line number Diff line change
Expand Up @@ -135,6 +135,7 @@ class IWritableTextureSource : public ITextureSource
IWritableTextureSource *createTextureSource();

#if ENABLE_GLES
bool hasNPotSupport();
video::IImage * Align2Npot2(video::IImage * image, irr::video::IVideoDriver* driver);
#endif

Expand Down

0 comments on commit 6c2ded4

Please sign in to comment.