all_fonts throws too much information away

[CryZe]

DirectWrite, Chrome and many other applications consider variable font instances to be different font faces. However the all_fonts iterator gets completely confused by this.

When it accesses the font count here, this is includes the font instances:

font-kit/src/sources/directwrite.rs

Line 43 in 1bfbc93

for font_index in 0..dwrite_family.get_font_count() {

The dwrite_font then still understands that we are talking about the specific font instance index, not a font face index:

font-kit/src/sources/directwrite.rs

Line 44 in 1bfbc93

let dwrite_font = dwrite_family.get_font(font_index);

However when it gets turned into the font handle:

font-kit/src/sources/directwrite.rs

Line 45 in 1bfbc93

handles.push(self.create_handle_from_dwrite_font(dwrite_font))

It turns the whole thing into a path + font face index pair, which completely throws away all the font instance information, meaning that for certain fonts, the iterator contains the same handle many times.

I'd say the iterator should either iterate over pairs of the handle and the font instance name or index, or that information should be part of the handle.