microsoft/Terminal

Investigate and ensure `wt.exe` is always `argv[0]`

Open

#4 170 ouverte le 9 janv. 2020

Voir sur GitHub
 (5 commentaires) (0 réactions) (0 assignés)C++ (3 212 forks)batch import
Area-CodeHealthHelp WantedIssue-TaskProduct-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

I have seen Windows executables receive arguments BOTH ways. I don't know what the circumstance is offhand that makes it happen one way or the other. You will want to deal with both the presence and absence of your own executable name at argv[0].

Originally posted by @miniksa in https://github.com/microsoft/terminal/pull/4023

If it's not, then we'll need to mitigate this somehow, since we're leaning on that pretty hard at the moment

Guide contributeur

Direction de recherche
Étudiez le comportement de argv[0] sur Windows pour wt.exe. Vérifiez si Windows omet parfois le nom de l'exécutable de argv[0] et dans quelles conditions. Reportez vous au commentaire de @miniksa dans la PR #4023. Examinez comment le terminal utilise actuellement argv[0] et envisagez d'utiliser GetCommandLineW() pour récupérer de manière fiable le chemin de l'exécutable. Assurez vous que si argv[0] n'est pas le nom de l'exécutable, le code puisse toujours fonctionner correctement.
Stack technique
cpp
Domaine
cli
Type d'issue
Recherche
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
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.
Investigation requise
Prérequis
C++Windows
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50