OK, I've a question: why do you remove the code entirely?  Didn't it
help enough to make tmp_win->HiliteImage NULL?

In message <[EMAIL PROTECTED]> on Sun, 21 May 2006 13:20:07 -0400, Todd Kover 
<[EMAIL PROTECTED]> said:

kovert> --- add_window.c        efc8288ef604525373100ce0eb1e04654bc7e6d4
kovert> +++ add_window.c        3f9659833e74eda8aebd20120b2defc3649af8d9
kovert> @@ -1764,7 +1766,15 @@
kovert>         } else {
kovert>             XFreePixmap (dpy, tmp_win->HiliteImage->pixmap);
kovert>         }
kovert> +#if 0
kovert> +       /* 
kovert> +        * XXX - this was being free'd twice for some reason.   
Commenting
kovert> +        * this out almost certainly introduces a memory leak, but 
they're
kovert> +        * better than core dumps.  :)
kovert> +        */
kovert>         free(tmp_win->HiliteImage);
kovert> +       tmp_win->HiliteImage = NULL;
kovert> +#endif
kovert>      }
kovert>  }
kovert>  


Cheers,
Richard

-----
Please consider sponsoring my work on free software.
See http://www.free.lp.se/sponsoring.html for details.

-- 
Richard Levitte                         [EMAIL PROTECTED]
                                        http://richard.levitte.org/

"When I became a man I put away childish things, including
 the fear of childishness and the desire to be very grown up."
                                                -- C.S. Lewis

Reply via email to