exebook / deodar

Классическая двухпанельная рабочая среда (коммандер) для О.С. Линукс
The Unlicense
72 stars 8 forks source link

lexer. исправлена ошибка: "'c' used out of scope" #6

Closed coderaiser closed 10 years ago

coderaiser commented 10 years ago

Объявление c перенесено из цикла в начало функции colorizeString.

exebook commented 10 years ago

крутые дреды хм, а что за странная новость, с каких пор имеет значение в каком месте функции стоит вар? javascript variables have functional scope? где бы вы её ее объявили, это всё равно, что в начале.

coderaiser commented 10 years ago

Спасибо :). Всё верно, значения не имеет, переменная всегда всплывёт наверх. Поэтому jshint рекомендует объявлять переменные вверху функции, хотя это, конечно, по-желанию.

P.S.: В ES6 появился оператор let и блочная область видимости. В лисе эти синтаксические конструкции уже поддерживаются. Остальные браузеры, в этом плане, отстают, или не сильно торопятся, что бы не ломать совместимость нового кода со старыми браузерами.