Adds a fix to the readline issues we've been seeing #96
No reviewers
Labels
No Label
bug
compatibility
documentation
duplicate
enhancement
future release
help wanted
invalid
non-code
question
refactor
testing
this release
wontfix
No Milestone
1 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: cmccabe/linkulator2#96
Loading…
Reference in New Issue
No description provided.
Delete Branch "readline-escape-fix"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Readline recognizes
\001
and\002
to designate beginning and ending of text that should be ignored by readline when calculating line length. We had been putting it around any text that had escape codes. This update adds a bool to thestyle_text()
function to signify if the text will be part of an input or not. If not, we do not add the readline bytes (they technically arent escape codes they are just low bytes that it recognizes).I tried it, but ahd not been experiencing the original issue. If you, @cmccabe , want to give it a try and post the results here that would be awesome.