search

unicode-rs / unicode-xid

Other
49 stars 28 forks source link

Find a location for Pattern_White_Space #1

Open erickt opened 8 years ago

erickt commented 8 years ago

rustc_unicode recently added support for Pattern_White_Space in https://github.com/rust-lang/rust/commit/24578e0fe555f267bef40528b8ac79bc7e898007. Syntex needs this support somewhere, and @kwantam mentioned that unicode-xid might be the right place for this functionality.

cc @SimonSapin

kwantam commented 8 years ago

@erickt thanks for raising this issue!

@SimonSapin my vote would be to insert this table in xid for now, and then when we eventually get around to making an ur--Unicode crate with all the tables, we can relocate it there.

The reason to put it in xid is that, roughly speaking, xid and pattern_white_space predicates are often both used in parsing, thus xid is a better match than any of the other existing crates; and I don't think it's necessary or productive to create an entirely new crate for pattern_white_space.

Thoughts?