microsoft/Terminal

Followups from the new C++/WinRT

Open

#14.872 geöffnet am 18. Feb. 2023

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)C++ (3.212 Forks)batch import
Area-CodeHealthHelp WantedIssue-TaskProduct-Meta

Repository-Metriken

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

Beschreibung

Interesting things we could do after #14869 landed:

### Tasks
- [x] remove all `InitializeComponent` calls - they do it automatically
- [ ] use `std::optional`<->`IReference` automatic binding
- [x] use `std::format` support (!) for json/uri/hostname/http stuff/all `IStringable`s
- [ ] potentially move to `/await:strict` for C++20 coroutines

Contributor Guide

Research-Richtung
Untersuche Methoden, die Windows::Foundation::IReference<T> zurückgeben. Ändere sie so, dass sie std::optional<T> zurückgeben. Stelle sicher, dass die automatische Bindung zwischen std::optional und IReference in C++/WinRT funktioniert, indem du <winrt/Windows.Foundation.h> oder die entsprechenden Header einbindest. Aktualisiere Aufrufer, die IReference erwarten, um std::optional zu verwenden. Teste die Kompilierung und das Verhalten.
Tech Stack
cpp
Domain
desktop
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++C++/WinRT
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60