Thread: [Mlterm-dev-ja] ~/.mlterm/main isn't made

久保田です。

mlconfig で、「save&exit」を選んだとき、そのときに ~/.mlterm/main
が存在していない場合、変更内容が ~/.mlterm/main に反映されません。
というか、~/.mlterm/main が作られません。

調べてみると、どうやら kiklib/src/kik_conf_io.c の
kik_conf_write_open() において、kik_open_file(name,"r") が
失敗したときにエラーになってしまうのが原因のようです。
~/.mlterm/main が存在しないとき、kik_open_file(name,"r")
は失敗するので、そういうときは conf->num = 0 のままで
fopen(name,"w") へと進むのが良いと思います。

---
久保田智広 Tomohiro KUBOTA <ku...@de...>
http://www.debian.or.jp/~kubota/

久保田です。

From: Tomohiro KUBOTA <ku...@de...>
Subject: [Mlterm-dev-ja] ~/.mlterm/main isn't made
Date: Fri, 04 Apr 2003 03:00:27 +0900 (JST)

> ~/.mlterm/main が存在しないとき、kik_open_file(name,"r")
> は失敗するので、そういうときは conf->num = 0 のままで
> fopen(name,"w") へと進むのが良いと思います。

まだ確認していないのですが、たぶんこれでもまだ問題があって、
~/.mlterm というディレクトリが存在しないときには作るように
しないといけないと思います。

---
久保田智広 Tomohiro KUBOTA <ku...@de...>
http://www.debian.or.jp/~kubota/

久保田です。

From: Tomohiro KUBOTA <ku...@de...>
Subject: [Mlterm-dev-ja] ~/.mlterm/main isn't made
Date: Fri, 04 Apr 2003 03:00:27 +0900 (JST)

> mlconfig で、「save&exit」を選んだとき、そのときに ~/.mlterm/main
> が存在していない場合、変更内容が ~/.mlterm/main に反映されません。
> というか、~/.mlterm/main が作られません。

パッチを作ってみました。ラベルを定義して goto を使ってみたり、
static 関数がひとつもない kik_conf_io.c に static 関数を書いて
みたり、非常に汚いものです。が、やりたいことはこういうことだ、
ということを示し実証するには日本語よりもパッチのほうがいいと
思うので、参考としてパッチをお送りします。

---
久保田智広 Tomohiro KUBOTA <ku...@de...>
http://www.debian.or.jp/~kubota/

	荒木です:-)

	Subject: Re: [Mlterm-dev-ja] ~/.mlterm/main isn't made
	From: Tomohiro KUBOTA <ku...@de...>
	Message-ID: <200...@tm...>
	Date: Sun, 06 Apr 2003 02:07:22 +0900 (JST)

>> パッチを作ってみました。ラベルを定義して goto を使ってみたり、
>> static 関数がひとつもない kik_conf_io.c に static 関数を書いて
>> みたり、非常に汚いものです。

        static 関数を追加すること自体は、全然汚くないと思いますが^^;

> 別パッチを作ってみました。実際の処理の内容はほぼ同じですが、
> goto を使う代わりに if(){...} の中にごっそりとファイル読み込み
> 処理を入れたり、mkdir_fopen() の処理を直接 kik_conf_write_open()
> に書いたりしています。
>
> これで cvs commit してよろしいでしょうか?

        ナナメ読みですが、いいと思います。
        お願いします。

	では
--
Araki Ken
ara...@us...

久保田です。

From: Araki Ken <ara...@us...>
Subject: Re: [Mlterm-dev-ja] ~/.mlterm/main isn't made
Date: Sun, 06 Apr 2003 16:01:24 +0900

> >> パッチを作ってみました。ラベルを定義して goto を使ってみたり、
> >> static 関数がひとつもない kik_conf_io.c に static 関数を書いて
> >> みたり、非常に汚いものです。
> 
> static 関数を追加すること自体は、全然汚くないと思いますが^^;

関数の置き方になにか方針があるかもしれなかったので。global にして
どこか別のところに置くべきなのかどうか、分からなかったのです。

新パッチのほうにしたので、関数は追加しません。


> > これで cvs commit してよろしいでしょうか?
> 
>  ナナメ読みですが、いいと思います。
>  お願いします。

了解。やりました。ついでに CP1251 と CP1255 をマニュアルに
追加しました。


> ということですが、free(NULL) した場合になにも起こらない(NULL
> ポインタを参照して core dump とかにはならない)ことは ANSI C
> 規格で保証されているはづです。

そうでしたか... やられました。恥ずかし〜
インデントをひとつ減らすことができたということで、
許してくださいませ。m(_ _)m

---
久保田智広 Tomohiro KUBOTA <ku...@de...>
http://www.debian.or.jp/~kubota/

久保田です。

From: Tomohiro KUBOTA <ku...@de...>
Subject: Re: [Mlterm-dev-ja] ~/.mlterm/main isn't made
Date: Sat, 05 Apr 2003 00:31:08 +0900 (JST)

> パッチを作ってみました。ラベルを定義して goto を使ってみたり、
> static 関数がひとつもない kik_conf_io.c に static 関数を書いて
> みたり、非常に汚いものです。が、やりたいことはこういうことだ、
> ということを示し実証するには日本語よりもパッチのほうがいいと
> 思うので、参考としてパッチをお送りします。

別パッチを作ってみました。実際の処理の内容はほぼ同じですが、
goto を使う代わりに if(){...} の中にごっそりとファイル読み込み
処理を入れたり、mkdir_fopen() の処理を直接 kik_conf_write_open()
に書いたりしています。

これで cvs commit してよろしいでしょうか?

---
久保田智広 Tomohiro KUBOTA <ku...@de...>
http://www.debian.or.jp/~kubota/