Date: Fri, 17 Aug 2001 14:25:11 +0300
From: "Eli Zaretskii" <eliz AT is DOT elta DOT co DOT il>
Sender: halo1 AT zahav DOT net DOT il
To: pavenis AT lanet DOT lv
Message-Id: <9791-Fri17Aug2001142511+0300-eliz@is.elta.co.il>
X-Mailer: Emacs 20.6 (via feedmail 8.3.emacs20_6 I) and Blat ver 1.8.9
CC: djgpp-workers AT delorie DOT com
In-reply-to: <3B7D1C46.23448.59E465@localhost> (pavenis@lanet.lv)
Subject: Re: _open.c commit?  (was Re: Selector Exhaustion)
References: <10108162209 DOT AA13700 AT clio DOT rice DOT edu> (sandmann AT clio DOT rice DOT edu) <3B7D1C46 DOT 23448 DOT 59E465 AT localhost>
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

> From: pavenis AT lanet DOT lv
> Date: Fri, 17 Aug 2001 13:29:42 +0300
> > 
> > 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?

What for?  I know that _windows_major is not initialized in the
current CVS.

>   /* Compute the version Windows reports via Int 2Fh/AX=1600h.  */
> ! static void __attribute__((constructor))
>   get_windows_version(void)

I'd prefer simply to call get_windows_version in the startup code (and
rename it to __get_windows_version or something).  But let's first
wait to see if the change I suggested, which is reason for changing
get_windows_version, is indeed accepted by others.