protocolbuffers / protocolbuffers/protobuf-javascript
Using JS typed arrays in JoinFloat
Nessuno ha ancora preso questa issue.
- Lingua principale
- JavaScript
- Stelle
- 471
- Fork
- 91
- Merge medio
- 3h 57m
- PR unite (30g)
- 2
Descrizione
Hi,
Has anyone thought of using JS typed arrays in the splitFloat and joinFloat methods for the JavaScript library (https://github.com/google/protobuf/blob/master/js/binary/utils.js#L388)?
One could imagine adding something like:
jspb.utils.joinFloat64 = function(bitsLow, bitsHigh) {
if (window.Uint32Array) {
bytes = Uint8Array.of(bitsLow, bitsHigh)
data = new Float64Array(bytes.buffer)
return data[0];
}
// ...
}
And vica-versa for the splitFloat method. This might increase readability (and depending on if it taps into the JS engine's native implementation, maybe even speed).
Happy to write the PR, just curious if it would be useful / appreciated!
Let me know,
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia in js/binary/utils.js intorno alla riga 388 e leggi le implementazioni di splitFloat e joinFloat e la logica di compatibilità circostante. Valuta se è possibile usare typed arrays in quel punto senza perdere il comportamento di fallback esistente; il lavoro è completo quando entrambi i metodi rimangono corretti in tutti gli ambienti supportati e la modifica è pronta per la revisione.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- javascript
- Ambito
- backend
- Tipo di issue
- Refactoring
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100