ekmett / ekmett/intern

`Ord` instance for `InternedText`

Open
#11 0 comments 1 reaction 0 assignees View on GitHub
Dominant language
Haskell
Stars
25
Forks
8
PR merge metrics
No merged PRs in 30d

Description

The `Ord` instance for `InternedText` compares identifiers rather than the underlying text. Presumably this is for efficiency, and is an entirely reasonable choice. However, I definitely think this behavior should be documented, since it is not obvious that a textual datatype will have anything other than the usual lexicographic sort order. People using `InternedText` are, in general, going to treat it as a drop-in hash-consed replacement for `Text`, unless there is documentation to the contrary.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.