google/closure-compiler

I think parseInt radix should be optional

Open

#3.230 geöffnet am 12. Feb. 2019

Auf GitHub ansehen
 (12 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Java (7.176 Stars) (1.197 Forks)batch import
P4good first issueinternal-issue-createdtriage-done

Beschreibung

I understand why this is. But if feels like a very old legacy issue...

https://github.com/google/closure-compiler/blob/b29d09f56b56e5b3ea78ba0317ae579b837fd821/externs/es3.js#L289-L300

Nowdays all browser use radix 10 as default and has done so for a long time. And the likelihood of someone starting the number with 0 and using IE8 is very slim. Many have stop supporting old browsers

* @param {number=} base

Contributor Guide

Tech Stack
javascript
Domain
tooling
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
mostly clear
Voraussetzungen
basic JavaScriptfamiliarity with closure compiler externs
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30
Research-Richtung
The issue requests making parseInt radix parameter optional. Discussion points to the externs file at https://github.com/google/closure compiler/blob/b29d09f56b56e5b3ea78ba0317ae579b837fd821/externs/es3.js lines 289-300. The change would involve modifying the @param annotation to make base optional, but there are concerns about legacy behavior and browser compatibility. Examine the discussion for maintainer feedback and any alternative solutions proposed.