google/closure-compiler

I think parseInt radix should be optional

Open

#3230 aperta il 12 feb 2019

Vedi su GitHub
 (12 commenti) (0 reazioni) (0 assegnatari)Java (1197 fork)batch import
P4good first issueinternal-issue-createdtriage-done

Metriche repository

Star
 (7176 star)
Metriche merge PR
 (Merge medio 12g 15h) (1 PR mergiata in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Esamina la specifica ECMAScript per il comportamento predefinito della radice di parseInt e verifica come gli externs di Closure Compiler influenzano gli avvisi. Considera le implicazioni di compatibilità del browser e testa le modifiche.
Tech stack
javascript
Dominio
backend
Tipo issue
Funzionalità
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
GitJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70