✅ Build sakura 1.0.2699 completed (commit e57f2b495f by @berryzplus)

修正対象は正規表現で抽出しました。

対象の抽出はそれで良しとして、効果の確認は個別ですよね？

本来コピーが不要な箇所でコピーをしてる事から直したほうが良いと思います。

void CTextInputStream::ReadLineW(CNativeW& line);

それはつまり上記のメソッドを追加する事になりますが、それよりとりあえず早期に不具合を直したい感じでしょうか？

レビューありがとうございます。
書き込み先バッファサイズを考慮してない問題がありますが、
crash対策にはなってると思うので、とりあえずマージしてしまいます。

@beru「動作確認はしてませんが

@beru「変更内容は問題ないと思います。

@beru「バッファオーバーランが起きうる記述のままなのが気になりますが別のPRで対処入れようと思います。

@berryzplus「書き込み先バッファサイズを考慮してない問題がありますが、crash対策にはなってると思うので

この PR がマージされたことにより Issue #1234 が閉じられました。何をもってそれを是としましたか。

@beru さんには特に次のような表現に警戒心を持ってもらいたい。信頼できる者を見極めるために。

@berryzplus「コンテキスト的に line から line.length() 文字分 szAbout にコピーする。

コンテキストで判断するところではありません。定義に基づいて理解すべきです。

@berryzplus「通常 line[length()] は '\0' なので。

「通常」とは？ 通常でないときがあるのなら危険なのでは？

実は知らなかったのですが、operator[] は範囲外アクセスに対して、特に size() と等しい添え字に対しては以前から、デフォルトコンストラクトされた文字への参照を返していたみたいです。意外な優しさ……。

もちろん範囲外は範囲外ですから、at() などは例外を投げます。範囲外にある何がコピーされることを期待したんでしょうか。

@berryzplus「crash対策にはなってると思うので、とりあえずマージしてしまいます。

何も確認していない。

@ds14050 さん
ちゃんと確認しろよ！って事ですね。はい、確認します。少々お待ちください。
まぁでもそのコメントをする手間を掛けられるなら、そちらで確認も出来るのでは…。:sweat:
いやまぁ、俺が何をするかは俺の自由だ、と言われればその通りなんですが。

もちろん落ちることは確認しています。

また、必ずしもレビュアに動作確認をさぼるなとは言いませんが、コードの読解が正しいことが条件になると思います。間違っていますから Issue の方を参照して下さい。

このPRの修正内容が不十分でまだ落ちるという事ですね？
それを確認したならまずそれを報告する事の方が有意義では…。

まぁモグラたたきいくらやってもしょうがないという事もあるので、コード変更したりレビューする側はちゃんと確認しないといけないというのはその通りではあります。

情報提供は Issue の方でやってるんですよ……

1. SJISエンコードのキーワードヘルプ辞書を設定するとき表示化け・Crashする #1234 (comment)
2. SJISエンコードのキーワードヘルプ辞書を設定するとき表示化け・Crashする #1234 (comment)
3. SJISエンコードのキーワードヘルプ辞書を設定するとき表示化け・Crashする #1234 (comment)

読まない、反応しない、理解しないことのフォローまではできません。

情報提供は Issue の方でやってるんですよ……

1. #1234 (comment)
2. #1234 (comment)
3. #1234 (comment)

読まない、反応しない、理解しないことのフォローまではできません。

PRの方にもお願いしますね。

@beru さんは Issue からやってきた私を含む二人と比較して前提知識に不利があったみたいですね。その点は同情します。しかし手を抜くときはくれぐれも相手を見定めて下さい。そこは迂闊だったと思います。

まぁレビューする際にちゃんとロジックと問題を理解せずにコードの差異だけ見て手抜きでヨシ！としてしまうのはザル過ぎるという事ですね。。

引きずられて同じレベルに評価を落とすのはもったいないですから。

いやいや自分は結構適当にやる人間ですよ。

情報提供は Issue の方でやってるんですよ……
1.#1234 (comment)
2.#1234 (comment)
3.#1234 (comment)

読まない、反応しない、理解しないことのフォローまではできません。

なんで自分でPRせずに他人のせいにしてるんですかね？

ローカルビルド環境を破壊されたままの私は、自分ができる範囲のことをしたというだけの報告です。勝手に「責められ」ないでください。

ローカルビルド環境を破壊されたままの私は、自分ができる範囲のことをしたというだけの報告です。勝手に「責められ」ないでください。

それも「PRしたら済む話やん？」なんです。

ごちゃごちゃしてて理解不能だ、とコメントしてあったと思います。
だったらアフォでも分かりそうなレベルまでバラして提示したらいいんだと思います。

もしそれができないってんなら、まだその域に達してないってことなんですよね、結局。
（個人的にはできんわけがなかろう、と思ってます。だから、放置を続けてるんです。）

何を指すかは通じているみたいですね。>ローカルビルド

ごちゃごちゃしてて理解不能だ、とコメントしてあったと思います。
だったらアフォでも分かりそうなレベルまでバラして提示したらいいんだと思います。

もしそれができないってんなら、まだその域に達してないってことなんですよね、結局。

私を絶望にたたき込むのがこの強さです。正論をものともしないこの強さです。

何を指すかは通じているみたいですね。>ローカルビルド

MinGWビルドでgoogletestのパッケージインストールを強要するのは理不尽だ、
という話だと勝手に理解して書いております。

別な話だったらすまんですが理解できてないと思います。

私を絶望にたたき込むのがこの強さです。正論をものともしないこの強さです。

正論には正論を打ち返すのが礼儀ですから。

正論と信じられるところがあなたの強さですよ。

信じると言えば自分も同じ。疑うこと検証すること他者の反応を伺うことをするかどうかが分かれ目でしょうか。