search

sakura-editor / sakura

SAKURA Editor (Japanese text editor for MS Windows)
https://sakura-editor.github.io/
Other
1.25k stars 163 forks source link

アセンブラ出力を有効にしませんか? #286

Closed m-tmatma closed 6 years ago

m-tmatma commented 6 years ago

アセンブラ出力を有効にしませんか?

以下で @berryzplus さんが文字コード変換で問題がないかの確認でアセンブラ出力を活用されました。 https://github.com/sakura-editor/sakura/pull/265#pullrequestreview-138741324

実際に生成されるコードを比較して確認すると心強いと思います。 なのでデフォルトで有効にしませんか?

昔、アセンブラ出力を行うとバイナリが生成されない仕様だったように思いますが、 15.7.5 で試すとアセンブラ出力もバイナリも生成されていました。

以下で設定できます。

assember

m-tmatma commented 6 years ago

asm ファイルも 別ファイルとして artifacts に含めようと思います。

zip 形式にすると win32 debug で 18MB ぐらい 7z 形式にすると win32 debug で 6 MB ぐらいなので 7z 形式にしようと思います。

7z 形式の欠点は (ローカルで試した限り) zip 形式では 8 秒で済んだのに対して 7z 形式では 圧縮に 47 秒ほどかかった。

kobake commented 6 years ago

良いと思います。

形式については自分としては容量よりも圧縮時間が短いことを優先したいです。 AppVeyor 上でどの程度かかるか分かりませんが、ローカルで試してもらった数字の差を見ても、あまり無視できる時間ではないな、と思いました。成果物4つできるわけですから、4倍の時間差になるわけですよね。

berryzplus commented 6 years ago

心配してるのは元ソースコードの文字コードが混在する場合にどうなるかの確認が取れてなかったことです。余分に時間かかることが分かっているなら、utf8対応の完了後にunicodeオプションは外してもいいと思っています。