google/closure-compiler

I think parseInt radix should be optional

Open

#3.230 aberto em 12 de fev. de 2019

Ver no GitHub
 (12 comments) (0 reactions) (0 assignees)Java (7.176 stars) (1.197 forks)batch import
P4good first issueinternal-issue-createdtriage-done

Description

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

Guia do colaborador

Pilha de tecnologia
javascript
Domain
tooling
Tipo Issue
feature
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
mostly clear
Prerequisites
basic JavaScriptfamiliarity with closure compiler externs
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
30
Direção de pesquisa
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.