-
Notifications
You must be signed in to change notification settings - Fork 107
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ВызватьИсключение c передачей контекста исключения #188
Comments
Original comment by EvilBeaver (Bitbucket: EvilBeaver, GitHub: EvilBeaver): Тогда сразу конкурс: как будет называться свойство в объекте ИнформацияОбОшибке, в котором мы будем получать то значение, которое передали в момент выброса?
|
Original comment by EvilBeaver (Bitbucket: EvilBeaver, GitHub: EvilBeaver): ДополнительныеДанные?
|
Original comment by EvilBeaver (Bitbucket: EvilBeaver, GitHub: EvilBeaver): Я за ДополнительныеДанные или за Параметры. @dmpas? |
Original comment by EvilBeaver (Bitbucket: EvilBeaver, GitHub: EvilBeaver): Принято. Только одно уточнение. Если мы будем передавать в конструктор некий anyType, то тогда это |
Original comment by EvilBeaver (Bitbucket: EvilBeaver, GitHub: EvilBeaver): @dmpas, @artbear - все-таки множественное или единственное число? |
Ура! |
Запрашиваю всестороннее тестирование |
Доработаю код в своем проекте https://github.com/artbear/1bdd |
@EvilBeaver Ты просил проверить, я проверил и нашел баг. Исправишь? |
Посмотрел код 756dc8b С точки зрения клиентского кода внутри кода обработки нельзя заранее знать, какое исключение было вызвано - обычное или параметризованное. |
@EvilBeaver Переоткрывай issue |
…vilBeaver/OneScript#188" вместо выбрасывания текстового исключения fix #44
Переоткрывать не надо, это новый баг, штатный процесс |
Да, я проглядел это... Спасибо @artbear. |
Тест проверки использования параметров исключения при обычном исключении #188
Originally reported by: artbear (Bitbucket: artbear, GitHub: artbear)
ВызватьИсключение Новый ИнформацияОбОшибке(/*Любой контекст*/);
По мотивам EvilBeaver/oscript-library#29 (comment)
The text was updated successfully, but these errors were encountered: