[PR #6328] [MERGED] Add initial support for VT DCS sequences #26653

Open
opened 2026-01-31 09:17:22 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/6328
Author: @skyline75489
Created: 6/3/2020
Status: Merged
Merged: 8/17/2020
Merged by: @DHowett

Base: masterHead: feature/init-dcs-support


📝 Commits (10+)

📊 Changes

5 files changed (+678 additions, -44 deletions)

View changed files

📝 .github/actions/spell-check/dictionary/dictionary.txt (+2 -0)
📝 src/terminal/parser/OutputStateMachineEngine.cpp (+6 -0)
📝 src/terminal/parser/stateMachine.cpp (+447 -42)
📝 src/terminal/parser/stateMachine.hpp (+20 -1)
📝 src/terminal/parser/ut_parser/OutputEngineTest.cpp (+203 -1)

📄 Description

As the title suggests, this commit adds initial support for the VT DCS
sequences. The parameters are parsed but not yet used. The pass through
data is yet to be handled. This effectively fixes #120 by making Sixel
graphics sequences ignored instead of printed.

Tests added.

References #448
Closes #120


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/microsoft/terminal/pull/6328 **Author:** [@skyline75489](https://github.com/skyline75489) **Created:** 6/3/2020 **Status:** ✅ Merged **Merged:** 8/17/2020 **Merged by:** [@DHowett](https://github.com/DHowett) **Base:** `master` ← **Head:** `feature/init-dcs-support` --- ### 📝 Commits (10+) - [`4cf85bb`](https://github.com/microsoft/terminal/commit/4cf85bb741390cadd96e2e9fcc1fb0d9bb953a5b) Init DCS support - [`a0a98b2`](https://github.com/microsoft/terminal/commit/a0a98b2d57e0622bea12e741d35c55afda4e2708) More DCS support - [`6bc3198`](https://github.com/microsoft/terminal/commit/6bc31983b00de28de381a0972985d2d27160fd7d) Spelling - [`5261fe2`](https://github.com/microsoft/terminal/commit/5261fe2581610ed7f164b3ee6e88acbeca985c23) Merge branch 'master' into feature/init-dcs-support - [`d76eedd`](https://github.com/microsoft/terminal/commit/d76eedd647ec058c9deefc8e905d99cbdeff420f) Various fix & test cases - [`1b7fdf9`](https://github.com/microsoft/terminal/commit/1b7fdf9c91f0315085aedc9f3571099727c270fa) More tests - [`1c8d197`](https://github.com/microsoft/terminal/commit/1c8d19759eb2a6d9c28fba943e71a995f0810294) Format - [`3d428fa`](https://github.com/microsoft/terminal/commit/3d428fa20b8c9b276d57c87f4b4efe52460223a6) CI failure - [`eeb07b7`](https://github.com/microsoft/terminal/commit/eeb07b723242be54eaa65614434ff1a1c4f3c835) Merge remote - [`f083f62`](https://github.com/microsoft/terminal/commit/f083f6202e95c03e2ee01383818fe252f71cc68e) j4james rocks! ### 📊 Changes **5 files changed** (+678 additions, -44 deletions) <details> <summary>View changed files</summary> 📝 `.github/actions/spell-check/dictionary/dictionary.txt` (+2 -0) 📝 `src/terminal/parser/OutputStateMachineEngine.cpp` (+6 -0) 📝 `src/terminal/parser/stateMachine.cpp` (+447 -42) 📝 `src/terminal/parser/stateMachine.hpp` (+20 -1) 📝 `src/terminal/parser/ut_parser/OutputEngineTest.cpp` (+203 -1) </details> ### 📄 Description As the title suggests, this commit adds initial support for the VT DCS sequences. The parameters are parsed but not yet used. The pass through data is yet to be handled. This effectively fixes #120 by making Sixel graphics sequences *ignored* instead of printed. * https://vt100.net/docs/vt510-rm/chapter4.html * https://vt100.net/emu/dec_ansi_parser Tests added. References #448 Closes #120 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 09:17:22 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#26653