protocolbuffers / protocolbuffers/protobuf-javascript
Using JS typed arrays in JoinFloat
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 471
- Forks
- 91
- Merge moyen
- 3 h 57 min
- PR mergées (30 j)
- 2
Description
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,
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez dans js/binary/utils.js vers la ligne 388 et lisez les implémentations de splitFloat et joinFloat ainsi que la logique de compatibilité qui les entoure. Évaluez si des typed arrays peuvent y être utilisés sans perdre le comportement de fallback existant ; le travail est terminé lorsque les deux méthodes restent correctes dans tous les environnements pris en charge et que la modification est prête pour la revue.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript
- Domaine
- backend
- Type d'issue
- Refactorisation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100