all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: "Jan Djärv" <jan.h.d@swipnet.se>
To: Darren Hoo <darren.hoo@gmail.com>
Cc: emacs-devel <emacs-devel@gnu.org>
Subject: Re: Problems with compositing-enabled metacity in gnome enviroment
Date: Fri, 11 Feb 2011 17:02:44 +0100	[thread overview]
Message-ID: <4D555DA4.8050303@swipnet.se> (raw)
In-Reply-To: <AANLkTinkspSqZ65wfAW402ZJx=PH3j8x0Cd_pKroMmpi@mail.gmail.com>

2011-02-11 15:53, Darren Hoo skrev:

> The version of  metacity I am using is 2.30.1-3 in Debian Unstable. Is
> there any related
> bug report of metacity that I can refer to?

No, but I guess you can file one.  BTW, I got 2.30.2-0 in ubuntu, and I always 
get the bug you see, even with emacs-23 that hasn't the commit you mention below.

>
> I had experienced the varnishes of  Gnome panel a long time ago, But
> since then I
> never experience it for at least half a year, aslo with  Chromium I
> never encounter
> such  symptoms as you mentioned.
>
> The only thing I want with compositing is the fancy tranparency of
> Emacs, switching to
> compiz seems not to be a choice for me, compiz is somewhat over-bloated to me.
>
>>
>> BTW, you probably meant (set-frame-parameter nil 'alpha 94).  Also, please
>> use report-emacs-bug for bugs like these.
> Got it
>
>> I reverted to the version before this commit, but the bug in metacity remains.  It is
>> probably a fluke that your meatcity works with>  that version.
>
> just FYI,  it is this specific hunk of the commit that triggers the
> (possible) bug of metacity
> Just revert this hunk of the commit Emacs works as expected at least for me.
>
> src/xterm.c:
>
> @@ -489,14 +489,14 @@ x_set_frame_alpha (struct frame *f)
>   			&data);
>
>       if (rc == Success&&  actual != None)
> -      if (*(unsigned long *)data == opac)
> -	{
> -	  XFree ((void *) data);
> -	  x_uncatch_errors ();
> -	  return;
> -	}
> -      else
> +      {
>   	XFree ((void *) data);
> +	if (*(unsigned long *)data == opac)
> +	  {
> +	    x_uncatch_errors ();
> +	    return;
> +	  }
> +      }
>       x_uncatch_errors ();
>     }

Ouch, this looks really bad.  Not only is data accesses after it has been 
free:d, but we may be doing catch errors twice and uncatch only once.

I fixed this, see if it works with your metacity.  It doesn't for me.

	Jan D.





  reply	other threads:[~2011-02-11 16:02 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-02-10 18:29 Problems with compositing-enabled metacity in gnome enviroment Darren Hoo
2011-02-11 13:52 ` Jan Djärv
2011-02-11 14:53   ` Darren Hoo
2011-02-11 16:02     ` Jan Djärv [this message]
2011-02-11 16:15       ` Darren Hoo

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4D555DA4.8050303@swipnet.se \
    --to=jan.h.d@swipnet.se \
    --cc=darren.hoo@gmail.com \
    --cc=emacs-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.