Re: fontconfig: Branch 'master' - 2 commits

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hmm, the commit message says it all but it looks hard to understand
that from the diff apparently.

a family name is being guessed and added from a filename in this scope
though, familylang wasn't. thus, there wasn't a corresponding
familylang to that.
I'll pay more attention to logs next time. Thanks for pointing that out.

On Mon, Sep 21, 2020 at 3:13 PM Behdad Esfahbod <behdad@xxxxxxxxxx> wrote:
>
> Can you justify the first change beyond what's in the commit message please? A "if ('en')" piece of code in fontconfig sounds *really* suspicious to me.
>
> Thanks,
> behdad
> http://behdad.org/
>
>
> On Mon, Sep 21, 2020 at 12:04 AM GitLab Mirror <gitlab-mirror@xxxxxxxxxxxxxxxxxxxxxx> wrote:
>>
>>  src/fcfreetype.c |   28 +++++++++++++++++++++++-----
>>  1 file changed, 23 insertions(+), 5 deletions(-)
>>
>> New commits:
>> commit dbef9d10de19288e56f58b7c1aa880d447e6ae85
>> Author: Akira TAGOH <akira@xxxxxxxxx>
>> Date:   Mon Sep 21 14:51:22 2020 +0900
>>
>>     Make sure a combination of family and familylang is available
>>
>> diff --git a/src/fcfreetype.c b/src/fcfreetype.c
>> index 3bdf92b..b9bb09f 100644
>> --- a/src/fcfreetype.c
>> +++ b/src/fcfreetype.c
>> @@ -1675,6 +1675,11 @@ FcFreeTypeQueryFaceInternal (const FT_Face  face,
>>             free (family);
>>             goto bail1;
>>         }
>> +       if (!FcPatternObjectAddString (pat, FC_FAMILYLANG_OBJECT, (FcChar8 *) "en"))
>> +       {
>> +           free (family);
>> +           goto bail1;
>> +       }
>>         free (family);
>>         ++nfamily;
>>      }
>> commit 5a291467a78b59bc9240f2d4ca2f14590e2e936c
>> Author: Akira TAGOH <akira@xxxxxxxxx>
>> Date:   Mon Sep 21 14:48:43 2020 +0900
>>
>>     Add Regular style when no meta data available to guess a style
>>
>>     This makes sure that fullname can be constructed at least even if a style is missing
>>     and "Regular" is omitted for fullname so this change won't affect in that case.
>>
>>     Fixes https://gitlab.freedesktop.org/fontconfig/fontconfig/-/issues/259
>>
>> diff --git a/src/fcfreetype.c b/src/fcfreetype.c
>> index 0001f38..3bdf92b 100644
>> --- a/src/fcfreetype.c
>> +++ b/src/fcfreetype.c
>> @@ -1625,13 +1625,26 @@ FcFreeTypeQueryFaceInternal (const FT_Face  face,
>>         ++nfamily;
>>      }
>>
>> -    if (!variable && !nstyle && face->style_name &&
>> -       FcStrCmpIgnoreBlanksAndCase ((FcChar8 *) face->style_name, (FcChar8 *) "") != 0)
>> +    if (!variable && !nstyle)
>>      {
>> -       if (FcDebug () & FC_DBG_SCANV)
>> -           printf ("using FreeType style \"%s\"\n", face->style_name);
>> +       const FcChar8 *style_regular = (const FcChar8 *) "Regular";
>> +       const FcChar8 *ss;
>>
>> -       if (!FcPatternObjectAddString (pat, FC_STYLE_OBJECT, (FcChar8 *) face->style_name))
>> +       if (face->style_name &&
>> +           FcStrCmpIgnoreBlanksAndCase ((FcChar8 *) face->style_name, (FcChar8 *) "") != 0)
>> +       {
>> +           if (FcDebug () & FC_DBG_SCANV)
>> +               printf ("using FreeType style \"%s\"\n", face->style_name);
>> +
>> +           ss = (const FcChar8 *) face->style_name;
>> +       }
>> +       else
>> +       {
>> +           if (FcDebug () & FC_DBG_SCANV)
>> +               printf ("applying default style Regular\n");
>> +           ss = style_regular;
>> +       }
>> +       if (!FcPatternObjectAddString (pat, FC_STYLE_OBJECT, ss))
>>             goto bail1;
>>         if (!FcPatternObjectAddString (pat, FC_STYLELANG_OBJECT, (FcChar8 *) "en"))
>>             goto bail1;
>> _______________________________________________
>> Fontconfig mailing list
>> Fontconfig@xxxxxxxxxxxxxxxxxxxxx
>> https://lists.freedesktop.org/mailman/listinfo/fontconfig
>
> _______________________________________________
> Fontconfig mailing list
> Fontconfig@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/fontconfig



-- 
Akira TAGOH
_______________________________________________
Fontconfig mailing list
Fontconfig@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/fontconfig



[Index of Archives]     [Fedora Fonts]     [Fedora Users]     [Fedora Cloud]     [Kernel]     [Fedora Packaging]     [Fedora Desktop]     [PAM]     [Gimp Graphics Editor]     [Yosemite News]

  Powered by Linux