mattn / go-runewidth

wcwidth for golang
MIT License
609 stars 94 forks source link

EastAsianWidth.txt, use the record with "F"(FULLWIDTH) field as same as "W" #33

Closed hymkor closed 5 years ago

hymkor commented 5 years ago

For the issue possible regression ? · Issue #32 · mattn/go-runewidth.

On the script/generate.go reading https://unicode.org/Public/12.1.0/ucd/EastAsianWidth.txt , the record which has "F"(FullWidth) field is not used as same as "W". So, I tried to fix to support and update checksums (Is it to remind go generate ?)

Would you like to merge if there are no problems ?

mattn commented 5 years ago

Thank you.

MichaelMure commented 5 years ago

Could you tag a release with this fix ? My problem is that with go 1.10 (before go module), the last release is downloaded which is the broken one at the moment.

mattn commented 5 years ago

Tagged v0.0.6

MichaelMure commented 5 years ago

Thanks :)