golang/go

cmd/cgo: _GoString_ is a reserved identifier

Aperta

#29.877 aperta il 22 gen 2019

Vedi su GitHubGuida contributor
 (5 commenti) (0 reazioni) (0 assegnatari)Go (19.008 fork)batch import
NeedsFixcompiler/runtimehelp wanted

Metriche repository

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

Descrizione

(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.

Guida contributor

Direzione di ricerca
Esamina l'implementazione di cgo nell'albero dei sorgenti Go per trovare tutti gli usi di ` GoString ` e proponi di rinominarlo in `GoString` mantenendo un alias deprecato per la compatibilità all'indietro.
Tech stack
go
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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
GoCcgo
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50