golang/go

os: document the Process.Pid field

Geschlossen

#36.726 geöffnet am 24.01.2020

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Go (19.008 Forks)batch import
DocumentationNeedsFixhelp wanted

Repository-Metriken

Stars
 (133.883 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuchen Sie die aktuelle Implementierung von Process.Pid unter Unix und Windows. Überprüfen Sie den Code in os/exec.go oder os/process.go, um das Verhalten nach Release zu verstehen. Prüfen Sie den erwähnten CL (https://golang.org/cl/4029053) und den golang nuts Thread. Stellen Sie sicher, dass die Dokumentation klarstellt, dass Pid nach dem Release möglicherweise nicht zuverlässig ist.
Tech Stack
go
Domain
documentation
Issue Type
Dokumentation
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GoGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70