This looks good to me:<div><br></div><div>Reviewed-by: Robert Bragg <<a href="mailto:robert@linux.intel.com">robert@linux.intel.com</a>></div><div><br></div><div>thanks,</div><div>- Robert</div><br><div class="gmail_quote">
On Tue, Nov 20, 2012 at 6:53 PM, Neil Roberts <span dir="ltr"><<a href="mailto:neil@linux.intel.com" target="_blank">neil@linux.intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The SDL 1 winsys now checks for the initial resizable state of the<br>
onscreen framebuffer when it is allocated and updates the video flags<br>
accordingly.<br>
---<br>
cogl/winsys/cogl-winsys-sdl.c | 10 +++++++++-<br>
1 file changed, 9 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/cogl/winsys/cogl-winsys-sdl.c b/cogl/winsys/cogl-winsys-sdl.c<br>
index ccb0764..e1dd528 100644<br>
--- a/cogl/winsys/cogl-winsys-sdl.c<br>
+++ b/cogl/winsys/cogl-winsys-sdl.c<br>
@@ -286,6 +286,7 @@ _cogl_winsys_onscreen_init (CoglOnscreen *onscreen,<br>
CoglContext *context = framebuffer->context;<br>
CoglDisplay *display = context->display;<br>
CoglDisplaySdl *sdl_display = display->winsys;<br>
+ CoglBool flags_changed = FALSE;<br>
int width, height;<br>
<br>
if (sdl_display->onscreen)<br>
@@ -299,9 +300,16 @@ _cogl_winsys_onscreen_init (CoglOnscreen *onscreen,<br>
width = cogl_framebuffer_get_width (framebuffer);<br>
height = cogl_framebuffer_get_height (framebuffer);<br>
<br>
+ if (cogl_onscreen_get_resizable (onscreen))<br>
+ {<br>
+ sdl_display->video_mode_flags |= SDL_RESIZABLE;<br>
+ flags_changed = TRUE;<br>
+ }<br>
+<br>
/* Try to update the video size using the onscreen size */<br>
if (width != sdl_display->surface->w ||<br>
- height != sdl_display->surface->h)<br>
+ height != sdl_display->surface->h ||<br>
+ flags_changed)<br>
{<br>
sdl_display->surface = SDL_SetVideoMode (width, height,<br>
0, /* bitsperpixel */<br>
<span class="HOEnZb"><font color="#888888">--<br>
1.7.11.3.g3c3efa5<br>
<br>
_______________________________________________<br>
Cogl mailing list<br>
<a href="mailto:Cogl@lists.freedesktop.org">Cogl@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/cogl" target="_blank">http://lists.freedesktop.org/mailman/listinfo/cogl</a><br>
</font></span></blockquote></div><br>