Next level!

[vs-kurkin]

Предпосылки для изменений:

• Хотелось реализовать поддержку стратегии обработки ошибок, при которой инстанс оборачивается на каждом логическом уровне приложения. Нехватало логирования стека всех экземпляров, а так же полного человекопонятного сообщения об ошибке. Про эту стратегию можно почитать здесь.
• В текущей реализации создание и логирование ошибок занимало заметное количество времени из-за формирования стека для каждого экземпляра, что совершенно ненужно. В нормальном боевом окружении разница в производительности может быть не заметна.

Changelog:

• Terror теперь может принимать вторым аргументом объект error.
• Если вторым аргументом была передана строка, она не попадает в свойство originalError. Существуют сторонние обертки над ошибками, использующие значение из этого свойства. Если в нем содержится что-то отличное от error, возникало исключение при работе с terror.originalError.stack.
• Свойство Terror#message содержит только оригинальное сообщение текущего объекта.
• Новое свойство Terror.stackTraceLimit позволяет ограничивать глубину формируемого стека ошибки (если поддерживается интерпретатором). Может быть переопределено для любого конструктора, наследуемого от Terror. С помощью данного свойства можно сильно снизить нагрузку на процессор, если устанавить его значение в 0 на высоких уровнях приложения.
• Terror.isTerror теперь не падает, если передать null или undefined.
• Terror#log теперь логирует стеки всех обернутых ошибок.
• Новый метод Terror#getFullMessage возвращает полное сообщение об ошибке, включая сообщения обернутых экземпляров. Сообщение каждого объекта ошибки воспринимается как предложение. Формат возвращаемого значения: ERR_CODE Can not create page. Can not instance widget. a is no a function.
• Новый метод Terror#getFullStack возвращает стеки всех обернутых экземпляров. Каждый последующий стек отбивается четыремя пробелами слева (стандартный отступ стека). К каждому сообщению об ошибке добавляется код.
• Terror#setMessage теперь не меняет сообщение в сформированном стеке.
• Кастомные конструкторы теперь имеют полноценное имя (MyError.name).
• Оптимизация производительности: cd benchmark; npm test.
• Тесты переписаны на jasmine.
• Terror теперь не зависит от NodeJS и может выполняться в любом окружении (Issuse #1).
• Мог что-то упустить.

Что нужно сделать:

• Портировать тесты на mocha, раз вы используете его. Удалить старые тесты.
• Решить с форматом Terror#getFullStack, так как он сейчас жестко зашит в метод. Я бы оставил и так, ничего криминального в этом нет. Terror#logMultilineError тоже этим грешит.
• Обновить README.md и JSDoc. Мой английский не позволяет это сделать грамотно.
• Что-нибудь еще?

[vs-kurkin]

Упс... fatal: unable to connect to github.com.

[narqo]

Я правильно понимаю, что единственный смысл этого — иметь именованный конструктор от name?

[vs-kurkin]

Функционально - да.
Так же есть небольшой профит в производительности, поскольку нет необходимости рекурсивно вызывать все конструкторы в цепочке наследования. Сейчас вызывается только текущий конструктор и Terror. Этого можно было бы добиться и с замыканием, но я их не люблю.

[narqo]

Так же есть небольшой профит в производительности, поскольку нет необходимости рекурсивно вызывать все конструкторы в цепочке наследования

Ты правда смог это воспроизвести на benchmark'ах? Или это видно по IR? Я конечно не копался очень детально, но разницы вообще никакой не увидел

[vs-kurkin]

Это очевидно, поскольку конструкторы неинлайновые и будут вызываться всегда. Профит стремится к нулю, зато префекционизм не страдает.

[kaero]

@B-Vladi squash commits, pls. I'll merge PR into temporary v1.2 branch to update tests and doc.

[vs-kurkin]

Done

[kaero]

@an9eldust as I can remember, you was interested in Terror implementation for client-side. This PR provides some support, except IE. Are you still interested in it?

[vs-kurkin]

Fixes for old IE

[kaero]

merged manually into branch v1.2