[uforead] uforead fails to set advance width when height is also defined in the glif file

[punchcutter]

In a glif file that has both advance height and width defined like <advance height="1000" width="1000"/> uforead fails to read the width and sets it to 0. If width comes first then it gets read. We need to check for height as well. Right now we can just ignore it because width is more important, but correctly using height is something that can be dealt with later (more info in fonttools/fonttools#2704 about using advance height from UFO to set vmtx/VORG).

afdko/c/shared/source/uforead/uforead.c

Line 1685 in c02a68d

if (xmlAttrEqual(attr, "width") || xmlAttrEqual(attr, "advance"))

[hatchzo]

I saw the same problem, when height value comes first, as in <advance height="1000" width="1000"/>, width value is ignored.

[kaydeearts]

Resolved by #1720.