Re: warningについて (Anthy-dev 1587) - Anthy

On Tue, 11 Jan 2005 16:32:18 +0900
YamaKen <yamak****@bp*****> wrote:

> At Tue, 11 Jan 2005 15:03:34 +0900,
> tkng****@xem***** wrote:
> > 　以下がgccが出すwarningとそれに関するコメントです。
> > 
> > $make > build.log
> > uim-scm.c:50:60: warning: "/*" within comment
> > uim-scm.c:51:6: warning: "/*" within comment
> > 
> > 　/*はコメント内に出てきてはいけないのですが、どうしたらいいかちょっ
> > とわかりません。
> 
> あれ、そうだったかな…と思ってISO/IEC 9899-1999で確認してみまし
> た。
> 
> 6.4.9 Comments ではネストしてはいけないとは書いてあるけど "/*" 
> を含んではいけないという規定はないようです。
> 
> ただし、間違いではないwarningについてもuimとして撲滅運動を進める
> という事であれば、何とかしたいと思います。今のところいいアイディ
> アはありませんが。

あんま深いことは考えてませんが、 -std=gnu89 -pedanticとかしているぐらい
なので、warningは可能な限りなくすべきだと思っています。

(snip)


> > uim-custom.c: In function `extract_choice_list':
> > uim-custom.c:274: warning: suggest parentheses around assignment
> > used as truth value
> > 
> >  for (p = choice_sym_list; choice_sym = *p; p++) {
> > 
> > のchoice_sym = *pがまずいようです。
> 
> これは私の慣用なんですが、
> 
> >  for (p = choice_sym_list; *p != NULL; p++) {
> >      choice_sym = *p
> > 
> > でいいんでしょうか？
> 
> 先述したように間違いではないwarningについてもuimとして撲滅運動を
> 進めるのなら、このように書き換えます。
> 
> ただし私は「"!= NULL"は冗長だし目視文脈解析を妨げるので嫌いだよ
> 派」なので、
> 
>   for (p = choice_sym_list; *p; p++) {
>       choice_sym = *p
> 
> こうなりますが。これもuimプロジェクトとして筋の通った方針を立て
> るなら違うスタイルに変えます。

uimとしてコーディングスタイルを厳密に定めるつもりはありません。ですから
以下は単なる1コミッタの意見です。

>   for (p = choice_sym_list; *p; p++) {
>       choice_sym = *p

だと*pが目立たなくて見えづらいので、*p != NULLとする事を一応主張しておき
ます。まぁ、好みの問題だと思うので、お好きにどうぞ。


-- 
徳永拓之
tkng****@xem*****
http://kodou.net/

Anthy

[Anthy-dev 1587] Re: warningについて