golang/go

os: document the Process.Pid field

Fermée

#36 726 ouverte le 24 janv. 2020

Voir sur GitHubGuide contributeur
 (3 commentaires) (0 réaction) (0 personne assignée)Go (19 008 forks)batch import
DocumentationNeedsFixhelp wanted

Métriques du dépôt

Stars
 (133 883 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

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.

Guide contributeur

Direction de recherche
Examiner l'implémentation actuelle de Process.Pid sur Unix et Windows. Examiner le code dans os/exec.go ou os/process.go pour comprendre le comportement après Release. Vérifier le CL mentionné (https://golang.org/cl/4029053) et la discussion golang nuts. S'assurer que la documentation précise que Pid peut ne pas être fiable après la libération.
Stack technique
go
Domaine
documentation
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.
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.
Claire
Prérequis
GoGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70