• Advertisement
Sign in to follow this  

OpenGL strange texturing error

This topic is 4812 days old which is more than the 365 day threshold we allow for new replies. Please post a new topic.

If you intended to correct an error in the post then please contact us.

Recommended Posts

When I try to texture-map, the textures are oriented correctly and everything, but are tinted slighly orange. I don't think it's my computer or monitor so what's teh deal? Here is my code:
#include <SDL.h>
#include <sdl_opengl.h>

#include <stdio.h>
#include <stdlib.h>

// screen width, height, and bit depth
const int SCREEN_WIDTH = 640;
const int SCREEN_HEIGHT = 480;
const int SCREEN_BPP = 16;

// game constants
const int COMPUTERS_TURN = 0;
const int PLAYERS_TURN = 1;

const int EMPTY_SPACE = 0;
const int COMPUTER_SPACE = 1;
const int PLAYER_SPACE = 2;

// Flags to pass to SDL_SetVideoMode
int videoFlags;

// This is our SDL surface
SDL_Surface * surface;

GLuint texture[1]; // Storage for one texture

void Quit(int returnCode);
int resizeWindow(int width, int height);
void handleKeyPress(SDL_keysym * keysym);
int LoadGLTextures();
int initGL(GLvoid);

void drawQuad(GLfloat x, GLfloat y);
int drawGLScene(GLvoid);

int main(int argc, char ** argv)
	// game variables
	int turn;
	int board[9] = {0,0,0,0,0,0,0,0,0}; // The square numbers are:
										// 0 1 2
										// 3 4 5
										// 6 7 8
	bool gameover = false;

	// main loop variable
    int done = FALSE;
	// used to collect events
    SDL_Event event;
	// this holds some info about our display
    const SDL_VideoInfo * videoInfo;
	// whether or not the window is active
    int isActive = TRUE;

    // initialize SDL */
    if (SDL_Init(SDL_INIT_VIDEO) < 0)
	    fprintf(stderr, "Video initialization failed: %s\n",

    // Fetch the video info
    videoInfo = SDL_GetVideoInfo();

    if (!videoInfo)
	    fprintf(stderr, "Video query failed: %s\n",

    // the flags to pass to SDL_SetVideoMode
    videoFlags  = SDL_OPENGL;						// Enable OpenGL in SDL
    videoFlags |= SDL_GL_DOUBLEBUFFER;				// Enable double buffering
    videoFlags |= SDL_HWPALETTE;					// Store the palette in hardware
    videoFlags |= SDL_RESIZABLE;					// Enable window resizing

    // This checks to see if surfaces can be stored in memory
    if (videoInfo->hw_available)
		videoFlags |= SDL_HWSURFACE;
		videoFlags |= SDL_SWSURFACE;

    // This checks if hardware blits can be done
    if (videoInfo->blit_hw)
		videoFlags |= SDL_HWACCEL;

    // Sets up OpenGL double buffering
    SDL_GL_SetAttribute(SDL_GL_DOUBLEBUFFER, 1);

    // get a SDL surface

    // Verify there is a surface
    if (!surface)
	    fprintf(stderr, "Video mode set failed: %s\n", SDL_GetError());

	// Set window caption
	SDL_WM_SetCaption("Tic Tac Toe - Press F1 to switch to fullscreen", NULL);

    // initialize OpenGL
    if (initGL() == FALSE)
	    fprintf(stderr, "Could not initialize OpenGL.\n");

    // Resize the initial window

    // wait for events
    while (!done)
	    // handle the events in the queue

	    while (SDL_PollEvent(&event))
					// Something's happened with our focus
				    // If we lost focus or we are iconified, we
				    // shouldn't draw the screen
				    if (event.active.gain == 0)
						isActive = FALSE;
						isActive = TRUE;
				    // handle resize event
				    surface = SDL_SetVideoMode(event.resize.w,
								16, videoFlags);
					if (!surface)
					    fprintf(stderr, "Could not get a surface after resize: %s\n", SDL_GetError());
					resizeWindow(event.resize.w, event.resize.h);

				case SDL_KEYDOWN:
				    // handle key presses
				case SDL_QUIT:
					// handle quit requests
					done = TRUE;

	    // draw the scene
	    if (isActive)

    // clean ourselves up and exit

    // Should never get here

// function to release/destroy our resources and restoring the old desktop
void Quit(int returnCode)
    // clean up the window

    // and exit appropriately

// function to reset our viewport after a window resize
int resizeWindow(int width, int height)
    // Height / width ration
    GLfloat ratio;

    // Protect against a divide by zero
    if (height == 0)
	height = 1;

    ratio = (GLfloat)width / (GLfloat)height;

    // Setup our viewport.
    glViewport(0,0, (GLint)width,(GLint)height);

    // change to the projection matrix and set our viewing volume.

    // Set our perspective
    gluPerspective(45.0f, ratio, 0.1f, 100.0f);

    // Make sure we're changing the model view and not the projection

    // Reset the view


// function to handle key press events
void handleKeyPress(SDL_keysym * keysym)
    switch (keysym->sym)
			// ESC key was pressed
		case SDLK_F1:
			// F1 key was pressed
		    // this toggles fullscreen mode
		    // the flags to pass to SDL_SetVideoMode

			// Verify there is a surface
			if (!surface)
				fprintf(stderr, "Video mode set failed: %s\n", SDL_GetError());


// function to load in bitmap as a GL texture
int LoadGLTextures()
	// Status indicator
	int Status = FALSE;

	// Create storage space for the texture
	SDL_Surface * TextureImage[1]; 

	// Load The Bitmap, Check For Errors, If Bitmap's Not Found Quit
	if ((TextureImage[0] = SDL_LoadBMP("square.bmp")))

		// Set the status to true
		Status = TRUE;

		// Create The Texture
		glGenTextures(1, &texture[0]);

		// Typical Texture Generation Using Data From The Bitmap
		glBindTexture(GL_TEXTURE_2D, texture[0]);

		// Generate The Texture
		glTexImage2D(GL_TEXTURE_2D, 0, 3, TextureImage[0]->w,
			  TextureImage[0]->h, 0, GL_RGB,
			  GL_UNSIGNED_BYTE, TextureImage[0]->pixels);

	    // Linear Filtering

    // Free up any memory we may have used
    if (TextureImage[0])

    return Status;

// general OpenGL initialization function
int initGL(GLvoid)
	/* Load in the texture */
    if (!LoadGLTextures())
	return FALSE;

    /* Enable Texture Mapping ( NEW ) */
	// Enable smooth shading

    // Set the background black

    // Depth buffer setup

    // Enables depth testing

    // The type of depth test to do

    // Really nice perspective calculations

    return 1;

void drawQuad(GLfloat x, GLfloat y)
		glTexCoord2f( 1.0f, 1.0f );glVertex3f(x,  y, 0.0);  // bottom right
		glTexCoord2f( 1.0f, 0.0f );glVertex3f(x,  y+2,0.0); // top right
		glTexCoord2f( 0.0f, 0.0f );glVertex3f(x-2,y+2,0.0); // top left
		glTexCoord2f( 0.0f, 1.0f );glVertex3f(x-2,y,  0.0); // bottom left

// Here goes our drawing code
int drawGLScene(GLvoid)
    // These are to calculate our fps
    static GLint T0     = 0;
    static GLint Frames = 0;

    // Clear the screen and the depth buffer

	// draw code goes here
	/* Select Our Texture */
    glBindTexture( GL_TEXTURE_2D, texture[0] );

	// row 1
	drawQuad(-1.0f,1.0);  // 0
	drawQuad( 1.0f,1.0);  // 1
	drawQuad( 3.0f,1.0);  // 2
	// row 2
	drawQuad(-1.0f,-1.0); // 3
	drawQuad(1.0f,-1.0);  // 4
	drawQuad(3.0f,-1.0);  // 5

	// row 2
	drawQuad(-1.0f,-3.0); // 6
	drawQuad(1.0f,-3.0);  // 7
	drawQuad(3.0f,-3.0);  // 8
    // Draw it to the screen

    // Gather our frames per second
		GLint t = SDL_GetTicks();
		if (t - T0 >= 5000)
		    GLfloat seconds = (t - T0) / float(1000.0);
		    GLfloat fps = Frames / seconds;
			printf("%d frames in %g seconds = %g FPS\n", Frames, seconds, fps);
		    T0 = t;
			Frames = 0;

    return 1;

Share this post

Link to post
Share on other sites
Ok, I think I figured out the problem: I changed the code:
glTexImage2D( GL_TEXTURE_2D, 0, 3, TextureImage[0]->w,
TextureImage[0]->h, 0, GL_BGR,
GL_UNSIGNED_BYTE, TextureImage[0]->pixels );

glTexImage2D( GL_TEXTURE_2D, 0, 3, TextureImage[0]->w,
TextureImage[0]->h, 0, GL_RGB,
GL_UNSIGNED_BYTE, TextureImage[0]->pixels )

I did this b/c it my compiler didn't recognize GL_BGR, but that caused it to invert the colors. I then changed it GL_RGB to GL_RGB_EXT and it now works fine, I just have one question. Is my app still portable or is GL_RGB_EXT a windows only constant? If so, is there a work around?

Share this post

Link to post
Share on other sites
glTexImage2D( GL_TEXTURE_2D, 0, GL_RGB, TextureImage[0]->w,
TextureImage[0]->h, 0, GL_BGR,
GL_UNSIGNED_BYTE, TextureImage[0]->pixels )

and include this file somewhere glext.h (get from www.opengl.org)
use BGR not BGR_EXT, also use RGB not 3

Share this post

Link to post
Share on other sites
Sign in to follow this  

  • Advertisement
  • Advertisement
  • Popular Tags

  • Advertisement
  • Popular Now

  • Similar Content

    • By QQemka
      Hello. I am coding a small thingy in my spare time. All i want to achieve is to load a heightmap (as the lowest possible walking terrain), some static meshes (elements of the environment) and a dynamic character (meaning i can move, collide with heightmap/static meshes and hold a varying item in a hand ). Got a bunch of questions, or rather problems i can't find solution to myself. Nearly all are deal with graphics/gpu, not the coding part. My c++ is on high enough level.
      Let's go:
      Heightmap - i obviously want it to be textured, size is hardcoded to 256x256 squares. I can't have one huge texture stretched over entire terrain cause every pixel would be enormous. Thats why i decided to use 2 specified textures. First will be a tileset consisting of 16 square tiles (u v range from 0 to 0.25 for first tile and so on) and second a 256x256 buffer with 0-15 value representing index of the tile from tileset for every heigtmap square. Problem is, how do i blend the edges nicely and make some computationally cheap changes so its not obvious there are only 16 tiles? Is it possible to generate such terrain with some existing program?
      Collisions - i want to use bounding sphere and aabb. But should i store them for a model or entity instance? Meaning i have 20 same trees spawned using the same tree model, but every entity got its own transformation (position, scale etc). Storing collision component per instance grats faster access + is precalculated and transformed (takes additional memory, but who cares?), so i stick with this, right? What should i do if object is dynamically rotated? The aabb is no longer aligned and calculating per vertex min/max everytime object rotates/scales is pretty expensive, right?
      Drawing aabb - problem similar to above (storing aabb data per instance or model). This time in my opinion per model is enough since every instance also does not have own vertex buffer but uses the shared one (so 20 trees share reference to one tree model). So rendering aabb is about taking the model's aabb, transforming with instance matrix and voila. What about aabb vertex buffer (this is more of a cosmetic question, just curious, bumped onto it in time of writing this). Is it better to make it as 8 points and index buffer (12 lines), or only 2 vertices with min/max x/y/z and having the shaders dynamically generate 6 other vertices and draw the box? Or maybe there should be just ONE 1x1x1 cube box template moved/scaled per entity?
      What if one model got a diffuse texture and a normal map, and other has only diffuse? Should i pass some bool flag to shader with that info, or just assume that my game supports only diffuse maps without fancy stuff?
      There were several more but i forgot/solved them at time of writing
      Thanks in advance
    • By RenanRR
      Hi All,
      I'm reading the tutorials from learnOpengl site (nice site) and I'm having a question on the camera (https://learnopengl.com/Getting-started/Camera).
      I always saw the camera being manipulated with the lookat, but in tutorial I saw the camera being changed through the MVP arrays, which do not seem to be camera, but rather the scene that changes:
      Vertex Shader:
      #version 330 core layout (location = 0) in vec3 aPos; layout (location = 1) in vec2 aTexCoord; out vec2 TexCoord; uniform mat4 model; uniform mat4 view; uniform mat4 projection; void main() { gl_Position = projection * view * model * vec4(aPos, 1.0f); TexCoord = vec2(aTexCoord.x, aTexCoord.y); } then, the matrix manipulated:
      ..... glm::mat4 projection = glm::perspective(glm::radians(fov), (float)SCR_WIDTH / (float)SCR_HEIGHT, 0.1f, 100.0f); ourShader.setMat4("projection", projection); .... glm::mat4 view = glm::lookAt(cameraPos, cameraPos + cameraFront, cameraUp); ourShader.setMat4("view", view); .... model = glm::rotate(model, glm::radians(angle), glm::vec3(1.0f, 0.3f, 0.5f)); ourShader.setMat4("model", model);  
      So, some doubts:
      - Why use it like that?
      - Is it okay to manipulate the camera that way?
      -in this way, are not the vertex's positions that changes instead of the camera?
      - I need to pass MVP to all shaders of object in my scenes ?
      What it seems, is that the camera stands still and the scenery that changes...
      it's right?
      Thank you
    • By dpadam450
      Sampling a floating point texture where the alpha channel holds 4-bytes of packed data into the float. I don't know how to cast the raw memory to treat it as an integer so I can perform bit-shifting operations.

      int rgbValue = int(textureSample.w);//4 bytes of data packed as color
      // algorithm might not be correct and endianness might need switching.
      vec3 extractedData = vec3(  rgbValue & 0xFF000000,  (rgbValue << 8) & 0xFF000000, (rgbValue << 16) & 0xFF000000);
      extractedData /= 255.0f;
    • By Devashish Khandelwal
      While writing a simple renderer using OpenGL, I faced an issue with the glGetUniformLocation function. For some reason, the location is coming to be -1.
      Anyone has any idea .. what should I do?
    • By Andrey OGL_D3D
      Hi all!
      I try to use the Sun shafts effects via post process in my 3DEngine, but i have some artefacts on final image(Please see attached images).
      The effect contains the following passes:
      1) Depth scene pass;
      2) "Shafts pass" Using DepthPass Texture + RGBA BackBuffer texture.
      3) Shafts pass texture +  RGBA BackBuffer texture.
      Shafts shader for 2 pass:
      // uniform sampler2D FullSampler; // RGBA Back Buffer uniform sampler2D DepthSampler; varying vec2 tex; #ifndef saturate float saturate(float val) {     return clamp(val, 0.0, 1.0); } #endif void main(void) {     vec2 uv = tex;     float sceneDepth = texture2D(DepthSampler, uv.xy).r;     vec4  scene        = texture2D(FullSampler, tex);     float fShaftsMask     = (1.0 - sceneDepth);     gl_FragColor = vec4( scene.xyz * saturate(sceneDepth), fShaftsMask ); } final shader:
      // uniform sampler2D FullSampler; // RGBA Back Buffer uniform sampler2D BlurSampler; // shafts sampler varying vec4 Sun_pos; const vec4    ShaftParams = vec4(0.1,2.0,0.1,2.0); varying vec2 Tex_UV; #ifndef saturate  float saturate(float val) {     return clamp(val, 0.0, 1.0); } #endif vec4 blendSoftLight(vec4 a, vec4 b) {   vec4 c = 2.0 * a * b + a * a * (1.0 - 2.0 * b);   vec4 d = sqrt(a) * (2.0 * b - 1.0) + 2.0 * a * (1.0 - b);       // TODO: To look in Crysis what it the shit???   //return ( b < 0.5 )? c : d;   return any(lessThan(b, vec4(0.5,0.5,0.5,0.5)))? c : d; } void main(void) {     vec4 sun_pos = Sun_pos;     vec2    sunPosProj = sun_pos.xy;     //float    sign = sun_pos.w;     float    sign = 1.0;     vec2    sunVec = sunPosProj.xy - (Tex_UV.xy - vec2(0.5, 0.5));     float    sunDist = saturate(sign) * saturate( 1.0 - saturate(length(sunVec) * ShaftParams.y ));     sunVec *= ShaftParams.x * sign;     vec4 accum;     vec2 tc = Tex_UV.xy;     tc += sunVec;     accum = texture2D(BlurSampler, tc);     tc += sunVec;     accum += texture2D(BlurSampler, tc) * 0.875;     tc += sunVec;     accum += texture2D(BlurSampler, tc) * 0.75;     tc += sunVec;     accum += texture2D(BlurSampler, tc) * 0.625;     tc += sunVec;     accum += texture2D(BlurSampler, tc) * 0.5;     tc += sunVec;     accum += texture2D(BlurSampler, tc) * 0.375;     tc += sunVec;     accum += texture2D(BlurSampler, tc) * 0.25;     tc += sunVec;     accum += texture2D(BlurSampler, tc) * 0.125;     accum  *= 0.25 * vec4(sunDist, sunDist, sunDist, 1.0);           accum.w += 1.0 - saturate(saturate(sign * 0.1 + 0.9));     vec4    cScreen = texture2D(FullSampler, Tex_UV.xy);           vec4    cSunShafts = accum;     float fShaftsMask = saturate(1.00001 - cSunShafts.w) * ShaftParams.z * 2.0;              float fBlend = cSunShafts.w;     vec4 sunColor = vec4(0.9, 0.8, 0.6, 1.0);     accum =  cScreen + cSunShafts.xyzz * ShaftParams.w * sunColor * (1.0 - cScreen);     accum = blendSoftLight(accum, sunColor * fShaftsMask * 0.5 + 0.5);     gl_FragColor = accum; } Demo project:
      Demo Project
      Shaders for postprocess Shaders/SunShaft/
      What i do wrong ?

  • Advertisement