microsoft/Terminal

pasting large multiline text is extremelly slow

Open

#9 232 ouverte le 20 févr. 2021

Voir sur GitHub
 (35 commentaires) (27 réactions) (0 assignés)C++ (3 212 forks)batch import
Area-PerformanceArea-TerminalControlHelp WantedIssue-BugPriority-2Product-Terminal

Métriques du dépôt

Stars
 (35 764 stars)
Métriques de merge PR
 (Merge moyen 27j 19h) (24 PRs mergées en 30 j)

Description

Environment

Windows Terminal Preview v1.6.10272.0 on windows 10 20h2

Steps to reproduce

  1. start wt with cmd
  2. launch neovim in terminal mode (no gui)
  3. enter insert mode with
  4. paste a large text, like 200k of json
  5. you get the warning about large text being pasted, accept it
  6. wait....I had to kill wt, not sure how long I would need to wait (many minutes?)

Expected behavior

the text gets pasted instantly

Actual behavior

you have to wait a lot.

I tried pasting the same text in neovim-qt (the official GUI version of neovim) and the text gets inserted instantly.

I understand pasting large text would be weird in normal terminal usage, but when you are using wt to host neovim it is a legit use case.

Guide contributeur

Direction de recherche
Examinez le traitement du collage dans le code de Windows Terminal, en particulier l'analyseur VT et la façon dont les grands blocs de texte sont traités. Cherchez des algorithmes en O(n²) ou une insertion séquentielle de grands textes. Envisagez d'utiliser WriteFile ou d'autres API pour une entrée en masse.
Stack technique
cpp
Domaine
desktop
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++Windows API
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
25