Re: Bug: listing fonts with missing BDF properties

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


Patrick Lam <plam@xxxxxxx> さんは書きました:

> Mike FABIAN wrote:
>> +		goto bail1;
> I think that should be bail1, because cs isn't freed yet.  I've
> committed with a goto to bail2.

I see.

Then what about the two "bail1" a few lines up?:

    if (!(face->face_flags & FT_FACE_FLAG_SCALABLE))
    for (i = 0; i < face->num_fixed_sizes; i++)
        if (!FcPatternAddDouble (pat, FC_PIXEL_SIZE,
                                           FcGetPixelSize (face, i)))
                                                          goto bail1;
(!FcPatternAddBool (pat, FC_ANTIALIAS, FcFalse))
                       goto bail1;

Shouldn't those be "bail2" as well?

Mike FABIAN   <mfabian@xxxxxxx>

Fontconfig mailing list

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

  Powered by Linux