Private Class Member not Initialized

[snow 56767]

Hi Guys,
I have another problem, I have a class that is used for displaying a main menu in a game that seems to be having issues with it's privatized members not being initialized. Whenever I try to assign a value to one of these members, I get a SEG FAULT error.

Here is the Class (Menu.h):

#ifndef _MENU_H
#define _MENU_H

class Menu
{
public:
    Menu(int width, int height, bool isWide);
    ~Menu();

    void DrawMenu();
    void Resize(int width, int height);
    uint Click(int x, int y); //This returns the ID of the object clicked. IF nothing was clicked, return is 0

private:
    void LoadMenuTextures(bool isWide);

    GLuint tex[4]; //There are 4 textures total
    Quad bg; //The is the background quad
    Quad btns[3]; //There are three buttons: Start, Options, and Exit/Quit
};

#endif //_MENU_H 

Here is the Implementation (Menu.cpp)

#include "Menu.h"

Menu::Menu(int width, int height, bool isWide)
{
    Resize(width, height);
    LoadMenuTextures(isWide);
}

Menu::~Menu() {}

void Menu::Resize(int width, int height)
{
    //SETUP THE BACKGROUND
    bg.x1 = 0;
    bg.y1 = 0;
    bg.x2 = width;
    bg.y2 = height;

    //SETUP THE BUTTONS
    double btnStartX = (width * 0.66666);
    double btnStartY = (height * 0.25);
    double btnWidth = (width * 0.25);
    double btnHeight = (height * 0.125);

    //SET THE BUTTONS
    btns[0].x1 = btnStartX;
    btns[0].y1 = btnStartY;
    btns[0].x2 = btnStartX + btnWidth;
    btns[0].y2 = btnStartY + btnHeight;

    btns[1].x1 = btnStartX;
    btns[1].y1 = btnStartY + (btnHeight *2);
    btns[1].x2 = btnStartX + btnWidth;
    btns[1].y2 = btnStartY + (btnHeight *3);

    btns[2].x1 = btnStartX;
    btns[2].y1 = btnStartY + (btnHeight *4);
    btns[2].x2 = btnStartX + btnWidth;
    btns[2].y2 = btnStartY + (btnHeight *5);
}

uint Menu::Click(int x, int y)
{
    //COLLISION TEST ON THE BUTTONS
    for(int i = 0; i < 3; ++i)
    {
        if(x > btns[i].x1 && y > btns[i].y1)
        {
            if(x < btns[i].x2 && y < btns[i].y2)
            {
                return (i + 1);
            }
        }
    }

    return 0;
}

void Menu::DrawMenu()
{
    glPushMatrix();

    //DRAW THE BACKGROUND
    glColor4d(1.0, 1.0, 1.0, 1.0);
    glBindTexture(GL_TEXTURE_2D, tex[0]);
    glBegin(GL_QUADS);
        glTexCoord2d(0, 0);
        glVertex2d(bg.x1, bg.y1);

        glTexCoord2d(0, 1);
        glVertex2d(bg.x1, bg.y2);

        glTexCoord2d(1, 1);
        glVertex2d(bg.x2, bg.y2);

        glTexCoord2d(1, 0);
        glVertex2d(bg.x2, bg.y1);
    glEnd();


    //DRAW THE BUTTONS
    glColor4d(1.0, 1.0, 1.0, 1.0); //The Default Color of the Buttons is Cyan

    for(int i = 0; i < 3; ++i)
    {
        glBindTexture(GL_TEXTURE_2D, tex[(i + 1)]);

        glBegin(GL_QUADS);
            glTexCoord2d(0, 0);
            glVertex2d(btns[i].x1, btns[i].y1);

            glTexCoord2d(0, 1);
            glVertex2d(btns[i].x1, btns[i].y2);

            glTexCoord2d(1, 1);
            glVertex2d(btns[i].x2, btns[i].y2);

            glTexCoord2d(1, 0);
            glVertex2d(btns[i].x2, btns[i].y1);
        glEnd();
    }

    glPopMatrix();
}

void Menu::LoadMenuTextures(bool isWide)
{
    YsRawPngDecoder *png = new YsRawPngDecoder();

    glGenTextures(4, tex);

    for(int i = 0; i < 4; ++i)
    {
        glBindTexture(GL_TEXTURE_2D, tex[i]);

        switch(i)
        {
            case 0:
            {
                if(isWide)
                {
                    png->Decode("Data/Art/16by9/menubg.png");
                }

                else
                {
                    png->Decode("Data/Art/4by3/menubg.png");
                }
                
                gluBuild2DMipmaps(GL_TEXTURE_2D, GL_RGBA, png->wid, png->hei, GL_RGBA, GL_UNSIGNED_BYTE, png->rgba);

                continue;
            }

            case 1:
            {
                if(isWide)
                {
                    png->Decode("Data/Art/16by9/menuStart.png");
                }

                else
                {
                    png->Decode("Data/Art/4by3/menuStart.png");
                }
                
                gluBuild2DMipmaps(GL_TEXTURE_2D, GL_RGBA, png->wid, png->hei, GL_RGBA, GL_UNSIGNED_BYTE, png->rgba);

                continue;
            }

            case 2:
            {
                if(isWide)
                {
                    png->Decode("Data/Art/16by9/menuOpt.png");
                }

                else
                {
                    png->Decode("Data/Art/4by3/menuOpt.png");
                }
                
                gluBuild2DMipmaps(GL_TEXTURE_2D, GL_RGBA, png->wid, png->hei, GL_RGBA, GL_UNSIGNED_BYTE, png->rgba);

                continue;
            }

            case 3:
            {
                if(isWide)
                {
                    png->Decode("Data/Art/16by9/menuExit.png");
                }

                else
                {
                    png->Decode("Data/Art/4by3/menuExit.png");
                }
                
                gluBuild2DMipmaps(GL_TEXTURE_2D, GL_RGBA, png->wid, png->hei, GL_RGBA, GL_UNSIGNED_BYTE, png->rgba);

                continue;
            }
        }
    }

    glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_LINEAR_MIPMAP_LINEAR);
    glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_LINEAR);
    glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_S, GL_CLAMP);
    glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_T, GL_CLAMP);

    free(png);
}

Here is how the custom data types are defined as so:

typedef unsigned int uint;

typedef struct _Quad
{
    double x1, y1, x2, y2;
}Quad;

(If it is an help, here is the disassembly):

    //SETUP THE BACKGROUND
    bg.x1 = 0;
004050A9  mov         eax,dword ptr [this] 
004050AC  fldz             
004050AE  fstp        qword ptr [eax+10h] 

Sorry about the MASSIVE post, I didn't want to leave any details out.

The issues isn't with just this class, but ALL my classes that use Quad. It didn't used to do this. I haven't actually changed any of this code since I wrote it. When I wrote it, it compiled fine and ran well. Now it compiles, but won't run.

Also, I am using MSVC 2008, Win32 API, OpenGL, OpenAL, libsndfile, and a PNG library called YsPng.

[Moschops]

I see you're mixing new and free. That's bad. If you create using new, destroy using delete.

[snow 56767]

Ok,
I replaced free with delete.

I also fixed my own problem, It turned out that this code is perfectly fine. The problem used to lie in my main file where Menu::Resize was getting called before the constructor! So I fixed it there, now it works just fine!

Thank you for pointing out that I shouldn't mix new and free.
I'll use delete from now on.