facebook / facebook/flow

String substr/substring first argument should be optional

Aperta
#2,802 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Library definitions
Lingua principale
Rust
Stelle
22.3k
Fork
1.9k
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

https://github.com/facebook/flow/blob/master/lib/core.js#L262-L263

```typescript
substr(from: number, length?: number): string;
substring(start: number, end?: number): string;
```

As I understand it, [ES5 B.2.3](http://www.ecma-international.org/ecma-262/5.1/#sec-B.2.3) and [15.5.4.15](http://www.ecma-international.org/ecma-262/5.1/#sec-15.5.4.15) do not require the first argument to be defined. Step 2 of substr (step 4 of substring) converts the argument to integer, and an undefined value will converted to zero.

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.