Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for Text #80

Open
EarthCitizen opened this issue Mar 26, 2018 · 3 comments
Open

Support for Text #80

EarthCitizen opened this issue Mar 26, 2018 · 3 comments

Comments

@EarthCitizen
Copy link

Many applications use Text as the "string" type instead of String. It would be great if Haskeline directly supported IO with Text.

@srghma
Copy link

srghma commented Jun 16, 2019

@judah how would you implement this?

would you:

  1. just change String to Text
  2. use IsString everywhere

@bjartur
Copy link

bjartur commented Sep 27, 2019

  1. Just use the generalized string functions from the strings package.

Is library support for Text & Text.Lazy purely a matter of improving the ergonomics of Text and Text.Lazy? So that users can use Text and Text.Lazy consistently, avoiding performance problems in less interactive code paths? Or has anyone found Haskeline noticeably slow?

@odanoburu
Copy link

Or has anyone found Haskeline noticeably slow?

personally, no! but I'd say it's a matter of the rest of the application using Text (which might be for performance reasons), and it being cumbersome/boring/slower to convert from String to Text

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants