From jca02266 ＠ nifty.com Sat Sep 24 22:43:06 2005 From: jca02266 ＠ nifty.com (Koji Arai) Date: Sat, 24 Sep 2005 22:43:06 +0900 (JST) Subject: [Lha-users] LHa for UNIX (autoconf) 2005-09-24 release Message-ID: <20050924.224306.74740525.jca02266@nifty.com> 新井です。 LHa for UNIX 2005-09-24 版をリリースしました。 変更点は以下の通りです。 o use *my* getopt_long() library. (not GNU) x allow option name abbreviation like `--system' instead of `--system-kanji-code'. o more better directory traversal hack. x can extract relative path like "./xxx". (cannot extract "../xxx"). x remove leading relative path when adding to archive. -- 新井康司 (Koji Arai) From sakai ＠ fan.gr.jp Sun Sep 25 14:36:15 2005 From: sakai ＠ fan.gr.jp (Hiroto Sakai) Date: Sun, 25 Sep 2005 14:36:15 +0900 Subject: [Lha-users] Re: LHa for UNIX (autoconf) 2005-09-24 release In-Reply-To: <20050924.224306.74740525.jca02266@nifty.com> References: <20050924.224306.74740525.jca02266@nifty.com> Message-ID: <20050925143615.166505.942214bc@fan.gr.jp> 坂井です。 On Sat, 24 Sep 2005 22:43:06 +0900 (JST), Koji Arai wrote: > LHa for UNIX 2005-09-24 版をリリースしました。 Mac OS X 10.4.2 (gcc 4.0) で make したところ Warning が出ますね。 実害はないようですが。 gcc -DHAVE_CONFIG_H -I. -I. -I.. -DSUPPORT_LH7 -O2 -c `test -f 'header.c' || echo './'`header.c header.c: In function 'get_extended_header': header.c:602: warning: pointer targets in passing argument 2 of 'calccrc' differ in signedness header.c: In function 'get_header_level2': header.c:932: warning: pointer targets in passing argument 2 of 'calccrc' differ in signedness header.c: In function 'get_header_level3': header.c:1013: warning: pointer targets in passing argument 2 of 'calccrc' differ in signedness header.c: In function 'seek_lha_header': header.c:1158: warning: pointer targets in passing argument 1 of 'calc_sum' differ in signedness header.c: In function 'write_header_level2': header.c:1569: warning: pointer targets in passing argument 2 of 'calccrc' differ in signedness gcc -DHAVE_CONFIG_H -I. -I. -I.. -DSUPPORT_LH7 -O2 -c `test -f 'util.c' || echo './'`util.c util.c: In function 'copyfile': util.c:41: warning: pointer targets in passing argument 1 of 'fread_txt' differ in signedness util.c:62: warning: pointer targets in passing argument 1 of 'fwrite_txt' differ in signedness util.c:75: warning: pointer targets in passing argument 2 of 'calccrc' differ in signedness -- 坂井 浩人 (Hiroto Sakai) e-mail: sakai ＠ fan.gr.jp web: http://www.fan.gr.jp/~sakai/ From oku.ns ＠ dream.com Sun Sep 25 15:14:53 2005 From: oku.ns ＠ dream.com (OKUMURA N. Shin-ya) Date: Sun, 25 Sep 2005 15:14:53 +0900 Subject: [Lha-users] Re: LHa for UNIX (autoconf) 2005-09-24 release In-Reply-To: <20050925143615.166505.942214bc@fan.gr.jp> References: <20050924.224306.74740525.jca02266@nifty.com> <20050925143615.166505.942214bc@fan.gr.jp> Message-ID: <20050925061451.GA15179@okupc1.no-ip.org> おくむら＠練馬です。 On Sun, Sep 25, 2005 at 02:36:15PM +0900, Hiroto Sakai wrote: > Mac OS X 10.4.2 (gcc 4.0) で make したところ Warning が出ますね。 多分 gcc-4.x が行儀作法に厳しくなっているせいでは ないでしょうか？ 手元 (Gentoo/amd64) で試してみたところ、gcc-4.0.1 では 坂井さんとほぼ同じように警告が出ました。 $ (CC=gcc-4.0.1 ./configure && make clean && make) \ > | grep '警告' Makefile:287: 警告: ターゲット `lha-test' へのコマンドを置き換えます Makefile:157: 警告: ターゲット `lha-test' への古いコマンドは無視されます header.c: In function 'get_extended_header': header.c:602: 警告: pointer targets in passing argument 2 of 'calccrc' differ in signedness header.c: In function 'get_header_level2': header.c:932: 警告: pointer targets in passing argument 2 of 'calccrc' differ in signedness header.c: In function 'get_header_level3': header.c:1013: 警告: pointer targets in passing argument 2 of 'calccrc' differ in signedness header.c: In function 'seek_lha_header': header.c:1158: 警告: pointer targets in passing argument 1 of 'calc_sum' differ in signedness header.c: In function 'write_header_level2': header.c:1569: 警告: pointer targets in passing argument 2 of 'calccrc' differ in signedness util.c: In function 'copyfile': util.c:41: 警告: pointer targets in passing argument 1 of 'fread_txt' differ in signedness util.c:62: 警告: pointer targets in passing argument 1 of 'fwrite_txt' differ in signedness util.c:75: 警告: pointer targets in passing argument 2 of 'calccrc' differ in signedness Makefile:287: 警告: ターゲット `lha-test' へのコマンドを置き換えます Makefile:157: 警告: ターゲット `lha-test' への古いコマンドは無視されます 一方、gcc-3.4.4 では特に何も言われません。 $ (CC=gcc-3.4.4 ./configure && make clean && make) \ > | grep '警告' Makefile:287: 警告: ターゲット `lha-test' へのコマンドを置き換えます Makefile:157: 警告: ターゲット `lha-test' への古いコマンドは無視されます Makefile:287: 警告: ターゲット `lha-test' へのコマンドを置き換えます Makefile:157: 警告: ターゲット `lha-test' への古いコマンドは無視されます ぱっと見に、仮引数と実引数を char か unsigned char の一方に 揃えればいいだけのようですので、まあ警告を黙らせるだけの修正 であれば大した面倒でもないのかな... と勝手に思っています。 # と他人には書きつつ自分が提案した i18n 改良をしないとは # なんて酷い人間だ。(^^; > 自分 -- OKUMURA N. Shin-ya (oku.ns at dream.com) if (it == misfeature && it != bug) it = bug; From jca02266 ＠ nifty.com Sun Sep 25 22:24:50 2005 From: jca02266 ＠ nifty.com (Koji Arai) Date: Sun, 25 Sep 2005 22:24:50 +0900 (JST) Subject: [Lha-users] Re: LHa for UNIX (autoconf) 2005-09-24 release In-Reply-To: <20050925061451.GA15179@okupc1.no-ip.org> References: <20050924.224306.74740525.jca02266@nifty.com> <20050925143615.166505.942214bc@fan.gr.jp> <20050925061451.GA15179@okupc1.no-ip.org> Message-ID: <20050925.222450.74735636.jca02266@nifty.com> 新井です。 坂井さん、おくむらさん報告ありがとうございます。 パッチ作ってみました。問題なければ commit しようと思います。 In message "[Lha-users] Re: LHa for UNIX (autoconf) 2005-09-24 release" on Sun, 25 Sep 2005 15:14:53 +0900, "OKUMURA N. Shin-ya" wrote: > おくむら＠練馬です。 > # と他人には書きつつ自分が提案した i18n 改良をしないとは > # なんて酷い人間だ。(^^; > 自分 まあ私も LHa for UNIX に大した修正入れてないので。ただまあ、 こんなペースでいいんじゃあないかと思ってます。(なんとなく1年 1 回リリースを厳守できてる ^^) Index: src/crcio.c =================================================================== RCS file: /cvsroot/lha/lha/src/crcio.c,v retrieving revision 1.11 diff -u -r1.11 crcio.c --- src/crcio.c 19 Jan 2003 22:13:22 -0000 1.11 +++ src/crcio.c 25 Sep 2005 13:18:43 -0000 @@ -35,7 +35,7 @@ unsigned int calccrc(crc, p, n) unsigned int crc; - unsigned char *p; + char *p; unsigned int n; { while (n-- > 0) @@ -47,9 +47,9 @@ int fread_crc(crcp, p, n, fp) unsigned int *crcp; - unsigned char *p; - int n; - FILE *fp; + void *p; + int n; + FILE *fp; { if (text_mode) n = fread_txt(p, n, fp); @@ -67,9 +67,9 @@ void fwrite_crc(crcp, p, n, fp) unsigned int *crcp; - unsigned char *p; - int n; - FILE *fp; + void *p; + int n; + FILE *fp; { *crcp = calccrc(*crcp, p, n); #ifdef NEED_INCREMENTAL_INDICATOR @@ -141,11 +141,13 @@ /* ------------------------------------------------------------------------ */ int -fwrite_txt(p, n, fp) - unsigned char *p; - int n; - FILE *fp; +fwrite_txt(vp, n, fp) + void *vp; + int n; + FILE *fp; { + unsigned char *p = vp; + while (--n >= 0) { if (*p != '\015' && *p != '\032') { #ifdef EUC @@ -163,13 +165,16 @@ /* ------------------------------------------------------------------------ */ int -fread_txt(p, n, fp) - unsigned char *p; - int n; - FILE *fp; +fread_txt(vp, n, fp) + void *vp; + int n; + FILE *fp; { int c; int cnt = 0; + unsigned char *p; + + p = vp; while (cnt < n) { if (getc_euc_cache != EOF) { Index: src/header.c =================================================================== RCS file: /cvsroot/lha/lha/src/header.c,v retrieving revision 1.86 diff -u -r1.86 header.c --- src/header.c 7 May 2005 21:16:05 -0000 1.86 +++ src/header.c 25 Sep 2005 13:18:43 -0000 @@ -57,12 +57,12 @@ int calc_sum(p, len) - char *p; + void *p; int len; { int sum = 0; - while (len--) sum += *p++; + while (len--) sum += *(unsigned char*)p++; return sum & 0xff; } Index: src/lha_macro.h =================================================================== RCS file: /cvsroot/lha/lha/src/lha_macro.h,v retrieving revision 1.46 diff -u -r1.46 lha_macro.h --- src/lha_macro.h 20 Jul 2003 18:26:23 -0000 1.46 +++ src/lha_macro.h 25 Sep 2005 13:18:43 -0000 @@ -197,7 +197,7 @@ #define CRCPOLY 0xA001 /* CRC-16 (x^16+x^15+x^2+1) */ #define INITIALIZE_CRC(crc) ((crc) = 0) #define UPDATE_CRC(crc, c) \ - (crctable[((crc) ^ (c)) & 0xFF] ^ ((crc) >> CHAR_BIT)) + (crctable[((unsigned char)(crc) ^ (c)) & 0xFF] ^ ((unsigned char)(crc) >> CHAR_BIT)) /* dhuf.c */ #define N_CHAR (256 + 60 - THRESHOLD + 1) Index: src/prototypes.h =================================================================== RCS file: /cvsroot/lha/lha/src/prototypes.h,v retrieving revision 1.33 diff -u -r1.33 prototypes.h --- src/prototypes.h 7 May 2005 18:39:11 -0000 1.33 +++ src/prototypes.h 25 Sep 2005 13:18:43 -0000 @@ -16,12 +16,12 @@ void init_putbits P_((void)); /* crcio.c */ void make_crctable P_((void)); -unsigned int calccrc P_((unsigned int crc, unsigned char *p, unsigned int n)); -int fread_crc P_((unsigned int *crcp, unsigned char *p, int n, FILE *fp)); -void fwrite_crc P_((unsigned int *crcp, unsigned char *p, int n, FILE *fp)); +unsigned int calccrc P_((unsigned int crc, char *p, unsigned int n)); +int fread_crc P_((unsigned int *crcp, void *p, int n, FILE *fp)); +void fwrite_crc P_((unsigned int *crcp, void *p, int n, FILE *fp)); void init_code_cache P_((void)); -int fwrite_txt P_((unsigned char *p, int n, FILE *fp)); -int fread_txt P_((unsigned char *p, int n, FILE *fp)); +int fwrite_txt P_((void *vp, int n, FILE *fp)); +int fread_txt P_((void *vp, int n, FILE *fp)); /* dhuf.c */ void start_c_dyn P_((void)); void decode_start_dyn P_((void)); @@ -32,7 +32,7 @@ /* extract.c */ int decode_lzhuf P_((FILE *infp, FILE *outfp, size_t original_size, size_t packed_size, char *name, int method, size_t *read_sizep)); /* header.c */ -int calc_sum P_((char *p, int len)); +int calc_sum P_((void *p, int len)); void convert_filename P_((char *name, int len, int size, int from_code, int to_code, char *from_delim, char *to_delim, int case_to)); boolean get_header P_((FILE *fp, LzHeader *hdr)); int seek_lha_header P_((FILE *fp)); cvs diff: Diffing tests -- 新井康司 (Koji Arai) From jca02266 ＠ nifty.com Sun Sep 25 23:48:18 2005 From: jca02266 ＠ nifty.com (Koji Arai) Date: Sun, 25 Sep 2005 23:48:18 +0900 (JST) Subject: [Lha-users] Re: LHa for UNIX (autoconf) 2005-09-24 release In-Reply-To: <20050925.222450.74735636.jca02266@nifty.com> References: <20050925143615.166505.942214bc@fan.gr.jp> <20050925061451.GA15179@okupc1.no-ip.org> <20050925.222450.74735636.jca02266@nifty.com> Message-ID: <20050925.234818.79545598.jca02266@nifty.com> 新井です。 In message "[Lha-users] Re: LHa for UNIX (autoconf) 2005-09-24 release" on Sun, 25 Sep 2005 22:24:50 +0900 (JST), Koji Arai wrote: > 新井です。 > > 坂井さん、おくむらさん報告ありがとうございます。 > > パッチ作ってみました。問題なければ commit しようと思います。 パッチ当てるのも人によっては面倒でしょうから CVS に commit しておきました。 -- 新井康司 (Koji Arai) From oku.ns ＠ dream.com Mon Sep 26 22:24:14 2005 From: oku.ns ＠ dream.com (OKUMURA N. Shin-ya) Date: Mon, 26 Sep 2005 22:24:14 +0900 Subject: [Lha-users] Re: LHa for UNIX (autoconf) 2005-09-24 release In-Reply-To: <20050925.234818.79545598.jca02266@nifty.com> <20050925.222450.74735636.jca02266@nifty.com> References: <20050925143615.166505.942214bc@fan.gr.jp> <20050925061451.GA15179@okupc1.no-ip.org> <20050925.222450.74735636.jca02266@nifty.com> <20050925.234818.79545598.jca02266@nifty.com> <20050924.224306.74740525.jca02266@nifty.com> <20050925143615.166505.942214bc@fan.gr.jp> <20050925061451.GA15179@okupc1.no-ip.org> <20050925.222450.74735636.jca02266@nifty.com> Message-ID: <20050926132414.GA4620@okupc1.no-ip.org> おくむら＠練馬です。 対応ありがとうございました。 > 新井さん On Sun, Sep 25, 2005 at 10:24:50PM +0900, Koji Arai wrote: > パッチ作ってみました。問題なければ commit しようと思います。 試してみました。 当方の環境では gcc-3.4.4/4.0.1 とも warning が消えました。 また、./tests のテストも全て ok と出ています。 > In message "[Lha-users] Re: LHa for UNIX (autoconf) 2005-09-24 release" > on Sun, 25 Sep 2005 15:14:53 +0900, > "OKUMURA N. Shin-ya" wrote: > > おくむら＠練馬です。 > > > # と他人には書きつつ自分が提案した i18n 改良をしないとは > > # なんて酷い人間だ。(^^; > 自分 > > まあ私も LHa for UNIX に大した修正入れてないので。ただまあ、 > こんなペースでいいんじゃあないかと思ってます。(なんとなく1年 > 1 回リリースを厳守できてる ^^) 私の方は、言い訳をお許しいただけるのであれば、 ・種々雑多な Unices に種々雑多な iconv が存在して 　非互換性を巧く autoconf に吸収できない。 ・そもそも autoconf/automake にあんまり詳しくない。 < ダメやん という状況です。_(_ _)_ # 全ての iconv(1) で iconv -l が使えるのなら # デフォルトエンコーディング名を決定するのに # 少しは楽ができるのですけれど... # Solaris [78] ではダメみたいです。 On Sun, Sep 25, 2005 at 11:48:18PM +0900, Koji Arai wrote: > パッチ当てるのも人によっては面倒でしょうから CVS に commit しておきました。 すみません、CVS の方は試していません。 -- OKUMURA N. Shin-ya (oku.ns at dream.com) if (it == misfeature && it != bug) it = bug; From jca02266 ＠ nifty.com Mon Sep 26 23:36:01 2005 From: jca02266 ＠ nifty.com (Koji Arai) Date: Mon, 26 Sep 2005 23:36:01 +0900 (JST) Subject: [Lha-users] Re: LHa for UNIX (autoconf) 2005-09-24 release In-Reply-To: <20050926132414.GA4620@okupc1.no-ip.org> References: <20050925061451.GA15179@okupc1.no-ip.org> <20050925.222450.74735636.jca02266@nifty.com> <20050926132414.GA4620@okupc1.no-ip.org> Message-ID: <20050926.233601.74728255.jca02266@nifty.com> 新井です。 In message "[Lha-users] Re: LHa for UNIX (autoconf) 2005-09-24 release" on 26 Sep 2005 22:24:14 +0900, "OKUMURA N. Shin-ya" wrote: > おくむら＠練馬です。 > ・種々雑多な Unices に種々雑多な iconv が存在して > 　非互換性を巧く autoconf に吸収できない。 > ・そもそも autoconf/automake にあんまり詳しくない。 < ダメやん > > という状況です。_(_ _)_ autoconf/automake の対応は後回しにして良いと思います。 とりあえず、今設定しているゴールって、euc や sjis や eucJP-ms や cp932 などなど(iconv(）がサポートしている)任意の コードを指定可能にするってことでいいのですよね？ # iconv でどこまでできるのかは知らないのですが ちょっと時間を作って作業してみましょうかねえ。(週末？) > > パッチ当てるのも人によっては面倒でしょうから CVS に commit しておきました。 > > すみません、CVS の方は試していません。 あっ、それは構いません。同じものですので。 -- 新井康司 (Koji Arai)