■スレッドリストへ戻る■ 全部 1- 101- 201- 301- 401- 501- 601- 701- 801- 901- 最新50

read.cgi改良スレッド

802 ：775 ：01/09/09 01:51
>>801
その通りになってるつもりですがどうでしょうか(^^;

803 ：775 ：01/09/09 01:54
×なやつ全部消していいすかね? バッサリと。

×EXPIRES
×COOKIE
×EXPLICIT_RELEASE ; これは…残す? (^^;
×USE_INDEX ; これは残す?
×ALL_ANCHOR
×USE_INDEX2CGI

804 ：デフォルトの名無しさん ：01/09/09 01:58
>>802
datには"投稿日"なんて文字は元からないのか。
今バージョンでは、"："が表示されてません。

805 ：名無し娘。 ◆vP.bOZFQ ：01/09/09 01:59
>>803
NEWBA,GSTR2 の旧部分も削っちゃってよいと思います。
>>796 NEWBA のかちゅ〜しゃ規制部分も、おそらく大丈夫そうですし。

806 ：775 ：01/09/09 02:00
>>804
こりゃまた失敬。直しました。

807 ：775 ：01/09/09 02:12
とりあえず以下のものをバッサリ削除した。

NEWBAの旧部分
EXPIRES
USE_INDEX2CGI
COOKIE

808 ：775 ：01/09/09 02:21
LASTMODはON固定ってことでcondition消しますか?

GSTR2はすでに昔のコードは無いみたい。1文字か完全一致かの
条件分けになってますね。

809 ：デフォルトの名無しさん ：01/09/09 02:27
imodeのむりやり修正。
なんか根本的に間違っている気がするが、ないと困るので。
いんちきでごめん。

dat_out()中に一行追加。
　if (out_html(level, line, lineNo))
　　break;
+　if (lineNo==1 && is_imode() && nn_st==1) ++out_resN;

810 ：775 ：01/09/09 02:35
>>809
あてた。

811 ： ◆D69Zsbfg ＠夜勤 ★ ：01/09/09 02:36
どんなもんでしょ?
全ファイル(makefile含む)を どこかに上げていただけると
とっても嬉しいです。

で、kaba とかでテストしてみたいです。

まだのようでしたら、また明日　お伺いしますが、、、

812 ： ◆D69Zsbfg ＠夜勤 ★ ：01/09/09 02:36
本当に、みなさんすみませんねぇ、
感謝　感謝　です。

813 ：775 ：01/09/09 02:38
夜勤さん、ご苦労さまです。
最新ソースはcvsから随時
http://www.gedoh.org/aki/2ch/current/bbs/
にコピーされていて、
その全部が read.tgz に圧縮されているようです。

read2ch.h で設定をいろいろいじる必要があるかもしれません。
特にバージョン名（笑）

814 ：775 ：01/09/09 02:39
>>813
あぎゃ。
× ご苦労さまです
○ お疲れ様です

細かいかな(^^;

815 ：775 ：01/09/09 02:48
各設定の説明は
>>796-798
でなされていますが、
現時点の状態で出しちゃっていいんでしょうかね(笑)
他の皆はどうしたんだろう。寝た?

816 ：名無し娘。 ◆vP.bOZFQ ：01/09/09 02:52
>>815
ソース追いきれずにコメント不能状態です(氏
一度どこかに導入してもらって仕切り直せれば、それが一番楽かもしれませんね。

817 ：デフォルトの名無しさん ：01/09/09 02:57
>>784(>>788で補足)に書いてあることは、
ressplitter_split()内の
#if 1
#else
#endif
を
#ifndef シンボル名(NO_LINK_URL_BUSY,etc)
#else
#endif
にすれば、ほぼ実現するんじゃなかろうか？
(レス１の時の動作とi-mode対応がどうかな)

818 ： ◆D69Zsbfg ＠夜勤 ★ ：01/09/09 02:58
まず、全部ダウンロードして、サーバにアップして
make しました。
このように言われました。

gcc -c read.c -DHAVE_READ2CH_H -Izlib -g -O2 -Wall -funsigned-char -o read.o
read.c: In function `dat_read':
read.c:1254: `MAP_FAILED' undeclared (first use this function)
read.c:1254: (Each undeclared identifier is reported only once
read.c:1254: for each function it appears in.)
make: *** [read.o] Error 1

819 ：775 ：01/09/09 03:00
>>818
USE_MMAP時のOS側の定義が足りないようですね。

read2ch.hで#define USE_MMAPをコメントアウトするといかがでしょうか。

820 ： ◆D69Zsbfg ＠夜勤 ★ ：01/09/09 03:02
USE_MMAP コメントアウチおしました。

こんどは、

gcc -c datindex.c -DHAVE_READ2CH_H -Izlib -g -O2 -Wall -funsigned-char -o datind
ex.o
datindex.c: In function `create_index':
datindex.c:372: `MAP_FAILED' undeclared (first use this function)
datindex.c:372: (Each undeclared identifier is reported only once
datindex.c:372: for each function it appears in.)
datindex.c: In function `open_dat':
datindex.c:412: `MAP_FAILED' undeclared (first use this function)
datindex.c: In function `datindex_open':
datindex.c:457: `MAP_FAILED' undeclared (first use this function)
make: *** [datindex.o] Error 1

821 ：デフォルトの名無しさん ：01/09/09 03:02
ってゆーか、Linuxでmmapが失敗したら返す値(シンボル)って何だろ？

822 ：デフォルトの名無しさん ：01/09/09 03:03
現バージョンでの範囲外へのリンクがますいです。
PATHで呼ばれた時は大丈夫だけど、現行形式で呼ばれたときに
read.cgi?bbs=tech&key=998997848101-150#145
なんてことになってしまいます。

転送量削減が主なら、PATHで呼ばれたかをチェックしてリンク先を変えることになるのかな。

823 ： ◆D69Zsbfg ＠夜勤 ★ ：01/09/09 03:03
×　コメントアウチおしました。
○　コメントアウトしました。

824 ：デフォルトの名無しさん ：01/09/09 03:04
>>820
ぐえ。datindex.cって、USE_INDEX時にしか使わないモジュールですね。
これ外しましょう(笑)
Makefileの 9行目、 OBJS = の行で、"datindex.o"を削除してください。

825 ：デフォルトの名無しさん ：01/09/09 03:07
>>821
http://www.linux.or.jp/JM/html/LDP_man-pages/man2/mmap.2.html
一応、MAP_FAILEDみたいですが。

826 ：デフォルトの名無しさん ：01/09/09 03:08
USE_INDEXとか、USE_PATHが#defineされたままだったりして

827 ： ◆D69Zsbfg ＠夜勤 ★ ：01/09/09 03:09
いぇーい

http://kaba.2ch.net/test/read5x.cgi?bbs=news&key=999960057

828 ：デフォルトの名無しさん ：01/09/09 03:10
>>825
ヘッダ構成の違い?

829 ： ◆D69Zsbfg ＠夜勤 ★ ：01/09/09 03:10
>>824 さんに言われた通り
datindex.o　を削除したら、できましたー。

830 ：775 ：01/09/09 03:11
>>827
おお! 動いてる!
read2ch.hはどのように変更しましたか? >夜勤さん
それを元に一旦CVS側で区切り(タグ)を付けたいですね。

831 ：デフォルトの名無しさん ：01/09/09 03:12
次のバージョンでは
#ifndef MAP_FAILED
#define MAP_FAILED (void *)(-1)
#endif
をread2ch.hに入れたほうが良さそうやね

832 ：デフォルトの名無しさん ：01/09/09 03:12
n=tが効いてないような…。
http://kaba.2ch.net/test/read.cgi?bbs=news&key=999960057&st=16&n=t

833 ： ◆D69Zsbfg ＠夜勤 ★ ：01/09/09 03:13
>>830
read2ch.h は、まだ何も変更してないですー。

834 ： ◆D69Zsbfg ＠夜勤 ★ ：01/09/09 03:14
>>832
あっまだ、read.cgi にはしてないです。
read5x.cgi です、

835 ：デフォルトの名無しさん ：01/09/09 03:15
CREATE_NAME_ANCHORとか、有効になってる。
夜勤さん、暇な時にでも>>796-798にある説明を見て、
何をON(define)にして何をOFFにするか決めちゃうと良いと思うです。

836 ：デフォルトの名無しさん ：01/09/09 03:17
ああなるほど、リンクをクリックするとread.cgiのほうに
飛んじゃうのか。自分の名前を取得するようにできる?

837 ：♯6411 ：01/09/09 03:17
>>818 おつかれさまです。

んっと、暇あったら、
$ fgrep -n MAP_FAILED /usr/include/*.h /usr/include/sys/*.h
の実行結果を教えていただけませんか?

何のディストロなんだろ??

838 ：デフォルトの名無しさん ：01/09/09 03:18
>>834 夜勤さんへ。
暫らくread5x.cgiでテストするのでしたら、read2ch.hで

#define CGINAME "read5x.cgi"

にしていただけないでしょうか。

839 ：デフォルトの名無しさん ：01/09/09 03:19
とりあえず、これだと次の10レスが表示されるけど、これでいいのかな？
http://kaba.2ch.net/test/read5x.cgi?bbs=news&key=999960057&imode=true&st=1&to=10

840 ：775 ：01/09/09 03:19
>>838
rewrite_href内でCGINAME使ってない(涙

841 ： ◆D69Zsbfg ＠夜勤 ★ ：01/09/09 03:19
なにもでないです >>837
fgrep -n MAP_FAILED /usr/include/*.h /usr/include/sys/*.h

842 ：♯6411 ：01/09/09 03:21
>>840 …油断してた…

>>841 了解、MAP_FAILED再定義すべきですね。

843 ：775 ：01/09/09 03:22
じゃあ、一度ここで判明したりしたいくつかの修正を加えて
cvsに上げますかね?
それをまた夜勤さんにdownloadしてもらう形とか。

夜勤さん、時間どれぐらいとれますか?

844 ：デフォルトの名無しさん ：01/09/09 03:22
>>837
http://www.maido3.com/server/usagi/ によると
　Linux2.2.19　Slackware
だそうです。

今時だとslackware使っている人は少ないか。

845 ： ◆D69Zsbfg ＠夜勤 ★ ：01/09/09 03:23
>>838
やりましたー
http://kaba.2ch.net/test/read5x.cgi?bbs=news&key=999920210

846 ：デフォルトの名無しさん ：01/09/09 03:25
http://kaba.2ch.net/test/read5x.cgi/news/999960057/

あひゃ
http://kaba.2ch.net/test/read5x.cgi/news/

847 ：デフォルトの名無しさん ：01/09/09 03:25
http://kaba.2ch.net/test/read5x.cgi?bbs=news&key=999920210#20
ここで>>9がなぜか#95にリンクしてる

848 ： ◆D69Zsbfg ＠夜勤 ★ ：01/09/09 03:25
>>843
あせりは禁物かと、、、
また、明日の今ごろ登場予定。。。

849 ：775 ：01/09/09 03:27
>>848
おお。>847 とかも出てるし、明日の夜までに直す形がイイ感じ?

850 ：デフォルトの名無しさん ：01/09/09 03:27
>♯6411氏
digest.[ch]いらなくなったのなら、MakefileのSRCSから消してちょうだい。
あと、datindex.cの中身全部を#ifdef USE_INDEXで括っておくように。(w

851 ：名無し娘。 ◆vP.bOZFQ ：01/09/09 03:30
夜勤さん＆みなさまお疲れさまですー。

MAP_FAILED >>831 でよろしそう。
gzipぱっと見問題なさそう。zlib使えてるってことかな？

852 ：音楽侍 ◆NtVkSITE ：01/09/09 03:30
確認しましたです。＞5x
HTTP1.0と1.1で「掲示板に戻る」のindex.htm(l)の切り替え確認しました。

51-　101-ですが、頭に「１」の書き込み表示がないのは仕様でしょうか？
もしかするとちょっと不便かもしれません。

853 ：デフォルトの名無しさん ：01/09/09 03:32
n=tはキャッシュが効かないからnofirst=trueにしたんじゃ
なかったっけ?

854 ：デフォルトの名無しさん ：01/09/09 03:35
>>848
お疲れ様でした〜

855 ：デフォルトの名無しさん ：01/09/09 03:47
USE_PATHで旧形式を指定されたときはリダイレクトするか
BASEを吐いて常に新形式でリンクするというのはどうします?

856 ：855 ：01/09/09 03:52
いまのところはGSTR2と一緒で隠し機能にしたほうがいいかな

857 ：- ：01/09/09 04:02
http://kaba.2ch.net/test/read5x.cgi?bbs=net&key=998373518&st=151&to=200&nofirst=true

>> のリンクが、最大レス番号を超えてないものでも一部切れてたりするんですが・・・
154 160 171 など

858 ：デフォルトの名無しさん ：01/09/09 04:10
とりあえずtag打っときなさいな。

859 ：857 ：01/09/09 04:11
あれ、仕様でしたか？582あたりの。

860 ：デフォルトの名無しさん ：01/09/09 04:24
>>858
tag名どうしまっか?

861 ：デフォルトの名無しさん ：01/09/09 04:27
>>857って、何かリンクする条件が逆のような気がする
表示範囲内→リンクしない(タグを削除)
表示範囲外→リンクする

とりあえず、今出ている不具合は、↑と
MAP_FAILEDの再定義>>831
リンク先(rewrite_hrefの#)>>847
CHUNK_ANCHORで1-が無い>>852
ぐらいか？

862 ：857 ：01/09/09 04:28
http://kaba.2ch.net/test/read5x.cgi?bbs=net&key=998373518&st=196&to=196&nofirst=true

>> のリンクは read.cgi を read5x.cgi に直したとしても飛べません

863 ：775 ：01/09/09 04:29
>>862
それ、>>840 (涙

864 ：デフォルトの名無しさん ：01/09/09 04:35
>>860
YAKIN0909 とか
ReleaseVersionになるのは、一日一回がせいぜいでしょ。

865 ：775 ：01/09/09 04:37
>>864
YAKIN20010909でtag打っときました

866 ：名無し ：01/09/09 04:43
バージョンを実際の５．ＸＸに合わせるべきだと思います。

867 ：デフォルトの名無しさん ：01/09/09 04:46
今回は、仮リリースだからね。ver14だし。
正式にインストールされた時は、実際のバージョンにすべきだけど。

868 ：775 ：01/09/09 04:50
>>831
read2ch.hだとincludeのタイミングが速かったりするので、read.hに入れた

869 ：857 ：01/09/09 04:51
>>863
えーと、st=xx&to=xxの形式なのに、↓で無条件に xx-xx を付けてるのが問題ではないかと。
わかりづらかったらゴメン

#ifdef USE_PATH
d += sprintf(d,
"<a href=\"%s%d-%d",
depth_expr,
mst, mto);
#else
d += sprintf(d,
"<a href=\"%s&st=%d&to=%d&nofirst=true",
depth_expr,
mst, mto);
#endif

870 ：775 ：01/09/09 04:52
>>840
CGINAME参照するようにした

871 ：775 ：01/09/09 04:53
>>850
>あと、datindex.cの中身全部を#ifdef USE_INDEXで括っておくように。(w
括った(笑)

872 ：デフォルトの名無しさん ：01/09/09 05:05
やっぱ　ネスケ4.72ではgzip圧縮した場合にContent-Lengthが入ってると
Last-Modifiedが無効になるみたい

HTTP/1.1 200 OK
Date: Sat, 08 Sep 2001 20:00:12 GMT
Server: Apache/1.3.6 (Unix) PHP/4.0.3pl1 mod_ssl/2.3.6 OpenSSL/0.9.3a
Content-Encoding: gzip
Last-Modified: Sat, 08 Sep 2001 19:33:13 GMT
Content-Length: 974
Connection: close
Content-Type: text/html

　　　　　　　　　　　　場所: http://kaba.2ch.net/test/read5x.cgi?bbs=news&key=999920210
　　　　　ファイルMIMEタイプ: text/html
　　　　　　　　　　　ソース: 現在、ディスクキャッシュに入っています
ローカル キャッシュ ファイル: M0IH2L6U
　　　　　　　　最終更新日時: 不明
　　　　　　　　最終更新日時: 不明
　　　　　　　　　内容サイズ: 38018
　　　　　　　　　　有効期限: 日付なし
　　　　　　　　　文字セット: Shift_JIS
　　　　　　　　セキュリティ: このドキュメントでは暗号化によるセキュリティ保護は行われていません。

HTTP/1.1 200 OK
Date: Sat, 08 Sep 2001 20:01:35 GMT
Server: Apache/1.3.6 (Unix) PHP/4.0.3pl1 mod_ssl/2.3.6 OpenSSL/0.9.3a
Content-Encoding: gzip
Last-Modified: Sat, 08 Sep 2001 19:33:13 GMT
Connection: close
Content-Type: text/html

　　　　　　　　　　　　場所: http://kaba.2ch.net/test/read.cgi?bbs=news&key=999920210
　　　　　ファイルMIMEタイプ: text/html
　　　　　　　　　　　ソース: 現在、ディスクキャッシュに入っています
ローカル キャッシュ ファイル: MUOF7EA1
　　　　　　　　最終更新日時: 09/09/01 04:33:13 地方標準時
　　　　　　　　最終更新日時: 09/08/01 19:33:13 世界標準時(GMT)
　　　　　　　　　内容サイズ: 41675
　　　　　　　　　　有効期限: 日付なし
　　　　　　　　　文字セット: Shift_JIS
　　　　　　　　セキュリティ: このドキュメントでは暗号化によるセキュリティ保護は行われていません。

873 ：775 ：01/09/09 05:13
>>869
PATHではなく旧形式で呼ばれたときに、USE_PATH時でも旧形式で >> のリンクを
生成するようにしてみた。

874 ：775 ：01/09/09 05:16
んじゃ寝るー。

875 ：名無し娘。 ◆vP.bOZFQ ：01/09/09 06:04
明日あたりに新スレ立ててきますので、それまでにこのスレのまとめが
できるようにお願いいたしますです。

実装・検討待ちの案一覧

gzip/zlib関連conditionの整理
　gzip/zlibをconditionなしで採用し整理

PATH_INFO
　>>869 >>873
　リンク先がいろいろおかしい
　・http://kaba.2ch.net/test/read5x.cgi?bbs=net&key=998373518&st=196&to=196&nofirst=true
　・http://kaba.2ch.net/test/read5x.cgi?bbs=net&key=998373518&st=1&to=10 の 1
　・http://kaba.2ch.net/test/read5x.cgi?bbs=net&key=998373518&st=1&to=1
　・http://kaba.2ch.net/test/read5x.cgi?bbs=net&key=998373518&st=151&to=200&nofirst=true

CHUNK_ANCHORで1-が無い
　>>852

CUTRESLINK, replace系関数
　>>736
　とにかく整理

URL記述に対するリンク設定も時間帯によらせる
　>>817

<a name>をつけるかつけないかをconditionによらせる

"投稿日"非表示を時間帯によらせる

スレスト時FORM
　>>11

dat制限サイズ警告
　>>740

876 ：名無し娘。 ◆vP.bOZFQ ：01/09/09 06:04
mmap時にlockすべきか
　>>139 >>425

行末の空白削除
　>>723

NN4.72でgzip圧縮&&Content-LengthがあるとLast-Modifiedが無効
　>>872

.hからいろいろなくなってるけど？
　>>670
　http://cocoa.2ch.net/test/read.cgi?bbs=unix&key=998695422&st=81&to=81&n=true

キャッシュであぼーんされたレスが見える？(詳細不明)
　http://teri.2ch.net/test/read.cgi?bbs=saku&key=996761078&st=408&to=408&nofirst=true

FORCE_304_TIMEの動的変動
　鯖負荷に応じて動的に変更する

外部設定ファイル
　>>798

dat,kako
　/kako/ に対応
　ツール作者さんに対応お願い
　　* raw=xxx.yyy の形式でリクエスト。 xxx=最終レス番号, yyy=そのときのサイズ。
　　* 一行目はステータス:
　　* [+OK] の場合は差分のみを送信する。
　　* [-INCR] (Incorrect)の場合はすべてのデータを送信する。
　　* [-ERR (テキスト)]の場合はなんかエラーが起きた。

datの一部についてのLastModを求める

877 ：名無し娘。 ◆vP.bOZFQ ：01/09/09 06:05
deflate
　そのうち対応

板ダイジェスト
　そのうち対応

スレッド一覧
　そのうち対応

スレインデックス
　>>680 >>696
　そのうち対応

condition一覧（更新しなくちゃ）
　>>796-798

r2chhtml.hいろいろ
　http://natto.2ch.net/test/read.cgi?bbs=hp&key=999020167&ls=20
　http://natto.2ch.net/test/read.cgi?bbs=hp&key=998774537&ls=20（前スレ）

その他
　http://www.gedoh.org/aki/2ch/tiki/

878 ：デフォルトの名無しさん ：01/09/09 06:55
/dat/が見れない状態が続くならdat制限サイズ警告は
優先度高いですね

879 ：デフォルトの名無しさん ：01/09/09 07:10
>>793でREWRITE_HREF2は削除したって書いてますけど
まだソースに残ってませんか?

880 ：デフォルトの名無しさん ：01/09/09 07:12
>>875
http://kaba.2ch.net/test/read5x.cgi?bbs=net&key=998373518&st=1&to=10 の 1
これは、2-5は現在表示中に全部含まれているので、#2で正しいと思いますが。

881 ：デフォルトの名無しさん ：01/09/09 08:08
lockに関しては　やるとしたらread.cgiではshared lock(F_RDLCK)
bbs.cgi等の書き込み側ではexclusive lock(F_WRLCK)になるんだろうけど
どっちにしろbbs.cgi等の書き込み側の協力がないと......

882 ：デフォルトの名無しさん ：01/09/09 08:17
http://kaba.2ch.net/test/read5x.cgi?bbs=news&key=999911192&ls=50
1000超えのスレッドや停止されたスレッドでは
「新レスの表示」も不要ですね。

883 ：デフォルトの名無しさん ：01/09/09 08:46
#ifdef Katjusha_Beta_kisei
"Kathusha",
#endif

つづりが間違っています。
"Katjusha",

884 ：デフォルトの名無しさん ：01/09/09 09:35
>>883
直した。

>>875
リンクを貼ってくれない場合や、#95などが出る原因を修正。

if (!*zz_http_user_agent && !*zz_http_language)
というのがあるんだが、*が付いてるのは理由があるのか？
NULLチェックのつもりだと思うんだが。

885 ：デフォルトの名無しさん ：01/09/09 09:51
>>884
if (!zz_http_user_agent)
zz_http_user_agent = KARA;
if (!zz_http_language)
zz_http_language = KARA;

ってのが別にあるからな。とはいえ
if ( zz_http_user_agent == KARA || zz_http_language == KARA )
とか
if ( zz_http_user_agent[0] == '\0' || zz_http_language[0] == '\0' )
って感じで分かりやすくして欲しい気はするが..

886 ：デフォルトの名無しさん ：01/09/09 10:32
nofirst=trueの表記を統一しました。
#define NO_FIRST "&n=t" で省略形式に統一可能。

>>879
REWRITE_HREF2の削除もれを消しました。

>>796-798に手を加えてconfig.txtとして追加した。
conditionをいじったら、config.txtもいじってね。
http://www.gedoh.org/aki/2ch/current/bbs/config.txt
実情にあってない所があったら指摘してください。

887 ：881 ：01/09/09 11:06
fcntl()でのshared/exclusive lockだと　read.cgiがひっきりなしに呼ばれていると
bbs.cgi等の書き込み側がexclusive lockを取得できなくなってしまう可能性もあるな...

となると　書き込み側では
　　1. まず別のファイル名でオープンして（例えば"987654321.new"）そこに書き込む
　　2. 書き込みが終わったら rename("987654321.new", "987654321.dat") を実行
って感じでやってもらう方がいいのかも　これならread.cgi側でのlockは
考えなくてもいいんだな　bbs.cgi等の書き込み側の排他制御は必要だろうけど

888 ：デフォルトの名無しさん ：01/09/09 11:58
CUTRESLINK関係の話が、
-DCUTRESLINKで加えられた変更全体を指すのか、
レスのリンクをカットする機能のみを指すのか、若干わかりにくい。

CUTRESLINKの名前をOPTIMIZE_SPLIT等に変更して、
本来のリンクをカットする部分のみを#ifdef CUTRESLINKで囲うことを希望。

889 ：775 ：01/09/09 14:10
>>886
REWRITE_HREF2削除に漏れあったのか。修正さんくす

890 ：775 ：01/09/09 14:17
CUTRESLINKはONの状態で採用ってことでOFF時のコード消さない?

CUTRESLINKまわりの二重状態がいちばんソース見難く、でかくしてる感じだし。

891 ：デフォルトの名無しさん ：01/09/09 14:31
とりあえずの行末の空白削除(タグ消し)対策
ressplitter_split()の、case ' ':
- 　if (bufp == *This->buffers) /* 名前欄が半角空白の場合 */
- 　　*bufp++ = ' ';
+ 　if (bufp == *This->buffers /* 名前欄が半角空白の場合は最低１文字必要 */
+ 　　|| (resnumber && isSJIS1(*bufp-1))) /* 行末の文字化け対応 */
+ 　　*bufp++ = ' ';
/*
resnumberのチェックを外せば名前・メール欄の
直後の空白もある程度消せるようになるが、
厳密に判定しないと(名前欄が)でこぼこに
なってしまうので、もう少し待ち。
*/
転送量的には、"名無しさん"の"ん"の直後の空白が(今のままでは)
削除できないのが痛い(3byte前から調べれば削除できる)ので、
空白が必要かを完全に判定する厳密なルーチン
(末尾から調べ、長くても数文字で終わるはず)を作成予定。
名前とメールの直後の空白が消せるし、
<br>前の空白もまだ結構残っているので、
処理を少し複雑にしてでもやる価値はあると見る。

もう一点。
index.htmlに合わせる場合、
メール欄が"0"の場合は""として扱うほうが良いのかな、と。
その場合、
splitting_copy()の最後に、
+　if (s[1][0] == '0' && s[1][1] == '\0')
+　　s[1][0] = '\0';
つければOK。
以上は、http://teri.2ch.net/accuse/dat/994071363.datを出力して動作確認。

http://teri.2ch.net/test/read.cgi?bbs=accuse&key=994071363&st=448&to=449
の問題は、dat内に'&','<',"http://"が現れるたびに直前の文字をチェックして
' 'を入れるか判定する必要がある割には、実際に壊れている割合は非常に少なく、
費用対効果の点で疑問が残るので、OFF推奨のconditionとして実装予定。

892 ：デフォルトの名無しさん ：01/09/09 14:58
>>890
個人的には賛成。
Ver5.12でも使われているし、ON前提で書かれている機能もけっこうある。

他にも同意する人がいて、反対する人が現れなかったら、
必要なのは「決断」？

893 ：名無し娘。 ◆vP.bOZFQ ：01/09/09 16:20
>>880
その通りでした（＾＾；
>>890 >>892
決断の一助となるべく賛成を表明（＾＾

894 ：775 ：01/09/09 16:21
>>893
賛成多数につき(笑)
やりまーす

895 ：775 ：01/09/09 16:33
>>894
ということでCUTRESLINKをON固定として条件削除しました。

896 ：892 ：01/09/09 16:46
＞賛成多数につき(笑)
笑った。

897 ：892 ：01/09/09 16:51
そういえば、LASTMODもまだON固定にしてなかったよね？
ネスケ関係の問題ってLastModifiedを吐く側には対処できそうにないし、
ON固定にして、万一不具合が出てもその都度対処すればよいのではないかと。

898 ：名無し娘。 ◆vP.bOZFQ ：01/09/09 17:01
>>895
おつかれですー。
>>897
賛成ですー。

899 ：名無し ：01/09/09 17:11
批判要望板では、曜日は表示して〜〜〜とのことです。
会社行く日なのか判断出来なくなっちゃうよ〜〜〜。

900 ：775 ：01/09/09 17:14
>>897
んじゃLASTMODもON固定にしときます。

>>899
確かに曜日賛成。

901 ：775 ：01/09/09 17:21
>>900
LASTMOD ON固定にした。

>>891
+ 　　|| (resnumber && isSJIS1(*bufp-1))) /* 行末の文字化け対応 */
これってSJISの1バイト目でも2バイト目でもつかえるコードがたまたまそこにあると
誤動作するよね?

333KB
新着レスの表示
スレッドリストへ戻る 全部 前100 次100 最新50

0ch BBS 2004-10-30