Implement Eq and maybe Ord for AugmentScriptSet

unicode-rs / unicode-security

Detect possible security problems with Unicode usage according to Unicode Technical Standard #39 rules.

Other

14 stars 4 forks source link

Implement Eq and maybe Ord for AugmentScriptSet #15

Closed crlf0710 closed 4 years ago

crlf0710 commented 4 years ago

Eq is necessary for using in hashmaps, and for rustc reproducible builds, it will be even better if it can also implement Ord so we can use it in BTreeMap.

crlf0710 commented 4 years ago

Eq is added in #17 . Not sure yet about the Ord part, maybe we can see whether we can sort the Span instead as a workaround.

crlf0710 commented 4 years ago

It turns on Ord part is not needed. closing.