From: pavenis AT lanet DOT lv
To: "Eli Zaretskii" <eliz AT is DOT elta DOT co DOT il>, djgpp-workers AT delorie DOT com
Date: Fri, 17 Aug 2001 13:29:42 +0300
MIME-Version: 1.0
Content-type: text/plain; charset=US-ASCII
Content-transfer-encoding: 7BIT
Subject: Re: _open.c commit?  (was Re: Selector Exhaustion)
Message-ID: <3B7D1C46.23448.59E465@localhost>
In-reply-to: <2561-Fri17Aug2001115601+0300-eliz@is.elta.co.il>
References: <10108162209 DOT AA13700 AT clio DOT rice DOT edu> (sandmann AT clio DOT rice DOT edu)
X-mailer: Pegasus Mail for Win32 (v3.12c)
Reply-To: djgpp-workers AT delorie DOT com
Errors-To: nobody AT delorie DOT com
X-Mailing-List: djgpp-workers AT delorie DOT com
X-Unsubscribes-To: listserv AT delorie DOT com
Precedence: bulk

On 17 Aug 2001, at 11:56, Eli Zaretskii wrote:

> > From: sandmann AT clio DOT rice DOT edu (Charles Sandmann)
> > Date: Thu, 16 Aug 2001 17:09:40 -0500 (CDT)
> > 
> > Should we sit on the current selector cleanup patch or commit it also?
> 
> I think it should be committed and we should ask people to build
> applications (such as Make, RHIDE, and Bash) with it and test it in a
> variety of environments.
> 
> However, I'd like to see that patch changed so that plain DOS systems
> aren't affected at all.  Why slow down systems which don't need that?
> We already have the _windows_major variable that can be used to easily
> test for whether we are on Windows, and _get_dos_version(1) can be
> used for NT/W2K/XP.

Have You tried  printf ("%d\n",_windows_major) at program startup?

(static function get_windows_version() was only called by 
__maybe_fix_w2k_ntvdm_bug() so it was not initialized at startup)

Is it Ok to commit?

Andris

*** djgpp/src/libc/go32/dpmiexcp.c~3	Mon Jul 30 14:35:52 2001
--- djgpp/src/libc/go32/dpmiexcp.c	Fri Aug 17 13:21:30 2001
*************** __djgpp_set_ctrl_c(int enable_sigs)
*** 589,595 ****
  unsigned short _windows_major, _windows_minor;
  
  /* Compute the version Windows reports via Int 2Fh/AX=1600h.  */
! static void
  get_windows_version(void)
  {
    if (!_windows_major)
--- 589,595 ----
  unsigned short _windows_major, _windows_minor;
  
  /* Compute the version Windows reports via Int 2Fh/AX=1600h.  */
! static void __attribute__((constructor))
  get_windows_version(void)
  {
    if (!_windows_major)