Applied, although I changed to return 0 not 1 (no need to redraw). Thanks!

On Fri, Oct 19, 2012 at 12:11:37AM +0900, Seiji Ohashi wrote:
> 
> At Thu, 18 Oct 2012 15:40:54 +0100,
> Thomas Adam wrote:
> > Isn't it easier to check for this before the loop and just "return;"?
> 
> Thank you. I remade a patch.
> 
> diff --git a/server-window.c b/server-window.c
> index 7220320..753ab10 100644
> --- a/server-window.c
> +++ b/server-window.c
> @@ -81,6 +81,8 @@ server_window_check_bell(struct session *s, struct winlink 
> *wl)
>  
>       visual = options_get_number(&s->options, "visual-bell");
>       action = options_get_number(&s->options, "bell-action");
> +     if (action == BELL_NONE)
> +             return (1);
>       for (i = 0; i < ARRAY_LENGTH(&clients); i++) {
>               c = ARRAY_ITEM(&clients, i);
>               if (c == NULL || c->session != s)
> 
> ------------------------------------------------------------------------------
> Everyone hates slow websites. So do we.
> Make your web apps faster with AppDynamics
> Download AppDynamics Lite for free today:
> http://p.sf.net/sfu/appdyn_sfd2d_oct
> _______________________________________________
> tmux-users mailing list
> tmux-users@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/tmux-users

------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_sfd2d_oct
_______________________________________________
tmux-users mailing list
tmux-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tmux-users

Reply via email to