protocolbuffers/protobuf

PHP: C extension has undocumented "append" function that has no package parity.

Aperta

#26.188 aperta il 3 mar 2026

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)C++ (16.128 fork)batch import
help wanted

Metriche repository

Star
 (71.223 stelle)
Metriche merge PR
 (Merge medio 2g 11h) (185 PR mergiate in 30 g)

Descrizione

What version of protobuf and what language are you using? Version: Any/all. Has existed from the introduction of the package Language: PHP

What did you do? https://github.com/protocolbuffers/protobuf/blob/main/php/ext/google/protobuf/array.c#L245-L263

RepeatedField and the now removed Internal\RepeatedField in the C extension have an append() function that has never had a matching append() function in the php-only package, which can cause static analysis and type confusion.

Guida contributor

Direzione di ricerca
Aggiungi la funzione append() al pacchetto PHP puro per corrispondere alla funzione append() dell'estensione C, garantendo coerenza e prevenendo confusione di tipo.
Tech stack
phpc
Dominio
backend
Tipo issue
Bug
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
Nessuno
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70