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

小笠原です。
カジュアルに日本語で聞いてしまいます。

とりあえずEasyHacksにチャレンジしてみようと思いまして、

https://gerrit.libreoffice.org/#/c/12103/1/pyuno/source/module/pyuno.cxx

ってたった2行変えただけのパッチ送ったら、間違いだらけで
大変なことになりました。ちょっと凹んでます。とほほ。

# 安倍さんのご指摘、コメント文字列の条件逆になってたのは、
# 自分でもなんでだかわかりません><;
# include/sal/log-areas.dox については完全に見落としで、
# ありがとうございます。


まあ恥をかいてレビュープロセスを体験するのは、それこそ
EasyHacksの目的にかなってるってことかなあと（人の手間を
取ってることは申し訳ないとして）思うのですが、それはさておき。

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

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

どなたかご存じの方がいれば。


では。
-- 
Naruhiko Ogasawara (naruoga@gmail.com)

-- 
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