From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.devel Subject: Re: [PING] ClearType Patch Date: Wed, 08 Jun 2005 21:22:30 +0300 Message-ID: References: <1117787455.42a0153fdf46f@webmail.freedom2surf.net> <1118072836.42a47004de422@webmail.freedom2surf.net> Reply-To: Eli Zaretskii NNTP-Posting-Host: main.gmane.org X-Trace: sea.gmane.org 1118254923 31372 80.91.229.2 (8 Jun 2005 18:22:03 GMT) X-Complaints-To: usenet@sea.gmane.org NNTP-Posting-Date: Wed, 8 Jun 2005 18:22:03 +0000 (UTC) Cc: emacs-devel@gnu.org Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Wed Jun 08 20:22:02 2005 Return-path: Original-Received: from lists.gnu.org ([199.232.76.165]) by ciao.gmane.org with esmtp (Exim 4.43) id 1Dg59L-0004qx-Pg for ged-emacs-devel@m.gmane.org; Wed, 08 Jun 2005 20:19:36 +0200 Original-Received: from localhost ([127.0.0.1] helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Dg5G7-0006Iw-6T for ged-emacs-devel@m.gmane.org; Wed, 08 Jun 2005 14:26:35 -0400 Original-Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1Dg5F7-0005mM-Jp for emacs-devel@gnu.org; Wed, 08 Jun 2005 14:25:33 -0400 Original-Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1Dg5F4-0005k8-QO for emacs-devel@gnu.org; Wed, 08 Jun 2005 14:25:31 -0400 Original-Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Dg5F4-0005Ue-0Q for emacs-devel@gnu.org; Wed, 08 Jun 2005 14:25:30 -0400 Original-Received: from [192.114.186.24] (helo=legolas.inter.net.il) by monty-python.gnu.org with esmtp (Exim 4.34) id 1Dg5Fq-0006iH-16 for emacs-devel@gnu.org; Wed, 08 Jun 2005 14:26:18 -0400 Original-Received: from HOME-C4E4A596F7 (IGLD-84-228-165-31.inter.net.il [84.228.165.31]) by legolas.inter.net.il (MOS 3.5.8-GR) with ESMTP id EOK96495 (AUTH halo1); Wed, 8 Jun 2005 21:22:30 +0300 (IDT) Original-To: David Abrahams In-reply-to: (message from David Abrahams on Wed, 08 Jun 2005 11:46:58 -0400) X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:38358 X-Report-Spam: http://spam.gmane.org/gmane.emacs.devel:38358 > From: David Abrahams > Cc: emacs-devel@gnu.org > Date: Wed, 08 Jun 2005 11:46:58 -0400 > > Sorry, but I have a lot on my plate, so once I start waiting a few > more days it's likely to drop off my radar screen. We all need to cope with this. > Agreed, but I think the use of -D_WIN32_WINNT=0x0400 on the compiler > command-line may be similarly bad (and is the reason I transcribed > those constants). Jason, could you perhaps tell why this compiler switch is used? If that is because we don't want to use features that are unavailable in older versions of Windows, we could replace that with run-time checks, like the one I suggest below, can't we? > SPI_GETFONTSMOOTHING > > This one is in fact available for _WIN32_WINNT == 0x400; I should > have used it directly. > > SPI_GETFONTSMOOTHINGTYPE > FE_FONTSMOOTHINGCLEARTYPE > > These two are only enabled for _WIN32_WINNT >= 0x0501 > > > > where they are defined, > > WinUser.h, IIRC The MinGW version of winuser.h doesn't have them; perhaps that's a MinGW bug. So I think we need to condition the code on SPI_GETFONTSMOOTHINGTYPE being defined, or perhaps provide a fallback definition in src/s/ms-w32.h for MinGW. > > Finally, MSDN seems to indicate that SPI_GETFONTSMOOTHINGTYPE is > > unsupported on Windows 9x, so it sounds like your code should be > > disabled for those versions of Windows. > > Right, I should use an #ifdef _WIN32_WINNT >= 0x0501 ... except of > course that the build process subverts that by forcing _WIN32_WINNT to > 0x400. I meant a run-time check, not a compile-time check (because Emacs built on XP could then be run on Windows 98, for example: think binary releases). We have a global variable os_subtype which can be used for such a run-time test, I think. > > A ChangeLog entry for the patches would also be nice, btw. > > I will be happy to write one. I assume the format to follow is used > in the ChangeLog file at the top of emacs' CVS tree? Yes. And in addition, the GNU Coding Standards (standards.texi) which you can find on the GNU Web site, has a section about writing ChangeLog entries. TIA