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

Fix CPR behavior #129

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@saitoha
Contributor

saitoha commented Jun 20, 2013

This patch fixes the following behavior.

  • CPR report should not respect origin mode
$ echo -e "\033c\033[?6h\033[5;10r\033[2;2H\033[6n" && cat
^[[2;2R

It should be "^[[6;2R"
I think it was influenced by Terminal.app's curious behavior.
I have already sent the bug report to Apple for this issue.

@saitoha

This comment has been minimized.

Show comment
Hide comment
@saitoha

saitoha Jun 22, 2013

Contributor

Today I received the response from Apple support.
He says it's correct, or rather xterm's behavior is incorrect.
The basis is this page: http://www.vt100.net/docs/vt100-ug/chapter3.html

The numbering of lines depends on the state of the Origin Mode

So I'll close this issue. Thanks.

Contributor

saitoha commented Jun 22, 2013

Today I received the response from Apple support.
He says it's correct, or rather xterm's behavior is incorrect.
The basis is this page: http://www.vt100.net/docs/vt100-ug/chapter3.html

The numbering of lines depends on the state of the Origin Mode

So I'll close this issue. Thanks.

@saitoha saitoha closed this Jun 22, 2013

@gnachman

This comment has been minimized.

Show comment
Hide comment
@gnachman

gnachman Jun 23, 2013

Owner

Thanks for researching this and congrats on getting a response from Apple!

Owner

gnachman commented Jun 23, 2013

Thanks for researching this and congrats on getting a response from Apple!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment