microsoft/Terminal

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

Open

#4.170 aberto em 9 de jan. de 2020

Ver no GitHub
 (5 comments) (0 reactions) (0 assignees)C++ (3.212 forks)batch import
Area-CodeHealthHelp WantedIssue-TaskProduct-Terminal

Métricas do repositório

Stars
 (35.764 stars)
Métricas de merge de PR
 (Mesclagem média 27d 19h) (24 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Investigue o comportamento de argv[0] no Windows para wt.exe. Verifique se o Windows omite o nome do executável de argv[0] e em quais condições. Consulte o comentário de @miniksa no PR #4023. Analise como o terminal atualmente usa argv[0] e considere usar GetCommandLineW() para recuperar de forma confiável o caminho do executável. Garanta que, se argv[0] não for o nome do executável, o código ainda possa funcionar corretamente.
Pilha de tecnologia
cpp
Domain
cli
Tipo Issue
Pesquisa
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Precisa de investigação
Prerequisites
C++Windows
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50