golang/go

os: document the Process.Pid field

Fechada

#36.726 aberto em 24 de jan. de 2020

Ver no GitHubGuia do colaborador
 (3 comentários) (0 reação) (0 responsável)Go (19.008 forks)batch import
DocumentationNeedsFixhelp wanted

Métricas do repositório

Stars
 (133.883 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

As discussed in https://groups.google.com/forum/#!topic/golang-nuts/ievQGPz1R7k, https://golang.org/cl/4029053 dropped the documentation for the Pid field.

I also propose to extend the documentation with something like:

The value of Pid after the process has been waited/released is not specified.

The reason is that on UNIX systems, Release set Pid to -1, but not on Windows. This means that a program can not check if Pid == -1 to determine if the process has been released.

The alternative is to change the Windows implementation.

Guia do colaborador

Direção de pesquisa
Investigar a implementação atual de Process.Pid no Unix e Windows. Revisar o código em os/exec.go ou os/process.go para entender o comportamento após Release. Verificar o CL mencionado (https://golang.org/cl/4029053) e a thread do golang nuts. Garantir que a documentação esclareça que Pid pode não ser confiável após a liberação.
Pilha de tecnologia
go
Domain
documentation
Tipo Issue
Documentação
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.
Claro
Prerequisites
GoGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70