Hiroto Sakai
Hiroto Sakai
It seems that filename encoding in the archive is ISO-8859-1. The current version of lha does not support it. ``` $ wget http://aminet.net/driver/audio/m68k-amigaos-ahidev.lha $ lha l m68k-amigaos-ahidev.lha ... -rw-r--r-- 901/901...
This is a patch for this issue. Now, we can unpack such archives with `--archive-kanji-code=latin1` option. However, I don't think it is a good solution. [42.diff.txt](https://github.com/jca02266/lha/files/12620859/42.diff.txt)
This is another solution. We can disable filename conversion with the `--disable-convert-filename-encoding` option to avoid the iconv error. [disable-convert-filename-encoding.diff.txt](https://github.com/jca02266/lha/files/12739568/disable-convert-filename-encoding.diff.txt)
K&Rスタイルでは変数宣言はブロックの先頭に書かなくてはならないとされており、今のコードは文法違反ですから、修正した方が良いと考えます。実際、修正前のソースをgcc-2.95.2 (Mac OS X 10.2)でコンパイルすると以下のエラーが出ました。 ``` gcc2 -DHAVE_CONFIG_H -I. -I. -I.. -DEUC -DSUPPORT_LH7 -g -O2 -c `test -f 'bitio.c' || echo './'`bitio.c bitio.c: In function `fillbuf': bitio.c:23: parse error before...
英語が苦手なので日本語で失礼します。 ぜひmergeしていただきたいと思います。一点、build-aux/gen-build-version.shの14行目と15行目でコマンドが違うのに違和感があります。 ``` HASH=$(git log -1 --pretty=format:"%h") ``` で良いのでは。