OpenGL bug? Nvidia driver bug? SFML bug? My fault?

[Laurent]

I know, this issue is very annoying. It's hard for me to fix it as I can't reproduce the error with my nVidia graphics card. I'll probably need help from you guys

[K-Bal]

I'll probably need help from you guys

What do you want us to do, send you an ATI card?

[Laurent]

What do you want us to do, send you an ATI card?

Testing my modifications and reporting success / failures should be enough

[sfPaul]

What do you want us to do, send you an ATI card?

Testing my modifications and reporting success / failures should be enough

Ok! I'm ready to help you if you want!

[Laurent]

The first thing you can test is an empty program linked against sfml-window

Code: [Select]

#include <SFML/Window.hpp>

int main()
{
    return 0;
}

And one with a sf::Window declared

Code: [Select]

#include <SFML/Window.hpp>

int main()
{
    sf::Window window(sf::VideoMode(800, 600), "test");
    return 0;
}

[Laurent]

Something else that you can test: someone reported me that the probem appeared at revision 1191; 1190 should work fine.

[sfPaul]

The first thing you can test is an empty program linked against sfml-window

Code: [Select]

#include <SFML/Window.hpp>

int main()
{
    return 0;
}

This works fine. No problems.

And one with a sf::Window declared

Code: [Select]

#include <SFML/Window.hpp>

int main()
{
    sf::Window window(sf::VideoMode(800, 600), "test");
    return 0;
}

But here the error appeared again.

Something else that you can test: someone reported me that the probem appeared at revision 1191; 1190 should work fine.

Sorry, but what do you mean with this?

[Laurent]

Thanks for your help.

Sorry, but what do you mean with this?

I'm talking about SVN revisions. The revision number 1191 is supposed to be the one where the problem appeared.

[K-Bal]

Revision 1190 works fine for me.

[Laurent]

Revision 1190 works fine for me.

And what about 1191?

[K-Bal]

Revision 1190 works fine for me.

And what about 1191?

GLX error

[Laurent]

Great

I have an idea, I'll try to submit a fix tomorrow.

[Lokk]

Ok, 1190 successed and 1191 failed for me too

[sfPaul]

Ok, 1190 successed and 1191 failed for me too

With me it was the same problem.

[Laurent]

New test:

Code: [Select]

// test.cpp
// --------

#include <X11/Xlib.h>
#include <GL/glx.h>
#include <GL/gl.h>
#include <GL/glxext.h>
#include <iostream>

class Context
{
public :

    Context(bool sameDisplay = true, const Context* shared = NULL)
    {
        if (shared && sameDisplay)
        {
            display = shared->display;
            sharedDisplay = true;
        }
        else
        {
            display = XOpenDisplay(NULL);
            sharedDisplay = false;
        }

        window = XCreateWindow(display,
                               RootWindow(display, DefaultScreen(display)),
                               0, 0,
                               1, 1,
                               0,
                               DefaultDepth(display, DefaultScreen(display)),
                               InputOutput,
                               DefaultVisual(display, DefaultScreen(display)),
                               0, NULL);

        XVisualInfo tpl;
        tpl.depth    = DefaultDepth(display, DefaultScreen(display));
        tpl.visualid = XVisualIDFromVisual(DefaultVisual(display, DefaultScreen(display)));
        tpl.screen   = DefaultScreen(display);

        int nbVisuals = 0;
        XVisualInfo* visuals = XGetVisualInfo(display, VisualDepthMask | VisualIDMask | VisualScreenMask, &tpl, &nbVisuals);
        if (!visuals || (nbVisuals == 0))
        {
            if (visuals)
                XFree(visuals);
            std::cout << "There is no valid visual for the selected screen" << std::endl;
            return;
        }

        context = glXCreateContext(display, &visuals[0], shared ? shared->context : NULL, true);
        if (!context)
        {
            std::cout << "Failed to create an OpenGL context for this window" << std::endl;
            return;
        }

        Window root = RootWindow(display, DefaultScreen(display));
        Colormap colorMap = XCreateColormap(display, root, visuals[0].visual, AllocNone);
        XSetWindowColormap(display, window, colorMap);

        XFree(visuals);

        glXMakeCurrent(display, window, context);

        std::cout << "Succeeded to create context!" << std::endl;
    }

    ~Context()
    {
        glXMakeCurrent(display, None, NULL);
        glXDestroyContext(display, context);
        XDestroyWindow(display, window);
       
        if (!sharedDisplay)
            XCloseDisplay(display);
    }

private :

    Display* display;
    Window window;
    GLXContext context;
    bool sharedDisplay;
};

int main()
{
    std::cout << "Creating the first context..." << std::endl;
    Context c1;

    std::cout << "Creating a shared context on the same display..." << std::endl;
    Context c2(true, &c1);

    std::cout << "Creating a shared context on a new display..." << std::endl;
    Context c3(false, &c1);

    return 0;
}

g++ test.cpp -o test -lX11 -lGL
./test

If I'm right, the two first contexts should succeed, and the last one should fail (with people having the GLX issue).