GNU bug report logs - #14041
XTring_bell() does not work in some Linux distributions

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: PV <frerot13 <at> gmail.com>
Subject: bug#14041: closed (Re: bug#14041: XTring_bell() does not work in
 some Linux distributions)
Date: Mon, 25 Mar 2013 18:04:02 +0000

[Message part 1 (text/plain, inline)]

Your bug report

#14041: XTring_bell() does not work in some Linux distributions

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 14041 <at> debbugs.gnu.org.

-- 
14041: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=14041
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Jan Djärv <jan.h.d <at> swipnet.se>
To: PV <frerot13 <at> gmail.com>
Cc: 14041-done <at> debbugs.gnu.org
Subject: Re: bug#14041: XTring_bell() does not work in some Linux distributions
Date: Mon, 25 Mar 2013 19:01:21 +0100

Hello.

This has been fixed in the trunk in another way.

	Jan D.

23 mar 2013 kl. 11:38 skrev PV <frerot13 <at> gmail.com>:

> Hi,
> 
> In GNU Emacs 23.4.2 (x86_64-unknown-linux-gnu, GTK+ Version 2.20.1)
> Windowing system distributor `The X.Org Foundation', version 11.0.10707000
> 
> The X11 function XBell is affected by this bug (e.g. in the Debian 6 Squeeze)
> but the function gdk_beep() is not,
> thus I suggest to modify XTring_bell () in xterm.c in the following way:
> 
> 
> /* Make audible bell.  */
> 
> void
> XTring_bell ()
> {
>  struct frame *f = SELECTED_FRAME ();
> 
>  if (FRAME_X_DISPLAY (f))
>    {
> #if defined (HAVE_TIMEVAL) && defined (HAVE_SELECT)
>      if (visible_bell)
> 	XTflash (f);
>      else
> #endif
> 	{
> 	  BLOCK_INPUT;
> #ifdef USE_GTK           	     // P.VINCENT
> 	  gdk_beep ();   	     // P.VINCENT
> #else			 	     // P.VINCENT
> 	  XBell (FRAME_X_DISPLAY (f), 0);
> 	  XFlush (FRAME_X_DISPLAY (f));
> #endif                   	     // P.VINCENT
> 	  UNBLOCK_INPUT;
> 	}
>    }
> }
> 
> Best regards,
> Patrick VINCENT
> 
> 

[Message part 3 (message/rfc822, inline)]

From: PV <frerot13 <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: XTring_bell() does not work in some Linux distributions
Date: Sat, 23 Mar 2013 11:38:37 +0100

Hi,

In GNU Emacs 23.4.2 (x86_64-unknown-linux-gnu, GTK+ Version 2.20.1)
Windowing system distributor `The X.Org Foundation', version 11.0.10707000

The X11 function XBell is affected by this bug (e.g. in the Debian 6 Squeeze)
but the function gdk_beep() is not,
thus I suggest to modify XTring_bell () in xterm.c in the following way:


/* Make audible bell.  */

void
XTring_bell ()
{
  struct frame *f = SELECTED_FRAME ();

  if (FRAME_X_DISPLAY (f))
    {
#if defined (HAVE_TIMEVAL) && defined (HAVE_SELECT)
      if (visible_bell)
	XTflash (f);
      else
#endif
	{
	  BLOCK_INPUT;
#ifdef USE_GTK           	     // P.VINCENT
	  gdk_beep ();   	     // P.VINCENT
#else			 	     // P.VINCENT
	  XBell (FRAME_X_DISPLAY (f), 0);
	  XFlush (FRAME_X_DISPLAY (f));
#endif                   	     // P.VINCENT
	  UNBLOCK_INPUT;
	}
    }
}

Best regards,
Patrick VINCENT

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.