■スレッドリストへ戻る■ 全部 1- 101- 201- 301- 401- 501- 601- 701- 801- 901- 最新50
read.cgi改良スレッド ２
- 36 ： ◆D69Zsbfg ＠夜勤 ★ ：01/09/10 01:25
- いやん、すまぬ、私のミス makefile まちがった。
read.cgi 用のを入れてた。
- 37 ：デフォルトの名無しさん ：01/09/10 01:26
- >>36
なるほど（笑）
- 38 ： ◆D69Zsbfg ＠夜勤 ★ ：01/09/10 01:26
- いぇーい
http://piza2.2ch.net/test/read5x.cgi?bbs=mona&key=1000048768&ls=50
- 39 ：デフォルトの名無しさん ：01/09/10 01:28
- >>38
またCGINAMEをread5x.cgiにしといてくださーい。
- 40 ： ◆D69Zsbfg ＠夜勤 ★ ：01/09/10 01:29
- >>39
しましたー
- 41 ：デフォルトの名無しさん ：01/09/10 01:29
- >>26
>>10のような状況が考えられるので、解除されるのが必ずしも
間違いとは言えない。
それで運用された実績もあるし。
- 42 ：デフォルトの名無しさん ：01/09/10 01:31
- cvsでtag打っときました( YAKIN20010910 )
- 43 ：デフォルトの名無しさん ：01/09/10 01:31
- なかなか良い感じですね。
http://piza2.2ch.net/test/read5x.cgi?bbs=mona&key=999190395
- 44 ：デフォルトの名無しさん ：01/09/10 01:32
- >>41
だから新スレを立てればいいじゃん。
- 45 ：デフォルトの名無しさん ：01/09/10 01:33
- http://piza2.2ch.net/test/read5x.cgi/tech/1000035521/44-
ふむふむ
- 46 ：前スレ857 ：01/09/10 01:34
- 「誤爆だけで・・・」っていうスレ、>> のリンク動作を確認するのに便利だ。
http://piza2.2ch.net/test/read5x.cgi?bbs=ff&key=998319724&st=301&to=350
ここの333だけリンクが有効になってます。今の時間帯だと全部オフが正しい?
- 47 ：デフォルトの名無しさん ：01/09/10 01:37
- >>46
いや、「全部読む」の増加対策で、今見えてる範囲外の
リンクは消さないのが正しい。
- 48 ：デフォルトの名無しさん ：01/09/10 01:38
- 「新レスの表示」サイコー
- 49 ：デフォルトの名無しさん ：01/09/10 01:39
- １は必ず表示した方がいいと思った。
- 50 ：デフォルトの名無しさん ：01/09/10 01:40
- 自分の行ってるスレのアドレスが出てちょっとうれしかった。
それだけ（藁
- 51 ：前スレ857 ：01/09/10 01:45
- 勘違いだったか・・・すまんです。
- 52 ：前スレ857 ：01/09/10 01:57
- http://piza2.2ch.net/test/read5x.cgi?bbs=kao&key=986050461&ls=5
1000を超えてたら、
「サイズが496KBを超えています。512KBを超えると表示できなくなるよ。」
これはいらないかもしれない。
- 53 ：デフォルトの名無しさん ：01/09/10 02:00
- サイズ超え警告が正常に働いてることはとりあえず確認できた
- 54 ：デフォルトの名無しさん ：01/09/10 02:04
- http://piza2.2ch.net/test/read5x.cgi?bbs=mona&key=999190395&st=101&to=201&nofirst=true
の110のレスで >>18 のURLリンクがおかしいです。
http://piza2.2ch.net/test/read5x.cgi?bbs=mona&key=999190395&st=1&to=50&nofirst=true#18
- 55 ：デフォルトの名無しさん ：01/09/10 02:06
- >>54
別におかしくないと思うけど?
- 56 ：デフォルトの名無しさん ：01/09/10 02:08
- >>55
今までの仕様だと
http://piza2.2ch.net/test/read5x.cgi?bbs=mona&key=999190395&st=18&to=18&nofirst=true
じゃないの？
- 57 ：55 ：01/09/10 02:10
- ごめん、おかしいね。1が表示されてない。
こういう場合&nofirst=trueが付いてると上の「1-」とリンク先が
共通にならなくてキャッシュの効果が落ちると思うけどどうだろ。
- 58 ：デフォルトの名無しさん ：01/09/10 02:12
- >>56
それはキャッシュの利用効率を高めるためのCHUNK_ANCHOR
の仕様。でもせっかくCHUNK単位でレスを区切っても&nofirst=true
が付いていては意味がないと思われ
- 59 ：56 ：01/09/10 02:16
- >>58
index.htmでの表示と違うのは何とも・・・
操作と結果は統一して欲しい気が。
スレ違いだったらごめんなさい。
- 60 ：デフォルトの名無しさん ：01/09/10 02:52
- >>59
それはテレホタイムにリンクをカットしてる時点ですでに
違ってるわけだし。
bbs.cgiのほうを修正できないとなんとも。
- 61 ：デフォルトの名無しさん ：01/09/10 02:56
- とりあえず今日わかったことは
・トップの1-51-等の表示のために、CHUNK_ANCHORはONにしたい
・>>xxxのリンク先がCHUNKEDになるのは嫌だ。
↓
仕方がないから、新しいcondition(CHUNK_RESLINK)をつけ、
>>xxxのリンク先をCHUNK単位に直すのはCHUNK_RESLINKの時だけにしよう
CHUNK_RESLINKがdefineされてCHUNK_ANCHORがdefineされてなかったらエラー。
(CHUNK_ANCHORは再定義できるけど、CHUNK_NUMの再定義は数値が必要だから)
・レス数とサイズの警告は、両方出てもよい。
・1000 Overの場合は、サイズ表示はいらない。
↓
#ifdef CAUTION_FILESIZE の直後に if (line <= RES_RED) {
#endif の直前に }
NAME_ANCHORは、あってもいいのかなぁ
- 62 ：デフォルトの名無しさん ：01/09/10 03:16
- BadAccessがrawmodeのときでもツールを弾いている気
がするんですが、ツールに読ませるためのモードが
ツールを弾くのはまずいのでは…。
- 63 ：デフォルトの名無しさん ：01/09/10 03:17
- 「名前：」をとっとと消せよ
- 64 ： ◆D69Zsbfg ＠夜勤 ★ ：01/09/10 03:19
- piza2 に全面的に導入しました。
ver 5.20 にしました。
- 65 ：デフォルトの名無しさん ：01/09/10 03:20
- >>xxxのリンク先を広げるconditonは、CHUNK_ANCHORより
CREATE_NAME_ANCHORであるべきではないでしょうか。
そうすればconditionを追加する必要もありませんし、
>>xxxのリンク先を広げる必要がなければname anchorも
いりませんからそのほうが合理的だと思います。
- 66 ：65 ：01/09/10 03:24
- よく考えるとCHUNK_ANCHORが有効でないと
>>xxxのリンク先を広げる意味もないか…。
- 67 ：デフォルトの名無しさん ：01/09/10 03:26
- CHUNK_ANCHORのとき、上に表示するリンクは範囲外の
ものだけにしたほうがいいと思う。
(たとえば「レスを全部読む」の場合にはぜんぜん必要ない)
- 68 ：デフォルトの名無しさん ：01/09/10 03:31
- それから現在の表示範囲より下のCHUNKへのアンカーは
上ではなくて下に表示したほうがいいと思うんですが…。
- 69 ：デフォルトの名無しさん ：01/09/10 03:31
- MAX_FILESIZE_BUSYが正式に採用されるまでの間、
dat_read()の最初
#ifndef MAX_FILESIZE_BUSY
if (zz_fileSize > MAX_FILESIZE)
html_error(ERROR_TOO_HUGE);
#else
if (zz_fileSize > MAX_FILESIZE || (isbusytime && zz_fileSize > MAX_FILESIZE_BUSY))
html_error(ERROR_TOO_HUGE);
#endif
正式にヘッダに入ったら、
Settingsにも加えたほうがいいでしょう。
(メンバー変数、初期値、#undef + #define、config.txt)
- 70 ： ◆D69Zsbfg ＠夜勤 ★ ：01/09/10 03:41
- ちょっと 質問でーす
teri にも ver 5.20 いれたんですけど、>>12 というのだけを
一日中有効にするのは、どうすれば良いのでしょうか?
- 71 ： ◆D69Zsbfg ＠夜勤 ★ ：01/09/10 03:43
- つまり >>レス番 の機能を teri サーバだけは一日中
有効にしたいのです。
- 72 ：デフォルトの名無しさん ：01/09/10 03:44
- >>70
read2ch.hの
#define LINKTAGCUT
をコメントアウトすればいいのではないかと…
- 73 ：72 ：01/09/10 03:46
- ごめんなさい、
#define LINKTAGCUT (0)
に変更、の間違いでした。
- 74 ： ◆D69Zsbfg ＠夜勤 ★ ：01/09/10 03:47
- >>72
さんきゅです。
さっそく teri でやってみます。
- 75 ： ◆D69Zsbfg ＠夜勤 ★ ：01/09/10 03:47
- >>73
了解でーす
- 76 ：デフォルトの名無しさん ：01/09/10 03:49
- ＞夜勤さん
Ver5.20と見た目、機能はほぼ同じで、
・>>nnのリンク先が広がるのを避けたい場合
read.cの440行目付近の
#if defined(CHUNK_ANCHOR) && defined(CREATE_NAME_ANCHOR)
を
#if 0
に変えるとよいです。
LINKTAGCUTは、例えば、
/accuse/SETTING_R.TXTに
LINKTAGCUT=0
と書いて置いておく方法もあります。
- 77 ： ◆D69Zsbfg ＠夜勤 ★ ：01/09/10 03:53
- どもです。
削除依頼板は、>>xx が効かないと、効率が悪いらしいです。
とりあえず、今日は >>73 をやりました。
- 78 ：デフォルトの名無しさん ：01/09/10 04:29
- 批判要望よりコピペ
697 名前：名無しさんの声 投稿日：01/09/10 04:20 ID:mhbyzSZQ
「新レスの表示」の
nofirst=true
は無いほうがいいな
- 79 ：デフォルトの名無しさん ：01/09/10 08:59
- ブランチは作らないの？
- 80 ：デフォルトの名無しさん ：01/09/10 09:52
- read.cgiではすでに行っているんですが index.htmlの方でもトップのJavaScriptのところで
----------------------------------------------------------------------
function ol(e){
var N=getCookie("NAME"),M=getCookie("MAIL"),i;
with(document) for(i=0; i<forms.length; i++)
if(forms[i].FROM&&forms[i].mail)
with(forms[i]){FROM.value=N;mail.value=M;}
}
onload=ol;
----------------------------------------------------------------------
というのを入れて その代わり投稿フォームのところでのdocument.write()を
やめてはどうでしょうか？ 一番上の
NameMail = "<INPUT TYPE=text NAME=FROM SIZE=19 VALUE=" + getCookie("NAME") + "> E-mail：<INPUT TYPE=text NAME=mail SIZE=19 VALUE=" + getCookie("MAIL") + ">";
と 各投稿フォームのところでの
<SCRIPT Language="JavaScript"><!--
document.write(NameMail);// --></SCRIPT><NOSCRIPT></NOSCRIPT>
が削れる分 トータルでは節約になると思いますが
- 81 ：デフォルトの名無しさん ：01/09/10 09:59
- >>80
スレ違いです
- 82 ：デフォルトの名無しさん ：01/09/10 10:05
- rawmodeのときは、圧縮に対応していないクライアントは
弾くようにしませんか?
rawmodeで非圧縮のデータを返すメリットってほとんど
ないように思うのですが…。
- 83 ：デフォルトの名無しさん ：01/09/10 10:30
- 全レス表示規制中に「次の100レス」でたどっていくと、
表示範囲が
1-100
100-199
199-298
：
というふうに変化していきますが、CHUNK_ANCHORとの
噛み合わせが悪いので
1-100
101-200
201-300
：
と増えていくようにしませんか?
(というかそもそも「次の100レス」の代わりに
「101- 151- 201-…」を下に持ってくればいいと思う)
- 84 ：デフォルトの名無しさん ：01/09/10 10:45
- read.cgi 5.20でも
http://piza2.2ch.net/test/read.cgi/mona/
で[ここ壊れてます]が連打する現象が解消されていません。
http://piza2.2ch.net/test/read.cgi/mona/?raw=0.0
とするとわかりますが、どうもsubject.txtを無理やり分解して
表示しようとしている模様。
- 85 ：デフォルトの名無しさん ：01/09/10 11:07
- >>84
#ifdef USE_PATH
/* スレ一覧を取りに逝くモード */
if (1 <= path_depth && path_depth < 3) {
sprintf(fname, "../%.256s/subject.txt", zz_bs);
zz_fileLastmod = getFileLastmod(fname);
}
#endif
こんなコードが入ってる割にこれ解析してないようだな。
これが原因か。
- 86 ：デフォルトの名無しさん ：01/09/10 11:20
- ↓こういうのって
<a href="read.cgi?bbs=tech&key=998997848&st=1&to=50">1-</a>
<a href="read.cgi?bbs=tech&key=998997848&st=51&to=100&nofirst=true">51-</a>
<a href="read.cgi?bbs=tech&key=998997848&st=101&to=150&nofirst=true">101-</a>
<a href="read.cgi?bbs=tech&key=998997848&st=151&to=200&nofirst=true">151-</a>
<a href="read.cgi?bbs=tech&key=998997848&st=201&to=250&nofirst=true">201-</a>
<a href="read.cgi?bbs=tech&key=998997848&st=251&to=300&nofirst=true">251-</a>
↓こうすると
<form><select onchange='
var st=parseInt(this[selectedIndex].text);
location="read.cgi?bbs=tech&key=998997848&st="+st+"&to="+(st+49)+"&nofirst=true";
'>
<option>1-
<option>51-
<option>101-
<option>151-
<option>201-
<option selected>251-
</select></form>
スリム化できるような気がする 現在表示中の範囲の記事番号のところは
<option>でselectedを入れておくと
- 87 ：デフォルトの名無しさん ：01/09/10 11:26
- PATH仕様が標準になってくれればなー。
<a href="1-50">1-</a>
<a href="51-100">51-</a>
<a href="101-150">101-</a>
<a href="151-200">151-</a>
<a href="201-250">201-</a>
<a href="251-300">251-</a>
それでもFORMにしたほうが若干小さくなるかな。
ただFORMだと>>67-68 >>83のような工夫がしづらくなる。
- 88 ：デフォルトの名無しさん ：01/09/10 11:38
- >>82の修正
BadAccessの最初に
if (rawmode && !gzip_flag)
return 1;
を追加。
- 89 ：デフォルトの名無しさん ：01/09/10 11:41
- >>88
if (rawmode)
return !gzip_flag;
のほうがよくないかな?
- 90 ：88 ：01/09/10 11:45
- あ、
#ifdef GZIP
#endif
で囲む必要があるか。
- 91 ：88 ：01/09/10 11:47
- >>89
>>62の修正も合わせてるわけね。納得。
- 92 ：デフォルトの名無しさん ：01/09/10 12:08
- >>85
・・・USE_PATHをundefすれば根本解決しそうな、、、
- 93 ：デフォルトの名無しさん ：01/09/10 12:18
- >>85 と >>89 あてた(85は#if 0にしただけ, 89はRAWOUTでも囲んだ)
- 94 ：デフォルトの名無しさん ：01/09/10 12:24
- >>92
USE_PATHしたときだけこんな処理が入るほうが変だと思うけど…
そもそもこのコードいつ誰がどういう目的で追加したの?
- 95 ：デフォルトの名無しさん ：01/09/10 12:28
- >>94
そのコードの下のほうでインデックス処理とかがあるようだけど
そっちは#if 0 /* #ifdef USE_PATH */で無効化されてたのに、
そのためにsubject.txtを読むように準備する部分が生き残ってたようだ。
- 96 ：デフォルトの名無しさん ：01/09/10 12:32
- こっちだけ殺して>>85は殺してなかったわけね
#if 0/* #ifdef USE_PATH */
else if (path_depth == 2) {
if (zz_ky[0] == '-')
dat_out_subback();/* スレ一覧 */
else
dat_out_index();/* 板ダイジェスト */
}
#endif
- 97 ：デフォルトの名無しさん ：01/09/10 12:39
- 板ダイジェストにcondition付けませんか?
#if 0で無効にしてるのって気持ち悪い
- 98 ：デフォルトの名無しさん ：01/09/10 12:41
- >>97
いらんだろ、
もともと余計な機能だったんじゃないか？
- 99 ：デフォルトの名無しさん ：01/09/10 12:52
- config.txt
×DIGEST
板ダイジェストを有効にする。
USE_PATHをONにしたときにのみ使用可能。
/test/read.cgi/tech/ 板ダイジェスト(index2.html相当)
/test/read.cgi/tech/- 全スレ一覧(subback.html相当)
read.cへの変更点
#if 0 /* #ifdef USE_PATH */
↓
#if defined(USE_PATH) && defined(DIGEST)
- 100 ：デフォルトの名無しさん ：01/09/10 12:54
- >>98
そうだね。じゃあ
#if 0 /* #ifdef USE_PATH */
となってるconditionはバッサリ削除
digest.c digest.hも削除。Makefileからも取り除く
ということで。
- 101 ：デフォルトの名無しさん ：01/09/10 12:57
- >>99-100
すでにMakefileのOBJSにはdigest.oは入ってなかったみたいだ。
従ってこのままではたとえconditionにしても使えん。
- 102 ：デフォルトの名無しさん ：01/09/10 13:01
- 本当だ。SRCSには残ってるのに。
じゃあやっぱりバッサリ削りましょう。
- 103 ：デフォルトの名無しさん ：01/09/10 13:02
- >>94
cvs見ると、6411さんが8/30に作ったようだが
- 104 ：デフォルトの名無しさん ：01/09/10 13:07
- >>85は、#if 0で囲うと、初期化されていないメモリを参照して、どうなるかわからない。
if (1 <= path_depth ...) {
#if 1 /* #ifndef USE_PATH_DIGEST */
html_error(ERROR_NOT_FOUND);
#endif
・・・
}
って感じにしてくれ。
理由は、皆が言うように、後半で参照されている部分が#if 0で(仮に)殺してあるため。
Expire関係の#if の#else節も生きているようだが、
これもいらない。
もうひとつ、細かいことだが
mainの最初の方に
if (st == 1 && to == 1)
zz_nf[0] = '\0';
というのがあるが、
CHUNK_ANCHORで1-50へのリンク(>>nn)にnofirst=trueがつくことを考えると、
if (st == 1) /* レス１から表示する場合はnofirst=falseにする */
zz_nf[0] = '\0';
の方が良いだろう。
- 105 ：デフォルトの名無しさん ：01/09/10 13:07
- 前スレですでにこういう話が出てた。
850 名前：デフォルトの名無しさん 投稿日：01/09/09 03:27
>♯6411氏
digest.[ch]いらなくなったのなら、MakefileのSRCSから消してちょうだい。
あと、datindex.cの中身全部を#ifdef USE_INDEXで括っておくように。(w
- 106 ：デフォルトの名無しさん ：01/09/10 13:10
- >>104
> >>85は、#if 0で囲うと、初期化されていないメモリを参照して、どうなるかわからない。
これはどういうこと?
fnameもzz_fileLastModもその前後で設定されているが。
- 107 ：デフォルトの名無しさん ：01/09/10 13:12
- ダイジェクト削除するならr2chhtml.h後半の
/* スレダイジェスト用HTML */
/* スレインデクス用HTML */
も不要。
- 108 ：デフォルトの名無しさん ：01/09/10 13:14
- >>106
失礼。その通りだった。
後でエラーになるね。
- 109 ：デフォルトの名無しさん ：01/09/10 13:22
- >>52修正
#ifdef CAUTION_FILESIZE 直後の行を変更
if (zz_fileSize > MAX_FILESIZE - CAUTION_FILESIZE * 1024) {
↓
else if (zz_fileSize > MAX_FILESIZE - CAUTION_FILESIZE * 1024) {
- 110 ：デフォルトの名無しさん ：01/09/10 13:26
- >>109
あてといた
- 111 ：デフォルトの名無しさん ：01/09/10 13:37
- >>104でも言ってるけどEXPIRESは全廃したんだから
これはいらないんじゃない?
char expires_str[1024];
#if 0
/* XXX これはウソ、Expires: は、
現在時間を基準にすべきである */
get_lastmod_str(expires_str, zz_fileLastmod + 5);
#else
{
/* ためしに廃棄期限をちょっと先に設定してみる */
time_t nw;
time(&nw);
get_lastmod_str(expires_str, nw + 10);
}
#endif
- 112 ：デフォルトの名無しさん ：01/09/10 13:39
- >>109より、
>>61の後半の形式の方がいいと思う。
>>109だと、レスが930まで来た時に突然読めなくなる可能性あり。
- 113 ：デフォルトの名無しさん ：01/09/10 13:40
- >>111
expires_strの宣言とともに削除した
- 114 ：デフォルトの名無しさん ：01/09/10 13:43
- >>112
賛成。
- 115 ：110 ：01/09/10 13:50
- >>112
そのようにあてた
- 116 ：デフォルトの名無しさん ：01/09/10 14:11
- >>104の最後の修正も当てたほうがいいと思うんだけどどう?
当てるなら、get_path_infoの最後のほうの
/* nofirstの仕様をごまかすためのkludge XXX */
if (!zz_nf[0])
strcpy(zz_nf,
(atoi(zz_st) == 1
? "false"
: "true"));
こっちは不要になる。
- 117 ：116 ：01/09/10 14:15
- いやごめん、やっぱり必要。
- 118 ：デフォルトの名無しさん ：01/09/10 14:22
- r2chhtml.hで
./%s? (←%sにはCGINAMEが入る)
となってる箇所がいくつかあるけど、「./」は不要。
消せば多少は転送量の節約になる。
- 119 ：デフォルトの名無しさん ：01/09/10 14:34
- >>118
細かいな(笑)
当てた
- 120 ：デフォルトの名無しさん ：01/09/10 16:28
- imode=true&ls=11や
imode=true&st=nnで nn+RES_IMODE==lineMax-1だった場合に
RELOADLINKが表示されずに「次の１０レス/最新レス１０」が表示されるバグの修正
out_html()の最後、
if (out_resN > RES_IMODE) {
のすぐ下に
if (lineNo != lineMax)
(その下のpPrintfをifの条件下に入れる)
dat_out()のループ内の判定がisprintedと重複していることもあり、
修正ついでに全体を少し整理。
int dat_out(int level)
{
int line;
int threadStopped=0;
char *s[20];
char p[SIZE_BUF];
for (line = 0; line < lineMax; line++) {
int lineNo = line + 1;
if (!isprinted(lineNo))
continue;
if (out_html(level, line, lineNo)) {
line++;
break; /* 非１が返るのは、エラー時とimodeのMaxに達した時 */
}
if (lineNo==1 && is_imode() && nn_st==1)
++out_resN;
}
out_html1(level); /* レスが１つも表示されていない時にレス１を表示する */
・・・
最後の #ifdef RELOADLINK 内部の lineLast(２箇所)をlineに変更
・・・
}
- 121 ：デフォルトの名無しさん ：01/09/10 16:33
- コメント違ってたー
×非１が返るのは
○非０が返るのは
コメントの間違いといえば、
isurltailのstrncmp(""")は、
'"'で囲まれたURLの末尾判定だと後で気がついた。
恥ずかしいので修正してもらえるとうれしかったり。
- 122 ：120,121 ：01/09/10 16:36
- isurltail
→geturltaillen
だし。逝ってきます。
- 123 ：デフォルトの名無しさん ：01/09/10 17:05
- >>120-122
当てた
一時2800行あったread.cは、現在2166行。だんだん減ってるな(笑)
- 124 ：デフォルトの名無しさん ：01/09/10 17:33
- #if 0の整理(ダイジェスト関連以外)
#if 0
if (!is_imode()) {/* no imode */
pPrintf(pStdout, "<DL>");
}
#endif
が2箇所あるけど、すでに同等の処理はhtml_head()に吸収
されているので不要。いい加減削除したほうがいいと思われ
#if 0
/* put 2048byte */
/* もう要らないんじゃないかな、
XXX 追試求む */
while (whitespace--)
gzputc(pStdout, ' ');
#endif
頭切れの報告がないのでそろそろ削除しても大丈夫だと思われ
- 125 ：デフォルトの名無しさん ：01/09/10 17:51
- 今のget_path_infoだと
/4 (st=4&to=4)
/4- (st=4)
/-6 (to=6)
/4-6 (st=4to=4)
しかないんだけど、
/. (nofirst=faluse) /5とかでnofirst=trueを基本にしたいから逆条件
/=10 (ls=10) /-10ってls=10扱いって話があったはずなんだけどな
/i (imode=true) path_infoでimodeは実装不充分だけどね
なんてのを追加し、かなり自由フォーマットで書けるようにしてあるんだけど
commitしていいかな。
.や=について意見求む。
- 126 ：名無し娘。 ◆vP.bOZFQ ：01/09/10 18:01
- 細かいことひとつだけ。
<r2chhtml.h>
#define R2CH_HTML_T_RELOAD(path) \
"<hr><center><a href=\"" path "\">新レスの表\示</a></center><hr>"
を
#define R2CH_HTML_T_RELOAD(path) \
"</dl><hr><center><a href=\"" path "\">新レスの表\示</a></center><hr><dl><dt><dd>"
にした方が、見た目が気持ちいいと思います（<hr>〜<hr>が<dd>にかかっているので
何となく気持ちが悪くて）。
HTML的には
#define R2CH_HTML_FOOTER \
"</dl><p>" CGIVER "</body></html>"
の</dl>をread.cの
#ifdef RELOADLINK
if (!level && lineMax == lineLast) {
html_reload(lineLast);/* Button: Reload */
}
#endif
の前あたりにもってきた方がすっきりしますが、どちらがいいかはおまかせします。
- 127 ：名無し娘。 ◆vP.bOZFQ ：01/09/10 18:15
- もちょっと。
>>77
同感です。あと、>>xx でchunk単位で50表示されるのは、若干のキャッシュ効果が
あるのかもしれませんが、それでも邪魔な気がします。
>>78
なにかと nofirst=true になっているのだけど、ない方がいいような気がしますね。
# 1も表示してくれた方がうれしいということです。
# スレ違い抑制のためにも役立ちますし。
>>83
賛成。
- 128 ：名無し娘。 ◆vP.bOZFQ ：01/09/10 18:26
- もいっこ。
target="_blank" 消えちゃってますね。すごく不便（＾＾；
- 129 ：デフォルトの名無しさん ：01/09/10 18:29
- >>125
/=10 でls=10扱いってのはいいね。
でも . とか i ってのは保留にしたいところ?
- 130 ：125 ：01/09/10 18:41
- >>129
.(nofirst=false)の扱い次第でプログラムの方針がかわるので
/10-20とかでnofirstをどうするか決めないといけない。
/10 (nofirst=true)
/10-20 (nofirst=false) #ここが問題
/100- (nofirst=false) #ここも同様
/-100 (nofirstはどうでもよい。1があるから）
/=10 (nofirst=false)
/n (nofirst=trueを強制)
/. (nofirst=falseを強制)
うーん。/10でnofirst=trueにしたいから変になってるなあ。
- 131 ：デフォルトの名無しさん ：01/09/10 19:17
- >>124
これ当てた(削除した)
- 132 ：(ﾟДﾟ)ﾊｧ?スレ発起人 ：01/09/10 20:16
- また，スレタイトルの末尾の空白を取ったようですが...
http://saki.2ch.net/test/read.cgi?bbs=entrance&key=1000120416
- 133 ：(ﾟДﾟ)ﾊｧ?スレ発起人 ：01/09/10 20:23
- リモホばっちり出てた(;´Д｀)
- 134 ：デフォルトの名無しさん ：01/09/10 22:36
- >>67-68に対応してみた。
批判要望板のレスも読んで、
テレホ外は「レスを全部読む」を入れた。
最新レスnnも入れ、その分CHUNK_ANCHORの数を減らした。
>>86も試験的にコードにしてみたが、
「レスを全部読む」等との統一感がないのでOFFにしてある。
(つーか、俺HTML知らないし。)
変更が大きいのでコンパイルが通るかだけでも確認できる人、
大変だけど、お願いできるかな？
http://piza2.2ch.net/test/read.cgi?bbs=tech&key=998997848&st=944&nofirst=true
- 135 ：デフォルトの名無しさん ：01/09/10 22:39
- それと、>>132の不具合は
r2chhtml.hの
#define R2CH_HTML_HEADER_1
#define R2CH_HTML_IMODE_HEADER_1
#define R2CH_HTML_HEADER_2
#define R2CH_HTML_HEADER_2_I
とかだね。
直せる人が見ればすぐ直せると思う。
325KB
新着レスの表示
スレッドリストへ戻る 全部 前100 次100 最新50
0ch BBS 2004-10-30