Re: Take monospace fonts into account in default font selectionalgorithm.

[Huw D M Davies]

On Sat, Aug 07, 2004 at 04:20:11PM +0100, Mike Hearn wrote:
> @@ -1717,9 +1719,12 @@
>   }
>      }
>
> +    /* match on charset and style */
>      if(!family) {
>          for(family = FontList; family; family = family->next) {
> -	    if(csi.fs.fsCsb[0] & family->FirstFace->fs.fsCsb[0])
> +            
> +	    if((csi.fs.fsCsb[0] & family->FirstFace->fs.fsCsb[0]) &&
> +               (family->FirstFace->fixed_width == (lf.lfPitchAndFamily & MONO_FONT ? TRUE : FALSE)))
>                  if(family->FirstFace->scalable || can_use_bitmap)
>                      break;
>   }

This also wouldn't work as you expect.  The way to create a fixed
pitch font is to use FIXED_PITCH not MONO_FONT.  Which app was
setting this flag?

Huw.
-- 
Huw Davies
huw(a)codeweavers.com

Take monospace fonts into account in default font selectionalgorithm.

[Mike Hearn]

On Thu, 2004-08-19 at 12:06 +0100, Huw D M Davies wrote:
> This also wouldn't work as you expect.  The way to create a fixed
> pitch font is to use FIXED_PITCH not MONO_FONT.  Which app was
> setting this flag?

There wasn't one, I assumed MONO_FONT meant monospace, which is what I
was trying to stop Wine selecting. I didn't have MSDN handy at the time
though so it was basically just a guess ...

Take monospace fonts into account in default font selectionalgorithm.

[Mike Hearn]

Mike Hearn wrote:
> > This also wouldn't work as you expect.  The way to create a fixed
> > pitch font is to use FIXED_PITCH not MONO_FONT.  Which app was
> > setting this flag?
>
> There wasn't one, I assumed MONO_FONT meant monospace, which is what I
> was trying to stop Wine selecting. I didn't have MSDN handy at the time
> though so it was basically just a guess ...

If I resubmit this patch using FIXED_PITCH not MONO_FONT (which seems to 
be undocumented) would that be correct?

/me wishing he had MSDN on his laptop ...