microsoft/Terminal

[Audit] Unsuppress OuptutCellIterator warning and solve custom construction/destruction warning

Offen

#2.675 geöffnet am 05.09.2019

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (9.275 Forks)batch import
Area-CodeHealthArea-OutputHelp WantedIssue-TaskProduct-ConhostProduct-Terminal

Repository-Metriken

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

Beschreibung

I suppressed the warning on the OutputCellIterator as a part of turning on audit mode because I couldn't figure it out (in a reasonable amount of time for how many other things I was fixing and trying to get turned on).

This represents removing the warning suppression and figuring out what the actual issue is that is causing the warning to flag when the iterator is advanced.

Contributor Guide

Research-Richtung
Untersuchen Sie die unterdrückte Warnung in OutputCellIterator und identifizieren Sie, warum der erweiterte Iterator die Warnung auslöst. Beheben Sie dann die Warnung oder implementieren Sie eine ordnungsgemäße Initialisierung/Zerstörung.
Tech Stack
cpp
Domain
cli
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
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.
65