open-i18n / rust-unic

UNIC: Unicode and Internationalization Crates for Rust

Home Page:https://crates.io/crates/unic

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

ucd/ident: Add test for Pattern Syntax/Whitespace data

declanvk opened this issue · comments

Hello,

I noticed that the pattern whitespace property is incorrectly using the pattern_syntax.rsv table.

pub struct PatternWhitespace(bool) {
abbr => "Pat_WS";
long => "Pattern_White_Space";
human => "Pattern Whitespace";
data_table_path => "../tables/pattern_syntax.rsv";
}

I've opened a PR with the simple substitution, #253

Thanks,
Declan

Thanks, @declanvk, for filing the issue!

Unfortunately we haven't had unit tests for these tables, hence not catching the problem earlier. Fix is being landed in #255, but let's keep this open to also add some basic unit tests!

I'm going to make a release, v0.9.0, soon, to deploy the fix for this.