[PHP-dev 499] Re: [PATCH] mb_parse_str の共通部分の統合と php_mbstr_strtok_r() の導入

[Rui Hirokawa]

廣川です。

ありがとうございます。

mbstringのコード内容は、私の不勉強さもあって問題点も多々あるかと思います。
ぜひ参考にさせていただき、php4-devの方に反映させていただきたいと思います。

sourceforgeのコード変更については、ぜひ、小泉さんにもCVSアカウントを取得
していただき、変更を適用していただくとよいかと思います。
今のところ、PHP 4.3.0のリリースブランチができた後に本家にsourceforge.jp
の変更内容を反映させるというスケジュールだったと思います。
いかがでしょうか? > 大垣さん

また、もしmbstringの開発に興味がおありで、今後も開発にコミットされる意志
がおありでしたら、本家php.netのCVSアカウントも申請されてはいかがでしょう
か？

http://www.php.net/cvs-php.php

mbstringの開発を行なうと書いていただければあおそらくアカウントは取得でき
ると思います。

現在、米国出張中のためなかなかPHP関連の作業ができないのですが、今月末に
は帰国しますので小泉さんの変更内容を詳しく見てみます。

http://ave.sytes.net/~koizumi/php.net/mbstring-mods-20020916.tar.gz
をダウンロードしましたので、出張先で変更内容を少し見てみようと思いますが、
変更点が多いので少し時間がかかるかもしれません。

On Mon, 16 Sep 2002 03:52:48 +0900
Moriyoshi Koizumi <readjust@deneb.freemail.ne.jp> wrote:

Moriyoshi> 小泉です。
Moriyoshi> 
Moriyoshi> この間は大変お騒がせしましたが、あのとき mbstring.c を見ていて、気になった
Moriyoshi> 部分を変えてみました。
Moriyoshi> [PATCH] とはつけてみましたが、もはやパッチではないかもしれません。
Moriyoshi> 
Moriyoshi> さて、変更点はといえば、メールのタイトルどおりなのですが、
Moriyoshi> 具体的には、
Moriyoshi> mb_parse_str 内のコードを、php_mbstr_encoding_handler_ex と共通化したのと、
Moriyoshi> php_mbstr_strtok_r を新設し、SJIS への対応を強化しました。
Moriyoshi> 
Moriyoshi> これにより、 arg_separator.input=\ となっていても、SJIS 文字列を
Moriyoshi> 正しくパースできるようになっています。
Moriyoshi> 
Moriyoshi> ただし、encoding detection の方法が変わったので、
Moriyoshi> 別の問題を生じるかもしれません。
Moriyoshi> 
Moriyoshi> diff するとかなり大きいので(30k程度)、自宅 adsl サーバで恐縮ですが、
Moriyoshi> 以下からダウンロードしてください。問題があれば、直接メールでお送りします。
Moriyoshi> 
Moriyoshi> [php-i18n.sourceforge.jp の方のコードベースへの変更]
Moriyoshi> 
Moriyoshi> http://ave.sytes.net/~koizumi/php-i18n.sourceforge.jp/mbstring.c.diff
Moriyoshi> http://ave.sytes.net/~koizumi/php-i18n.sourceforge.jp/post.c
Moriyoshi> http://ave.sytes.net/~koizumi/php-i18n.sourceforge.jp/post.h
Moriyoshi> 
Moriyoshi> (テストスクリプト) http://ave.sytes.net/~koizumi/php-i18n.sourceforge.jp/test.phps
Moriyoshi> 
Moriyoshi> 以上まとめたもの(二行に折り返しています)
Moriyoshi> http://ave.sytes.net/~koizumi/php-i18n.sourceforge.jp/
Moriyoshi> mbstring-mods-sfjp-20020916.tar.gz
Moriyoshi> 
Moriyoshi> [php.net の方のコードベースへの変更]
Moriyoshi> こちらは、かなり自分好みに変更してしまったために、原型をとどめていません。
Moriyoshi> いずれなんとかしたいと思います。基本的な部分は一緒です。
Moriyoshi> 
Moriyoshi> http://ave.sytes.net/~koizumi/php.net/mbstring.c
Moriyoshi> http://ave.sytes.net/~koizumi/php.net/mbstring.h
Moriyoshi> 
Moriyoshi> (ユニットテスト)
Moriyoshi> http://ave.sytes.net/~koizumi/php.net/021.phpt
Moriyoshi> http://ave.sytes.net/~koizumi/php.net/022.phpt
Moriyoshi> 
Moriyoshi> 以上まとめたもの
Moriyoshi> http://ave.sytes.net/~koizumi/php-net/mbstring-mods-20020916.tar.gz
Moriyoshi> 
Moriyoshi> なにか意見いただけると幸いです。

-- 
Rui Hirokawa <rui_hirokawa@ybb.ne.jp>