<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW --- - Improve size unparse?" href="https://bugs.freedesktop.org/show_bug.cgi?id=80873#c4">Comment # 4</a> on <a class="bz_bug_link bz_status_NEW " title="NEW --- - Improve size unparse?" href="https://bugs.freedesktop.org/show_bug.cgi?id=80873">bug 80873</a> from <a class="email" href="mailto:freedesktop@behdad.org" title="Behdad Esfahbod <freedesktop@behdad.org>"> Behdad Esfahbod</a> <pre>(In reply to <a href="show_bug.cgi?id=80873#c3">comment #3</a>) > I don't like NOT showing something that is set. Why is it set anyway? > > Given that a missing SIZE will match with score 0, and a SIZE of (0,inf) > also always match with score 0, I really prefer that we DON'T set size if > it's not available. > > I did a couple commits to that effect. Please see: > > <a href="https://github.com/behdad/fontconfig/commits/size-rework">https://github.com/behdad/fontconfig/commits/size-rework</a> > > Needs testing. On top of this, I want to see us clean up some more: > > - Remove the bits about range's is_inclusive, > > - Remove special-cases of ranges with start==end, > > - Move the part that sets FC_SIZE for bitmap-only fonts close to where > FC_PIXEL_SIZE is set, or remove it. I don't think we had that part before > and you introduced it during adding ranges. One more: - Accept integer for size. I think in some places we only accept range and double.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are on the CC list for the bug.</li> </ul> </body> </html>