Terminal: ColorTool: Parser-Bugfixes und Refactor für kleinen Code

Erstellt am 21. Apr. 2019  ·  3Kommentare  ·  Quelle: microsoft/terminal

Ich verwende ColorTool als Teil der Verwaltung eines Themas für cmd / Powershell und bin dabei auf einige Probleme gestoßen:

  1. Von nicht verwendeten Parsern werden Fehlermeldungen ausgegeben, z. B. versucht der INI-Parser, ausgeführt zu werden, selbst wenn Sie itermcolors . Daher werden Fehler ausgegeben, da XML keine gültige INI-Datei ist.
  2. Vordergrund- / Hintergrundindizes werden bei Verwendung von --defaults (oder --both ) nicht gespeichert.

Diese Korrekturen sind einfach genug und ich bin glücklich, sie zu beheben. Ich habe darüber nachgedacht, den Code ein wenig zu bereinigen, während ich dort bin, nämlich:

  1. Zulassen, dass Parser bedingt ausgeführt werden, sodass nicht alle Parser für jede Datei ausgeführt werden (sollte einige Fehler unterdrücken)
  2. Hinzufügen einer gemeinsamen Schnittstelle zum "Festlegen der aktuellen Konsoleneigenschaften" und "Konsolenstandards", sodass es schwieriger ist, die Implementierung von Funktionen für beide zu vergessen.

Bevor ich jedoch anfange, daran zu arbeiten, wollte ich Ihnen ein Heads-up geben und sicherstellen, dass ich nicht mit jemandem an dem Projekt zusammengearbeitet habe. Würden Sie sich für eine PR interessieren, die die oben genannten 2 Punkte behebt?

Vielen Dank!

Area-Settings Issue-Bug Product-Colortool Resolution-Fix-Available

Alle 3 Kommentare

@waf , das wäre eine sehr willkommene Abwechslung. Derzeit arbeitet niemand auf unserer Seite daran.

@ DHowett-MSFT danke, ich habe # 418 dafür geöffnet.

Ich glaube, dass dies jetzt in der Veröffentlichung verfügbar ist, die ich vor einigen Wochen gemacht habe.

War diese Seite hilfreich?
0 / 5 - 0 Bewertungen