Skip to content
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

独自例外を定義して使うようにしたい #356

Closed
berryzplus opened this issue Aug 24, 2018 · 1 comment
Closed

独自例外を定義して使うようにしたい #356

berryzplus opened this issue Aug 24, 2018 · 1 comment
Labels
enhancement ■機能追加

Comments

@berryzplus
Copy link
Contributor

独自例外を定義して使うようにしたいです。

#351 (review)

sakura_core/mem/CNativeA.cpp

77  - 		sprintf_s(cbuf, _countof(cbuf), "AppendStringF error. errno = %d", e);  
78  - 		throw std::exception(cbuf);  
 76 + 		DEBUG_TRACE(L"AppendStringF error. errno = %d", e);  
 77 + 		throw std::exception();  

m-tmatma

これは visual studio での動作を変えてしまうのでは?

berryzplus

目的はビルドできるようにすることなので、コードの目的を変えない範囲で変更しています。
変更前)エラー情報を例外に詰めて投げる・・・catchするコードがないのでそのまま落ちる
変更後)エラー情報をデバッガに出力しエラーを投げる・・・catchするコードがないのでそのまま落ちる

std::exceptionのコンストラクタがconstexprでない文字列を受け付けないのが原因なので、恒久対応としては専用の例外クラスを自作することになると思っています。

m-tmatma

チケットを作っておいてもらえますか?


いったんチケットだけ作ります。

サクラエディタには「構造的に見て例外を投げるべきでない箇所」がたくさんあります。
どう使うかを含めて周知しなければ独自例外を作る効果は薄い気がするので、対応はもう少し先(x64対応完了後)がよいと思います。

@berryzplus
Copy link
Contributor Author

モチベーションが尽きたので閉じてしまいます。 #1394

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ■機能追加
Projects
None yet
Development

No branches or pull requests

2 participants