Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

readline: handle input starting with control chars #5509

Closed
wants to merge 1 commit into from
Closed

readline: handle input starting with control chars #5509

wants to merge 1 commit into from

Conversation

eschrock
Copy link

Handle control characters only when there is a single byte in the
stream, otherwise fall through to the standard multibyte handling.

Fixes #5508

Handle control characters only when there is a single byte in the
stream, otherwise fall through to the standard multibyte handling.
@Nodejs-Jenkins
Copy link

Thank you for contributing this pull request! Here are a few pointers to make sure your submission will be considered for inclusion.

The following commiters were not found in the CLA:

  • Eric Schrock

Please see CONTRIBUTING.md for more information

@eschrock
Copy link
Author

FYI, I filled out the CLA this morning. Not sure if it hasn't propagated, or fails because I used '@gmail.com' in the CLA but my push uses '@delphix.com'. I will submit a CLA for the latter just in case.

@tjfontaine
Copy link

Ya, the bot generally keys on the commit email, before the push though a human usually verifies so it wouldn't have been a huge issue

@bnoordhuis
Copy link
Member

LGTM.

/cc @TooTallNate

@tjfontaine
Copy link

Thanks! Merged in 35ae696

@tjfontaine tjfontaine closed this Sep 24, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants