Re: [hatari-devel] Feedback SDL2 support

[David Savinkoff <dsavnkff@xxxxxxxxx>]

----- David Savinkoff wrote:
> ----- David Savinkoff wrote:
> > ----- Thomas Huth wrote:
> > > Am Sun, 24 May 2015 14:36:26 -0600 (MDT)
> > > 
> > >  Hi David,
> > > 
> > > sorry for the very late reply, ... finally I had some spare minutes to
> > > look into this...
> > > 
> > > schrieb David Savinkoff:
> > > 
> > > > ----- David Savinkoff wrote:
> > > > > 
> > > > > When I toggle SDL2 fullscreen <F11>, the mouse cursor gets
> > > > > repositioned because of the different screen sizes. The mouse
> > > > > cursor doesn't always get repositioned in a manner that I
> > > > > understand... probably in an incorrect manner. This probably causes
> > > > > the host computer to change 'window focus' when changing the screen
> > > > > sizes; giving bogus coordinates and wreaking havoc.
> > > > > 
> 
> Hi Thomas,
> 
> Here is a patch that gets Fullsrceen/Windowed toggling (F11)
> to work significantly better on my CentOS 5 computer. The
> FULLSCREEN and BORDERLESS flags are necessary to fix
> two separate problems.
> 
> Please try this patch on your system to investigate how
> things change for you.
> 
> Thank You,
> David Savinkoff

Hi Again,

This new patch works even better (instead of the previous patch).
I hope my digging will provide you with clues, or maybe the solution.

Sincerely,
David Savinkoff

--- hatari/src/control.old.c	2015-10-18 03:37:26.000000000 -0700
+++ hatari/src/control.new.c	2015-10-22 04:06:57.000000000 -0700
@@ -553,19 +553,15 @@
 #endif
 	if (noembed)
 	{
-#if !WITH_SDL2
 		/* show WM window again */
-		XMapWindow(display, wm_win);
-#endif
+		XMapWindow(display, sdl_win);
 	}
 	else
 	{
 		char buffer[12];  /* 32-bits in hex (+ '\r') + '\n' + '\0' */
 
-#if !WITH_SDL2
 		/* hide WM window for Hatari */
-		XUnmapWindow(display, wm_win);
-#endif
+		XUnmapWindow(display, sdl_win);
 		/* reparent main Hatari window to given parent */
 		XReparentWindow(display, sdl_win, parent_win, 0, 0);