Open Jane
Jane Lovely
Jane Style 3つに共通するプログラムのミスについて。
procedure TDaemon.SynchroProcess;内
procedure ProcessLogger;
procedure ProcessCaller;の二種で。
ミューテックス所有直後tmpListを介して、self.reqListとself.savedReqListの参照を交換しているだけの行為(一応実体は2つ)。
ミューテックス解放後にsavedReqList.Clearを行っているので、
2つの別ワーカースレッドが追突 TDaemon.Post (かTDaemon.Log)しに来たらreqList.Addあたりでクラッシュしそう。
TStringList.Assign と TList.Assign とで一時変数 tmpList に割り当ててから、ミューテックス内で self.の方を.Clearすべきでは?
で最後にtmpList.Clearする。savedは廃止。
各オブジェクトのデストラクタの挙動とか知らないから間違ってたらごめん。
Jane Style4.00の問題
POST /v1/auth/ HTTP/1.0 大量ミス。
・Connection: keep-alive
→ closeにしよう。鯖のタイムアウトを待たず自分からコネクションを切断すれば、SSL関連リソースが解放される。
・Content-Type: Content-Type:
→ いつ直すの?今でしょ!
・メインスレッド(VCL)とワーカースレッドの両方がPOSTしている。解せない。
→ datを取りに行ったワーカー含め、これらのうち複数が同時に起動した時に固まりやすい。
・X-2ch-UA: JaneStyle/4.00
→ というわけで?POST毎にこの行が際限なく増える。変数スコープや再入のミス臭い。APIや浪人リトライの局所ループに注意。
・HTTP/1.0かHTTP/1.1を統一せよ。
・Proxyを使用する、における挙動は知らない。
探検
Jane Style Part146 スレ番修正
■ このスレッドは過去ログ倉庫に格納されています
38名無しさん@お腹いっぱい。
2018/05/23(水) 23:24:35.16ID:N16DDRJU0■ このスレッドは過去ログ倉庫に格納されています
ニュース
- 【少子化】少子化は女のせい?男のせい?SNSで不毛すぎる議論が勃発…「産ませる発言」vs「淘汰される遺伝子」泥沼バトルへ ★2 [ぐれ★]
- 植田日銀総裁 「利上げが遅れれば、米欧のように非常に高いインフレが起きて、日本は大幅な利上げが必要となる」 [お断り★]
- へずまりゅう氏が言葉失う 街中で女性から「息子はあなたみたいな人間に育たぬよう教育しています」 [jinjin★]
- 日テレ、国分太一の「答え合わせ」を却下 「答え合わせをするまでもない」「心当たりがあると述べられている」★ 2 [muffin★]
- 【TV】ファン5万人がガチで投票! プロ野球総選挙、栄えある1位は [牛丼★]
- 【速報】政府、与党がNISA未成年解禁を検討 ★2 [蚤の市★]
- 高市早苗「いいから黙って全部アタシに投資しなさい!」国際金融会議で発言し周囲ドン引き [165981677]
- お前ら悪口はビブラートち包めよ
- 早苗のおまんこパンパンパン🏡
- 00:00:00.000
- プラトンの「哲人政治」は正しかったのでは? アホな大衆に政治家を選ばせるとロクなことにならない [653462351]
- すまん「音威子府村」って嫌儲でも上位の不快コテだよな… [114278951]
