microsoft/Terminal

Move u8State from WriteConsoleAImpl to SCREEN_INFORMATION

Open

#6.982 geöffnet am 20. Juli 2020

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)C++ (3.212 Forks)batch import
Area-CodeHealthHelp WantedIssue-TaskPriority-3Product-Conhost

Repository-Metriken

Stars
 (35.764 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 27T 19h) (24 gemergte PRs in 30 T)

Beschreibung

Environment

Windows build number: [run `[Environment]::OSVersion` for powershell, or `ver` for cmd] 10.0.19041.388
Windows Terminal version (if applicable): 1.0.1811.0 + recent preview

In #4493 @miniksa pointed out that the state should be a member of SCREEN_INFORMATION rather than a static variable in WriteConsoleAImpl. Refer to https://github.com/microsoft/terminal/pull/4493#discussion_r389024806 In the current code base it's still in WriteConsoleAImpl though. https://github.com/microsoft/terminal/blob/d0ff5f6b5e7687c438e0a8b120280be307eda778/src/host/_stream.cpp#L1107 Since I closed the PR I just don't want this information to get lost.

Contributor Guide

Research-Richtung
Beziehen Sie sich auf die PR Diskussion #4493 und den aktuellen Code unter src/host/ stream.cpp. Verschieben Sie die statische Variable u8State von WriteConsoleAImpl in die Klasse SCREEN INFORMATION und stellen Sie sicher, dass alle Verweise aktualisiert werden.
Tech Stack
cpp
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75