"Failed to load glyph" for specific characters in specific fonts about msdfgen HOT 11 CLOSED

@LautrecOfCarim This turned out to be a way more complex problem than it seemed inititally, since the contours are supplied in a completely randomized order, but I believe I solved it, so it should work now. It took me two or three afternoons, but it was a very interesting problem. See #6.

from msdfgen.

I also have seen this problem with Helvetica neue lt std on the lower-case and upper-case letter 'o'. It looks like it pulls out of the inner for loop and resets the state variable to NONE and when it resamples face->glyph->outline.tags[index], it comes back with QUADRATIC_POINT and on state==NONE it's REQUIRED(pointType == PATH_POINT). I'm looking deeper into it now.

from msdfgen.

I was able to get the 'o' to work after taking out the REQUIRED(pointType == PATH_POINT). Here's the result...

...I wonder if there's a case that isn't being accounted for, as in a glyph that has a different structure. I will continue researching it. Let me know if that makes sense. Here's a link to the font I'm using.

from msdfgen.

Are you using the latest version? There is no line containing REQUIRE(pointType == PATH_POINT) anymore.

from msdfgen.

@Chlumsky - Did some digging on this with the latest version of the code. Looks like the letter 'o' in some fonts (e.g. Neue Helvetica Pro 55 Roman has no PATH_POINT in the first contour in TTF format. So eventually the REQUIRE macro forces it out of the loop with a failure. But if one comments out the REQUIRED macro, an infinite loop occurs due to no exit of the loop.

from msdfgen.

@Chlumsky, I updated and have gotten the 'o' in Helvetica Neue Bold to work, thanks for that. I'm also seeing the same thing as @chaoticbob.

from msdfgen.

Here's my fix for loadGlyph wrt the issue I mentioned above:

            switch (state) {
                case NONE:
                    if (pointType == PATH_POINT) {
                        firstPathPoint = index;
                        startPoint = point;
                        state = PATH_POINT;
                    } else if((face->glyph->outline.tags[first] == FT_Curve_Tag_Conic) && (face->glyph->outline.tags[last] == FT_Curve_Tag_Conic)) {
                        firstPathPoint = index;
                        Point2 firstPoint( face->glyph->outline.points[first].x/64., face->glyph->outline.points[first].y/64.);
                        Point2 lastPoint( face->glyph->outline.points[last].x/64., face->glyph->outline.points[last].y/64.);
                        startPoint = .5*(firstPoint + lastPoint);
                        controlPoint[0] = point;                        
                        state = QUADRATIC_POINT;
                    }
                    break;

I found this discussion to be helpful for contours that are all quadratic points, aka uses the all quadratic shorthand.

from msdfgen.

You should be using FT_Outline_Decompose() instead of manually decoding the outline.

from msdfgen.

This font also throws that Failed to load glyph from font file error https://fonts.google.com/specimen/Londrina+Solid on some characters for example character a (0x61)

from msdfgen.

Alright, I changed it to FT_Outline_Decompose() as per @behdad's comment. Everything should be fine now.

from msdfgen.

I'm getting similar problems with some characters. The '$' glyph from Crimson-Italic has overlapping contours and doesn't render properly.

from msdfgen.