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

console: use correct escape sequence check #1919

Merged
merged 1 commit into from Nov 12, 2017

Conversation

2 participants
@brauner
Member

brauner commented Nov 11, 2017

Signed-off-by: Christian Brauner christian.brauner@ubuntu.com

console: use correct escape sequence check
Signed-off-by: Christian Brauner <christian.brauner@ubuntu.com>
@hallyn

This comment has been minimized.

Show comment
Hide comment
@hallyn

hallyn Nov 12, 2017

Member

Going by the comment in console.h, only -1 should technically be ignored, though I don't know what 0 would really mean.

Member

hallyn commented Nov 12, 2017

Going by the comment in console.h, only -1 should technically be ignored, though I don't know what 0 would really mean.

@hallyn hallyn merged commit af73a0b into lxc:master Nov 12, 2017

4 checks passed

Branch target Branch target is correct
Details
Signed-off-by All commits signed-off
Details
Testsuite Testsuite passed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment