JS: Handling Infinity values for Float fields

Offen

#49 erstellt am 17.5.2019 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 2/5
Geschätzter Aufwand: 1-3 Stunden
Anfängerfreundlichkeit: 42/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: javascript
Bereich: backend

Rechercherichtung

Beginne in encoder.js bei BinaryEncoder.prototype.writeFloat und untersuche die verknüpfte Implementierung. Verfolge anschließend, wie splitFloat32 und writeUint32 mit Infinity umgehen. Die Aufgabe ist abgeschlossen, wenn das akzeptierte Verhalten für Infinity konsistent mit dem Methodenkommentar implementiert und klar dokumentiert ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug javascript triaged

Hello!

I noticed that in encoder.js, the method does not allow writeFloat does not accept Infinity as its value

https://github.com/protocolbuffers/protobuf/blob/d9ccd0c0e6bbda9bf4476088eeb46b02d7dcd327/js/binary/encoder.js#L365-L375

/**
 * Writes a single-precision floating point value to the buffer. Numbers
 * requiring more than 32 bits of precision will be truncated.
 * @param {number} value The value to write.
 */
jspb.BinaryEncoder.prototype.writeFloat = function(value) {
  goog.asserts.assert((value >= -jspb.BinaryConstants.FLOAT32_MAX) &&
                      (value <= jspb.BinaryConstants.FLOAT32_MAX));
  jspb.utils.splitFloat32(value);
  this.writeUint32(jspb.utils.split64Low);
};

And its comment seems misleading because it says Numbers requiring more than 32 bits of precision will be truncated. but in reality, it throws an error.

Shouldn't writeFloat remove this restriction or at least this assert?

Thanks in advance,

Vorherrschende Sprache: JavaScript
Sterne: 471
Forks: 91
Ø Merge: 3 Std. 57 Min.
Gemergte PRs (30 T.): 2

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus protocolbuffers/protobuf-javascript

Confusing duplication of libraries - @protocolbuffers/protoc-gen-js and protoc-gen-js Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100

protocolbuffers/protobuf-javascript#287 · Erstellt am 10.9.2026
Support for ES Modules (import/export) in protoc --js_out Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 30/100

protocolbuffers/protobuf-javascript#248 · 1 Kommentar · 13 Reaktionen · Erstellt am 23.9.2025
BUG: Timestamp.toObject() returning zeroed value Offen

question

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100

protocolbuffers/protobuf-javascript#222 · 9 Kommentare · Erstellt am 13.9.2024
Why map.js sort keys? Offen

enhancement port-fix triaged

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100

protocolbuffers/protobuf-javascript#185 · 1 Kommentar · Erstellt am 17.1.2024
Generate alternate accessor for fields with presence Offen

enhancement port-fix triaged

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100

protocolbuffers/protobuf-javascript#182 · 3 Kommentare · 1 Reaktion · Erstellt am 30.12.2023

Alle Issues in protocolbuffers/protobuf-javascript