Skip to content

some fixes about JavaScript#1

Open
qsona wants to merge 1 commit intokobake:masterfrom
qsona:js-fix
Open

some fixes about JavaScript#1
qsona wants to merge 1 commit intokobake:masterfrom
qsona:js-fix

Conversation

@qsona
Copy link
Copy Markdown

@qsona qsona commented Jan 22, 2016

JavaScriptに関して、気づいた点を修正しました。

  • 仕様上NaN === NaNはfalseになります。したがってNaNであるのチェックはNumber.isNaNを使う必要があります。
    • 自分との比較がfalseになるのはNaNだけなので、if (v === v) というイディオムも有りますが・・・
  • ECMAScript5 (ES5) ベースで書かれていますが、そうでないものにはES2015と注釈を入れました。
    • 定数、文字列への埋め込みはいずれもES2015からの仕様です。

@kobake
Copy link
Copy Markdown
Owner

kobake commented Jan 25, 2016

PRありがとうございます!対応遅くてすみませんが週末あたり確認しますー

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants