Re: [AD] Shutdown error

[Richard Reeve <r.e.reeve@xxxxxxxxxx>]

Hi,

For me this was very annoying since every allegro application caused this
error and managed to crash my task list every time it shut. Here's a patch
which eliminates the problem, but doesn't fix it - it just removes the
XCloseDisplay and XDestroyWindow calls which are causing it - it seems to
not affect the closing of the windows though (? I guess that happens when
the application exits anyway). It also fixes a problem with the detection
of gcc v3.0.x .  Apply it in the allegro directory with "patch -p1 <
patch-allegro". Hope that helps, but if anyone has a better solution I'd
be delighted - it is a very annoying bug.

Cheers,

Richard Reeve

On Wed, 19 Dec 2001, Lo'oRiS il Kabukimono wrote:

> LALI-HO!
>
> Do you know that strange error message that appears sometimes when closing
> an Allegro application under X? (if i remember correctly this isn't fixed,
> is it?)
> Today i found that message on the console from witch i started X, but i
> didn't run any Allegro program (afaik)...
> I don't know if this can help, but if u want me to send some logs, please
> tell me what files to send and i'll do.
>
>
> "User has performed an invalid operation and will be terminated."
>                 Lorenzo Petrone, Windows 2020 Cyberpunk Edition
>
> _.-:/°^^°\:-._.-:/°^^°\:-._          __.         ____
>  - Lo'oRiS il Kabukimono -          / /|       /^___ \
> Real Name: Lorenzo Petrone         / / /      / /L_/ /
> e-mail: _lano_@xxxxxxxxxx          / / /      /  ___-°/
> irc.azzurra.org #lano            / /_/__    /  /|__-°
> digilander.iol.it/lano666       /______/|  /__/ /
> ^°\:-.__.-:/°^°\:-.__.-:/°^     |______|/  |__L/
>

diff -Naur allegro-4.0.0/aclocal.m4 allegro-new/aclocal.m4
--- allegro-4.0.0/aclocal.m4	Wed Dec  5 23:46:32 2001
+++ allegro-new/aclocal.m4	Mon Dec 17 20:58:03 2001
@@ -460,7 +460,7 @@
 AC_DEFUN(ALLEGRO_ACTEST_GCC_VERSION,
 [AC_MSG_CHECKING(whether -fomit-frame-pointer is safe)
 AC_CACHE_VAL(allegro_cv_support_fomit_frame_pointer,
-[if test $GCC = yes && $CC --version | grep -q '3\.0[[012]]$'; then
+[if test $GCC = yes && $CC --version | grep -q '3\.0\.[[012]]$'; then
   allegro_cv_support_fomit_frame_pointer=no
 else
   allegro_cv_support_fomit_frame_pointer=yes
diff -Naur allegro-4.0.0/configure allegro-new/configure
--- allegro-4.0.0/configure	Mon Dec 10 00:38:57 2001
+++ allegro-new/configure	Mon Dec 17 20:58:32 2001
@@ -1002,7 +1002,7 @@
 if eval "test \"`echo '$''{'allegro_cv_support_fomit_frame_pointer'+set}'`\" = set"; then
   echo $ac_n "(cached) $ac_c" 1>&6
 else
-  if test $GCC = yes && $CC --version | grep -q '3\.0[012]$'; then
+  if test $GCC = yes && $CC --version | grep -q '3\.0\.[012]$'; then
   allegro_cv_support_fomit_frame_pointer=no
 else
   allegro_cv_support_fomit_frame_pointer=yes
diff -Naur allegro-4.0.0/src/x/xwin.c allegro-new/src/x/xwin.c
--- allegro-4.0.0/src/x/xwin.c	Sat Nov 24 00:59:23 2001
+++ allegro-new/src/x/xwin.c	Mon Dec 17 20:58:24 2001
@@ -332,7 +332,7 @@
 
    if (_xwin.display != 0) {
       _xwin_destroy_window();
-      XCloseDisplay(_xwin.display);
+      //XCloseDisplay(_xwin.display);
       _xwin.display = 0;
    }
 
@@ -472,7 +472,7 @@
 
    if (_xwin.window != None) {
       XUnmapWindow(_xwin.display, _xwin.window);
-      XDestroyWindow(_xwin.display, _xwin.window);
+      //XDestroyWindow(_xwin.display, _xwin.window);
       _xwin.window = None;
    }
 }