microsoft/Terminal

Documentation of use of ConPTY together with STARTF_USESTDHANDLES is lacking.

Open

#4 585 ouverte le 14 févr. 2020

Voir sur GitHub
 (2 commentaires) (2 réactions) (0 assignés)C++ (3 212 forks)batch import
Area-ServerHelp WantedIssue-DocsProduct-Conpty

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

All existing documentation of ConPTY seem to only mention the case where stdin, stdout and stderr are all to be connected to the pty and no STARTF_USESTDHANDLES. The only mention that the proper value to have in StartupInfo.hStdInput et al when STARTF_USESTDHANDLES is active is NULL is indirectly in the documentation for GetStdHandle().

Using STARTF_USESTDHANDLES is useful if you want to keep stdout and stderr separate or want to redirect one of them to eg a file.

An explicit mention in conjunction with STARTUPINFOA and/or PROC_THREAD_ATTRIBUTE_PSEUDOCONSOLE would be appreciated.

Thanks.

Guide contributeur

Direction de recherche
Localisez la documentation ConPTY dans le depot et ajoutez une note sur l'interaction avec STARTF USESTDHANDLES, en precisant que hStdInput etc. doivent etre NULL lors de l'utilisation de ce drapeau.
Stack technique
Aucun
Domaine
documentationapi
Type d'issue
Documentation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
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
Windows APIConPTY
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60