On Mon, Mar 18, 2013 at 12:58 PM, Mike Williams
<[email protected]> wrote:
> On 15/03/2013 05:40, mattn wrote:
>>>
>>> attached patch adds a new field called 'a' into guifont spec for
>>> gui_win32. This allows Windows user to control font antialiasing style
>>> in same fashion as Putty or Mintty does.
>>> As this is my first Vim patch and also my first patch for MS Windows
>>> related code I am not sure about MS SDK macro usage policy, precisely
>>> if I can use WINVER check directly in case of CLEARTYPE_QUALITY
>>> constant (which was introduced in 7.0A). If it is not good enough just
>>> tell me how to do it properly and I will rework my patch.
>>
>>
>> Nice. It's working good for me. One thing, for about checking WINVER, I
>> prefer to #ifdef CLEARTYPE_QUALITY. This make be possible to work on
>> WindowsXP.
>
>
> You must have special build runes.  ;-)  WinGDI.h checks for
> _WIN32_WINNT_WINXP as you say but the supplied Windows makefiles set WINVER
> to 0x0400 or 0x0500 which map to NT4 and 2K, so the cleartype flag will not
> be compiled in by default.

Good to know. What about removing the cleartype style from patch
completely. As the most of Win7 and Win8 installations come with
cleartype enabled system-wide it will be still available through the
'default' setting. Such patch will not change build-time compatibility
from the previous state. Also, despite knowing the rule #1 'never
predict what user may want' I assume that user who wants cleartype in
Vim probably wants cleartype everywhere else so 'default' which is
default is ok. Sounds this reasonable to you for retaining the
pre-cleartype Windows compatibility?

Regards,

-- 
Ondrej Balaz

e-mail: [email protected]

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.


Raspunde prin e-mail lui