editor/src/term.rs, branch mainA terrible text editorhttps://code.roseh.moe/editor/atom/src/term.rs?h=main2024-01-13T20:31:42ZReject files with invalid utf-8.2024-01-13T20:31:42ZRose Hogensonrosehogenson@posteo.net2024-01-13T20:31:42Zurn:sha1:1e32f5d480ecefa3a1c8fd1c43ac6b4260459266
It's easier to only support files with valid utf-8.
Support the delete key.2024-01-12T16:47:02ZRose Hogensonrosehogenson@posteo.net2024-01-12T16:47:02Zurn:sha1:b0e2ef021ce6f9c157356b896cf089b029cf8c2b
For now it's exactly the same as backspace.
Allow 4~ for End.2024-01-12T16:41:32ZRose Hogensonrosehogenson@posteo.net2024-01-12T16:41:32Zurn:sha1:8fd566044e3a18110161f429a06df9dbb779815b
This seems to be what GNU screen provides, at least.
Fix a bug where cursor_pos would return old data.2024-01-12T16:31:12ZRose Hogensonrosehogenson@posteo.net2024-01-12T16:31:12Zurn:sha1:d333343b15aab5747ccf74c495cb085b4e32b029
For now I'm just counting the failures, and aborting if cursor_pos
immediately failed before. We have to skip the incoming status reports
with read_key before cursor_pos will start working again. This should
flush any old status reports, and prevent reading them incorrectly.
Implement Home, End, PgUp, and PgDn.2024-01-12T05:31:39ZRose Hogensonrosehogenson@posteo.net2024-01-12T00:38:04Zurn:sha1:e23b1c7254aa21c0eb2fd18f720df262725c6d81Apply a clippy suggestion.2024-01-12T04:08:57ZRose Hogensonrosehogenson@posteo.net2024-01-12T04:08:57Zurn:sha1:e8685c54228ca3337060520f7fbeb5629281764cDon't call read immediately in cursor_pos.2024-01-12T03:08:29ZRose Hogensonrosehogenson@posteo.net2024-01-12T03:08:29Zurn:sha1:0f01c2aa5cca178ca2c46e512931302b069886af
I forgot that read can block. We actually don't want cursor_pos to block
for user input before querying, since it will fail nearly every time. It
did give me a good opportunity to test the error handling though.
Don't read the wrong value in cursor_pos.2024-01-12T02:48:07ZRose Hogensonrosehogenson@posteo.net2024-01-12T02:48:07Zurn:sha1:f842e04c4b5579342c1e74b41e867d02e1727f3d
It seems like sometimes a slow terminal like lxterminal can take a
little while to respond with the cursor position, so the read can time
out. If it does, a later call to cursor_pos could read it in. To prevent
this, we can call read before even querying the position. If there are
any previous pending status responses, this should hopefully read them
in, and we can abort early without getting confused. If there is no
buffered data on stdin, then it's hopefully safe to write the status
query and read the response.
Rewrite the entire thing.2024-01-11T08:20:11ZRose Hogensonrosehogenson@posteo.net2024-01-11T08:20:11Zurn:sha1:db087d43659f0760a02d6c32dc7dc0baf2f914f0
They say you can always do a better job on the rewrite, and to a certain
extent that was true here. I kept a lot of it the same though, since
there was a lot I liked from the original design. The main improvements
are in efficiency and code clarity.
Support backspace in xterm.2024-01-07T23:26:04ZRose Hogensonrosehogenson@posteo.net2024-01-07T23:26:04Zurn:sha1:57f527f2a1d9bc4ed8c862b014867634803831ba
Backspace:
Cooler backspace: