<div dir="ltr"><br><br>On Thu, Feb 25, 2016 at 7:29 AM, Matteo Bruni <<a href="mailto:matteo.mystral@gmail.com">matteo.mystral@gmail.com</a>> wrote:<br>><br>> 2016-02-24 5:12 GMT+01:00 Charles Davis <<a href="mailto:cdavis5x@gmail.com">cdavis5x@gmail.com</a>>:<br>> > Signed-off-by: Charles Davis <<a href="mailto:cdavis5x@gmail.com">cdavis5x@gmail.com</a>><br>> > ---<br>> > Try 3: Fix an assignment I missed. (Thanks, Ken.)<br>> ><br>> >  dlls/opengl32/tests/opengl.c |  2 +-<br>> >  dlls/winex11.drv/opengl.c    | 69 +++++++++++++++++++++++++-------------------<br>> >  2 files changed, 41 insertions(+), 30 deletions(-)<br>> ><br>> > diff --git a/dlls/opengl32/tests/opengl.c b/dlls/opengl32/tests/opengl.c<br>> > index e5e1507..c058c19 100644<br>> > --- a/dlls/opengl32/tests/opengl.c<br>> > +++ b/dlls/opengl32/tests/opengl.c<br>> > @@ -975,7 +975,7 @@ static void test_opengl3(HDC hdc)<br>> >           * expect drivers to ever offer it.<br>> >           */<br>> >          res = wglMakeCurrent(0, gl3Ctx);<br>> > -        ok(res == FALSE, "Wow, OpenGL 3.0 windowless rendering passed while it was expected not to!\n");<br>> > +        ok(res || broken(!res), "OpenGL 3.0 windowless rendering failed!\n");<br>><br>> That seems suspicious...<br>> AFAICS the comment right above this hunk is still valid (it is for<br>> current Nvidia drivers on Win7 at least), which probably means that<br>> this patch is not a good idea.<br><br>Yeah, I was afraid of that.<br><br>Truth be told, I don't need this for an application. <div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">​Besides the fact that OpenGL 3.0+ is supposed to allow this, ​</div>I added it because I thought it might be useful for wined3d (e.g. being able to create devices without an implicit swapchain, though now that I've looked into that possibility, <div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">​making</div> that<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">​ change​</div> would be much harder than I thought). In any case, I<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">​'d​</div> imagine from your comment that you disagree. If you do, I<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">​'ll​</div> retract this patch and revert the patch adding this to winemac.drv<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">​, since no known apps depend on this​</div>.<br><br>Chip<br></div>