BurntSushi / ucd-generate

A command line tool to generate Unicode tables as source code.
Apache License 2.0
95 stars 21 forks source link

Add option for generating case mapping tables #17

Closed thomcc closed 4 years ago

thomcc commented 4 years ago

I did this at roughly the same time as #16 and in my head for some reason I had thought it would be hard to rebase to not include the C stuff. I don't know why I thought that (AFAICT there would be no merge conflict, and even if there was, it looks like I did it months ago anyway. EDIT: Ah, actually there's a patch somewhere that goes on top of what's here for C output of this stuff I think. If you accept the C output patch I'll go dig that up too).

That said, there's less of a need for this in Rust as it's in libcore anyway (C on the other hand...).

It skips the conditional rules, which might be useful for some cases, but I'm not sure what code that handles them tends to look like, so I'm not sure what representation is convenient/efficient.

Alternatively if you have an opinion on how conditional rules should be output, let me know and I can do that as another flag maybe.

BurntSushi commented 4 years ago

Apologies, I ended up getting a release out today and I ran rustfmt. Hopefully a rebase isn't too messy.

thomcc commented 4 years ago

FYI I believe I've made the requested changes.

BurntSushi commented 4 years ago

This is on crates.io in ucd-generate 0.2.5.