Re: 1024 file descriptors should be enough for anyone

[Po Lu]

Robert Pluim <rpluim@gmail.com> writes:

> Is this the kind of change you were thinking of? Or do I need to call
> 'XtCloseDisplay' as well?

You have to call `XtCloseDisplay', yes.  Otherwise, that LGTM.

> diff --git a/src/xterm.c b/src/xterm.c
> index 285b1d625e..28bf81c0bd 100644
> --- a/src/xterm.c
> +++ b/src/xterm.c
> @@ -23232,13 +23232,18 @@ #define NUM_ARGV 10
>                        resource_name, EMACS_CLASS,
>                        emacs_options, XtNumber (emacs_options),
>                        &argc, argv);
> -    request_sigio ();
> -    turn_on_atimers (true);
> +    if (ConnectionNumber (dpy) >= FD_SETSIZE)
> +      dpy = 0;
> +    else
> +      {
> +     request_sigio ();
> +     turn_on_atimers (true);
>  
>  #ifdef HAVE_X11XTR6
>      /* I think this is to compensate for XtSetLanguageProc.  */
> -    fixup_locale ();
> +     fixup_locale ();
>  #endif
> +      }
>    }
>  
>  #else /* not USE_X_TOOLKIT */
>
> Robert