icu header (unistr.h) is not included when UnicodeString is used; compile error with ICU 59-to-be

googlefonts / sfntly

A Library for Using, Editing, and Creating SFNT-based Fonts

451 stars 162 forks source link

icu header (unistr.h) is not included when UnicodeString is used; compile error with ICU 59-to-be #72

Closed jungshik closed 7 years ago

jungshik commented 7 years ago

cpp/src/sample/chromium/subsetter_impl.cc uses icu::UnicodeString, but does not include unicode/unistr.h leading to a compile failure with ICU-59-to-be.