Re: [ja-discuss] LibreOfficeのソースコードで使うべきASSERT

On Tue, 2014-10-28 at 22:40 +0900, Naruhiko Ogasawara wrote:

> レビューにて「その条件は絶対成立しないとダメなやつだから、
> ワーニングじゃなくてアサートにしないとだめだろ」っていわれて、
> それは私も大変同意なんですが、ではアサートってどのマクロ
> 使うねん、ってのがわからなかったりします。
> 単にASSERTでソースgrepするとDBG_ASSERTとOSL_ASSERT
> が出てくるわけですが、どちらもdeprecatedだと書いてあって、
> うーんと悩んでいます。

ふつうに assert(...)を使って構わないと思います。以前はOSL_ASSERTを推奨し
ていたと思いましたが、今は言語で標準のassert(...)を推奨しています。僕は
いつもassert()使ってますね。

> このパッチ自体、もとはOSL_ASSERTだったものの書き換えなので、
> 取り下げするのが正しいのかなあ……。

いえ、assert()に置き換えて良いと思いますよ。

ではでは。



-- 
Unsubscribe instructions: E-mail to discuss+unsubscribe@ja.libreoffice.org
Posting guidelines + more: http://wiki.documentfoundation.org/Netiquette
List archive: http://listarchives.libreoffice.org/ja/discuss/
All messages sent to this list will be publicly archived and cannot be deleted