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

**名無しさん＠お腹いっぱい。** · 2020/04/23(木) 09:56:52.92

前スレ
サクラエディタふぁんくらぶ part18
https://egg.5ch.net/test/read.cgi/software/1495286392/

■公式・参考
サクラエディタ
https://sakura-editor.github.io/
https://github.com/sakura-editor/sakura/releases

SakuraEditorWiki
http://sakura.qp.land.to/

正規表現ライブラリ bregonig.dll Ver.4.20
http://k-takata.o.oo7.jp/mysoft/bregonig.html
https://bitbucket.org/k_takata/bregonig/downloads/

オープンソースの老舗テキストエディター「サクラエディタ」最新版v2.4.0.0が、2020(令和2)年4月19日に公開。
約2年ぶりのアップデート
https://forest.watch.impress.co.jp/docs/news/1248206.html

**名無しさん＠お腹いっぱい。** · 2020/04/23(木) 10:31:07.57

ちなみに俺は以下の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"

**名無しさん＠お腹いっぱい。** · 2020/04/23(木) 10:37:41.68

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

**名無しさん＠お腹いっぱい。** · 2020/04/23(木) 10:49:34.54

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

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

**名無しさん＠お腹いっぱい。** · 2020/04/23(木) 10:58:07.75

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

**名無しさん＠お腹いっぱい。** · 2020/04/23(木) 11:02:02.36

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

**名無しさん＠お腹いっぱい。** · 2020/04/23(木) 11:47:08.18

>>1
乙

**名無しさん＠お腹いっぱい。** · 2020/04/23(木) 12:04:17.98

乙保守

**名無しさん＠お腹いっぱい。** · 2020/04/23(木) 12:06:43.36

**名無しさん＠お腹いっぱい。** · 2020/04/23(木) 12:08:00.50

**名無しさん＠お腹いっぱい。** · 2020/04/23(木) 12:22:56.23

乙です

**名無しさん＠お腹いっぱい。** · 2020/04/23(木) 20:54:34.64

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

**sage** · 2020/04/23(木) 23:27:31.71

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

**名無しさん＠お腹いっぱい。** · 2020/04/24(金) 01:07:22.15

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

**名無しさん＠お腹いっぱい。** · 2020/04/24(金) 14:01:58.25

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

**名無しさん＠お腹いっぱい。** · 2020/04/24(金) 14:46:09.53

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

**名無しさん＠お腹いっぱい。** · 2020/04/24(金) 21:10:23.57

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

**名無しさん＠お腹いっぱい。** · 2020/04/24(金) 23:23:18.10

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

**名無しさん＠お腹いっぱい。** · 2020/04/25(土) 13:35:55.64

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

**名無しさん＠お腹いっぱい。** · 2020/05/05(火) 18:20:29.57

> 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 を呼び出す前に結果がわかる。

**名無しさん＠お腹いっぱい。** · 2020/05/05(火) 18:25:55.51

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

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

**名無しさん＠お腹いっぱい。** · 2020/05/05(火) 18:44:27.01

でも 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 であったとしてもエラーを抑止できるかはわからない。
(切り詰めて)ヌル文字を書き込むことができないのだから。

**名無しさん＠お腹いっぱい。** · 2020/05/05(火) 18:45:19.36

✖cchAvaialable
○cchAvailable

**名無しさん＠お腹いっぱい。** · 2020/05/05(火) 18:49:54.67

✖_TRUNC
○_TRUNCATE

**名無しさん＠お腹いっぱい。** · 2020/05/05(火) 18:51:42.80

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

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

**名無しさん＠お腹いっぱい。** · 2020/05/05(火) 19:34:23.39

✖cchAvailable
○cchAvailableSize

**名無しさん＠お腹いっぱい。** · 2020/05/05(火) 20:18:59.73

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

結論は変わらず。

**名無しさん＠お腹いっぱい。** · 2020/05/05(火) 20:27:03.82

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

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

**名無しさん＠お腹いっぱい。** · 2020/05/05(火) 21:10:18.39

githubでやれ

**名無しさん＠お腹いっぱい。** · 2020/05/05(火) 22:21:11.47

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

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 に向かって書いてるみたいだ。気い悪くせんといてな。