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.

Sign up for GitHub

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

Jump to bottom

vt100 support #2449

Merged
merged 1 commit into from
Jul 4, 2017
Merged

vt100 support #2449

merged 1 commit into from
Jul 4, 2017

Conversation

payonel
Copy link
Member

@payonel payonel commented Jul 3, 2017 •
edited
Loading

vt100 support

  • huge terminal performance boost
  • echo -e option, interpret backslashes
  • favor io.write instead of print on low level code as print is slightly more memory expensive
  • use vt100 ansi color coding for PS1
  • use tty as std io back end stream
  • consolidate buffer delay code
  • add ctrl+backspace and ctrl+w options in io.read
  • delay load more
  • term code cleanup
  • add buffer:size(), returns size of pending data on the stream
  • memory free 59k
  • openos 1.6.7

@payonel
vt100 support
0ae2ac1 
* huge terminal performance boost
* echo -e option, interpret backslashes
* favor io.write instead of print on low level code as print is slightly more memory expensive
* use vt100 ansi color coding for PS1
* use tty as std io back end stream
* consolidate buffer delay code
* add ctrl+backspace and ctrl+w options in io.read
* delay load more
* term code cleanup
* add buffer:size(), returns size of pending data on the stream
* memory free 59k
* openos 1.6.7
@payonel
Copy link
Member Author

payonel commented Jul 4, 2017 •
edited
Loading

Gifs demonstrating the terminal performance boost
It may appear that frames are missing in the openos 1.6.7 gif below, they are not, it's just that much faster

openos-1 6 6
openos-1 6 7

@payonel payonel merged commit 0527390 into MightyPirates:master-MC1.7.10 Jul 4, 2017
@payonel payonel deleted the vt100 branch July 4, 2017 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@payonel