サクラエディタふぁんくらぶ part19

でも std::min には意味がないし(いつでも cchAvaialable でいい)

>ttps://github.com/sakura-editor/sakura/blob/aaad2522e50880aec5a038533f60151ad3303b00/sakura_core/typeprop/CImpExpManager.cpp#L678
コピーした長さとして ncpyLength + 1 を足してるあたり、おかしなことをしてる。

これでは cchAvailable が負になることがある。

それをクリアしたとして、wcsncpy_s の numberOfElements パラメータが o のとき
count パラメータが _TRUNC であったとしてもエラーを抑止できるかはわからない。
(切り詰めて)ヌル文字を書き込むことができないのだから。

正常系だけ対処して異常系はまた後でとか、マジか

✖cchAvailable
○cchAvailableSize

>>3
フォントは [設定(O)]→[フォント設定(F)] で変更可。

フォントは個人的にVLゴシック( http://vlgothic.dicey.org/ )をオススメしてみたい。

てめえか！おらあ！！

うざい！こんなところに書いてんじゃねぇよ

> https://github.com/sakura-editor/sakura/blob/aaad2522e50880aec5a038533f60151ad3303b00/sakura_core/typeprop/CImpExpManager.cpp#L674

ここの if 条件で除外されてるから

> https://github.com/sakura-editor/sakura/blob/aaad2522e50880aec5a038533f60151ad3303b00/sakura_core/typeprop/CImpExpManager.cpp#L674

ここの std::min の結果は常に同じ。

では min でもってチェックしたつもりで漏れているチェックは
ncpyLength+1 < cchAvailableSize のばあい。
このとき収まりきらなかったキーワードは黙って切り詰められている。

そもそも ncpyLength を事前に計っているのだから
wcscpy_s を呼び出す前に結果がわかる。

9

newをfree、mallocをdeleteしてはいけないのを宗教とか言ってるヤバい認識の人がいるんだけど……

態度悪い開発者晒してこうぜ

>>30
態度悪いやつ晒してこうぜ
傲慢な態度取られたらプロジェクトのためにならんだろ

>>50
うわぁ、ほんとだ。
自分のフシアナっぷりにビックリ！
五年くらいそのまま使ってました。
ありがとうございます。

プログラマなんて出勤して賃金貰ってるだけの引きこもりニートだからな

鼬害啼貴気体

乙です

連投すまんね。まとめると致命的なのは

ttps://github.com/sakura-editor/sakura/blob/aaad2522e50880aec5a038533f60151ad3303b00/sakura_core/typeprop/CImpExpManager.cpp#L674
と
ttps://github.com/sakura-editor/sakura/blob/aaad2522e50880aec5a038533f60151ad3303b00/sakura_core/typeprop/CImpExpManager.cpp#L678
が噛み合ってないこと。

思わせぶりな書き方をしてしまったけど、そこをクリアしたら cchAvailableSize が
ゼロになったり落ちたりということはない。

あとは長さを事前に計ってるなら wcsncpy_s を呼ばずに済ませられるケースも
あるんじゃないかという効率の話なので大したことではない。

master にリンクを張るとそのときそのときの最新版に移って行ってしまうから
現在のスナップショットとして refs/heads/master がポイントしてるコミットに
あえてリンクを張ってる。コミットの内容は無関係。

>githubでやれ

損しかしないので勘弁してほしい。

ここにもいるけど「社会人が」とか「馬鹿呼ばわり」とか、
事実認識をほっぽって属人的な部分でしか反論できないのがいるんだ。
「理解できない。理解させられないのが悪い」も２回聞いた。
なんか全部 >>16 に向かって書いてるみたいだ。気い悪くせんといてな。

連投すまんね。まとめると致命的なのは

ttps://github.com/sakura-editor/sakura/blob/aaad2522e50880aec5a038533f60151ad3303b00/sakura_core/typeprop/CImpExpManager.cpp#L674
と
ttps://github.com/sakura-editor/sakura/blob/aaad2522e50880aec5a038533f60151ad3303b00/sakura_core/typeprop/CImpExpManager.cpp#L678
が噛み合ってないこと。

思わせぶりな書き方をしてしまったけど、そこをクリアしたら cchAvailableSize が
ゼロになったり落ちたりということはない。

あとは長さを事前に計ってるなら wcsncpy_s を呼ばずに済ませられるケースも
あるんじゃないかという効率の話なので大したことではない。

master にリンクを張るとそのときそのときの最新版に移って行ってしまうから
現在のスナップショットとして refs/heads/master がポイントしてるコミットに
あえてリンクを張ってる。コミットの内容は無関係。

>githubでやれ

損しかしないので勘弁してほしい。

ここにもいるけど「社会人が」とか「馬鹿呼ばわり」とか、
事実認識をほっぽって属人的な部分でしか反論できないのがいるんだ。
「理解できない。理解させられないのが悪い」も２回聞いた。
なんか全部 >>16 に向かって書いてるみたいだ。気い悪くせんといてな。

>>15
宗教ではないし戦争にもならないと思う。

今日『C++オブジェクトモデル 内部メカニズムの詳細』を読んでいて、
cfront に関わっていた人が書いた本なんだけど、delete と delete[] が
分離しているのは、いつでも delete[] 相当の処理をしたくないから、
配列のサイズを探り出してから必要な回数デストラクタを呼び出して
メモリを解放するというコストを払いたくないから、プログラマに使
い分けの面倒がふりかかることになったというようなことが書かれて
いた(195ページ)。

だから仮に主張するなら、コンストラクタ・デストラクタの呼び出しという
new/delete の仕事の半分を無視していて明らかに誤っている

> MSVCの実装だとnew/deleteとmalloc/freeは同じなので気にしなくていいと思います。

ではなくて、「delete[] は delete よりコストのかかる上位互換なので〜」
だったと思う。これが現在未来において正しいと信じられるならば、だけど。

なんかワロタw

✖これでは cchAvailable が負になることがある。

不正確だけど符号のあるなしについてはもう指摘済みなので……。

10

さて、現在この５ちゃんねる「ソフトウェア」板では一体どこまで保守すればいいんだろ？
10レスくらいまで保守すればいいのかな？
でも今ちょっと俺のほうが都合が悪くなってきた。しばらく退散。あとはヨロシク。

✖_TRUNC
○_TRUNCATE

githubでは大人ぶったいい子ちゃんコメントして
5chで匿名でぶっ叩くの気持ちいいです(＾＾)

>>1
乙

Grepの基準フォルダですが、初期値が前回実施時のものになります。
これを今開いているファイルの格納フォルダが自動的に設定されるようにする方法はありますでしょうか。
（秀丸を使っていたときはこんな挙動でした。）
Alt+Uを毎回押すのが億劫で、、、
詳しい方、ご教示いただけると助かります。

>>42
お前がやれよ
もしくは5chをフォークして6ch作ってそっちに移住しろよ

>それをクリアしたとして、wcsncpy_s の numberOfElements パラメータが o のとき
> count パラメータが _TRUNC であったとしてもエラーを抑止できるかはわからない。

(古い開発環境だけど)落ちた。

ちなみに俺は以下の1行だけ記述したGREP用バッチファイルをSendToフォルダに入れて使う事がある。
↓
START 【sakura.exeのフルパス】 -GREPMODE -GREPDLG -GOPT=SRP2 -GCODE=99 -GFILE=”*.*” -GFOLDER=”%‾f1″

気分次第で、以下の1行を記述したGREP用バッチファイル(サクラエディタの代わりにEmEditorが必要)も時々使う。
↓
START “” “C:¥Program Files¥EmEditor¥EmEditor.exe” /fr /fd “%‾f1”

※Windows7〜8.1のSendtoフォルダ ＝ “C:¥Users¥【ユーザー名】¥AppData¥Roaming¥Microsoft¥Windows¥SendTo”

Ctrl＋W に選択系の「現在位置の単語選択」，Ctrl＋F4 にファイル操作系の「閉じる」が、
現在のサクラエディタの初期状態では割り当てられている。
そのため他の様々なソフトにおいて Ctrl＋W でタブを閉じる操作をしている人には、
以下の設定変更をオススメしたい。

[設定(O)]
→[共通設定(C)]
→[キー割り当て]タブ
→種別[選択系]の[現在位置の単語選択]から Ctrl＋W の割り当てを解除する。
→種別[ファイル操作系]の[閉じる]に Ctrl＋W を割り当てる。

ユーザー定義の辞書ファイルを読むのに正常系って何だ？という話も……
読み込めたら正常？ 落ちたら異常？ たしかに異常。
php.khp は落ちたから異常ファイル。

じゃあコテ付けてくれる？NGに入れるから

乙保守

落ちなくても相変わらずヌル終端されてないからゴミが表示される。
それが文字化けに見えてる。

態度悪いやつのアカウントどれよ？

6ch作ったら説得力増すんじゃね？ｗ
そんなにNGにしたかったんだーって思うよｗ
はげワロｗｗｗ

5chの総力上げて潰すぞ

長々書いておいて free と delete を取り違えていたというとんちき具合。

> ではなくて、「delete[] は delete よりコストのかかる上位互換なので〜」だったと思う。

訂正版はこう>「wchar_t には delete[] から呼び出されるようなデストラクタがないので〜」

糾弾する人をNGにしようとしてるのがその証拠だ
コテを付けさせようとする傲慢な態度もそっくりだな
サクラに関わるな

✖ncpyLength+1 < cchAvailableSize
○cchAvailableSize < ncpyLength+1 (<= MAX_REGEX_KEYWORDLEN)

結論は変わらず。

開発者が他メンバーを馬鹿呼ばわりしてるチームなんて先が知れてる。
そもそもあえて難しい日本語を使って悦に入ってるのかなんか知らんが、相手のレベルに合わせた日本語を使うなんて社会人の基本の基本だろう。

>ここの std::min の結果は常に同じ。

嘘嘘。同じじゃないや。
キーワードが不完全になっても切り詰めて取り込むという判断だ。

✖cchAvaialable
○cchAvailable

出勤しなくなったからただの引きこもりニートだな。

githubでやれ

>>31
よほど都合が悪いようだなｗぷっｗｗｗ

osdnでやれ

> GitHub投稿が難しい方はこちらをご利用ください、匿名投稿可

osdnって本当に匿名なんだっけか
管理者とかもした事ないから知らんけど

ホントにアホだな
こんな場末で連投するより自身でforkして修正したビルドを出す方が説得力増すだろ
ついでにpqするかどうかは好きにすればいいがな