Skip to content

Implement DECALN control sequence#27

Draft
viccie30 wants to merge 2 commits intoAetf:mainfrom
viccie30:DECALN
Draft

Implement DECALN control sequence#27
viccie30 wants to merge 2 commits intoAetf:mainfrom
viccie30:DECALN

Conversation

@viccie30
Copy link
Contributor

I'm trying to get libtsm to properly execute the vttest tests. The missing DECALN sequence is simply the first issue I ran into with the first tests.

I have implemented the behavior by reading the DEC STD 070 Video Systems Reference Manual and matching the output with that of xterm.

I plan to work my way through all the tests in vttest, do you think that's worthwhile?

viccie30 added 2 commits July 25, 2022 23:29
This makes it easy to get more information from tests, without
changing `test/test_common.h` and recompiling. If the `CK_VERBOSITY`
environment variable is not set, the default is the same as it was,
i.e. `CK_NORMAL`.
This control sequence is used by vttest and is present since VT100.
@viccie30 viccie30 marked this pull request as draft July 25, 2022 22:50
@viccie30
Copy link
Contributor Author

I've noticed some of these changes, for example resetting margins and moving the cursor to the origin appear often in implementations. Before making more changes, I'll see if I can factor those repetitive parts out.

@Aetf
Copy link
Owner

Aetf commented Jul 26, 2022

This is huge! Thanks for working on this.

I plan to work my way through all the tests in vttest, do you think that's worthwhile?

How "common" is the sequences used in vttest? I think we should aim for it, but if there're some obscure sequences that are too hard to implement, feel free to skip them. (We should keep a list of known unimplemented sequences somewhere in the doc)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants