Author: AlbrechtS
Date: 2009-04-16 15:23:36 -0700 (Thu, 16 Apr 2009)
New Revision: 6767
Log:
STR 2185: Uncommented "// XCloseIM(xim_im); FIXME", as discussed in
STR 2185 and in fltk.development to prevent memory leak. The removed comment
about a crash in XFree86: _XlcCreateDefaultCharSet() appears to be outdated
(probably from 2005/2006).


Modified:
   branches/branch-1.3/src/Fl_x.cxx

Modified: branches/branch-1.3/src/Fl_x.cxx
===================================================================
--- branches/branch-1.3/src/Fl_x.cxx    2009-04-16 21:10:49 UTC (rev 6766)
+++ branches/branch-1.3/src/Fl_x.cxx    2009-04-16 22:23:36 UTC (rev 6767)
@@ -813,11 +813,7 @@
       fl_xim_im = NULL;
       fl_init_xim();
     } else {
-     // XCloseIM(xim_im); FIXME
-        /* XFree86 has a bug when closing IM it crashes in
-         * _XlcCreateDefaultCharSet() !  So don't close it.
-         * This will cause a memory leak :-(
-         */
+      XCloseIM(xim_im);        // see STR 2185 for comment
     }
     return 0;
   }

_______________________________________________
fltk-commit mailing list
[email protected]
http://lists.easysw.com/mailman/listinfo/fltk-commit

Reply via email to