golang/go

os: document the Process.Pid field

Chiusa

#36.726 aperta il 24 gen 2020

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (0 assegnatari)Go (19.008 fork)batch import
DocumentationNeedsFixhelp wanted

Metriche repository

Star
 (133.883 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Indagare l'implementazione corrente di Process.Pid su Unix e Windows. Esaminare il codice in os/exec.go o os/process.go per comprendere il comportamento dopo Release. Controllare il CL menzionato (https://golang.org/cl/4029053) e la discussione su golang nuts. Assicurarsi che la documentazione chiarisca che Pid potrebbe non essere affidabile dopo il rilascio.
Tech stack
go
Dominio
documentation
Tipo issue
Documentazione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GoGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70