golang/go

cmd/cgo: _GoString_ is a reserved identifier

Ouverte

#29 877 ouverte le 22 janv. 2019

Voir sur GitHubGuide contributeur
 (5 commentaires) (0 réaction) (0 personne assignée)Go (19 008 forks)batch import
NeedsFixcompiler/runtimehelp wanted

Métriques du dépôt

Stars
 (133 883 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

(Removing the template since this is not really a code bug or so.)

https://golang.org/cmd/cgo/#hdr-Go_references_to_C explains that there's a type _GoString_. However, that name is reserved (see https://en.cppreference.com/w/c/language/identifier#Reserved_identifiers), and using it is technically undefined behavior. I'd suggest renaming the type to just GoString in future versions, with the old name as undocumented and deprecated alias.

Guide contributeur

Direction de recherche
Examinez l'implémentation de cgo dans l'arborescence des sources Go pour trouver toutes les utilisations de ` GoString ` et proposez de le renommer en `GoString` tout en conservant un alias obsolète pour la rétrocompatibilité.
Stack technique
go
Domaine
backend
Type d'issue
Refactorisation
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.
Une demi journée
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
GoCcgo
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50