debug 配下のファイルを UTF-8 (BOM付) に変換 #265
Conversation
There was a problem hiding this comment.
LGTMです。
現状のmasterとこのコミットでアセンブラリストを出力し、CDlgAbout.asm以外に差がないことを確認しました。
|
↑ データ定義部分も一致してました？ |
|
あと手順もお願いします |
|
@m-tmatma さん
生成に使った変更vcxprojとasmです。 |
|
#290 でアセンブラ出力するPR を投げました。 |
|
このPRを、確認できるのは2日ほど後になります。 |
sakura_core/debug/Debug1.cpp
Outdated
| ::OutputDebugStringA( szText ); | ||
| if( -1 == ret ){ | ||
| ::OutputDebugStringA( "(��̂Ă܂���...)\n" ); | ||
| ::OutputDebugStringA( "(切り捨てました...)\n" ); |
There was a problem hiding this comment.
それとも、実行時のコードを未指定の場合は現在のコードページでコンパイルされるのでしたっけ？
There was a problem hiding this comment.
アセンブラを比較しましたが、問題ないようです。
変更前後で CDlgAbout.asm 以外の部分は一致してました。
|
サイズが大きいので Win32 Release だけ共有しておきます。 新しいバージョン sakura-buildLocal-Win32-Release-Asm.zip 古いバージョン |
cd sakura_core/debug nkf --overwrite --oc=UTF-8-BOM *.cpp nkf --overwrite --oc=UTF-8-BOM *.h
|
#290 を適用するため rebase しました |
sakura_core/debug/Debug2.cpp
Outdated
|
|
||
| #ifdef _DEBUG | ||
| //!�f�o�b�O���b�Z�[�W�o�� | ||
| //!デバッグメッセージ出力 | ||
| void debug_output(const char* str, ...) | ||
| { | ||
| static char buf[_MAX_PATH+150]; |
There was a problem hiding this comment.
この PR とは関係ないが、このコードは static なバッファを使っているので再入可能ではない。
|
CDlgAbout.asm 以外のファイルが以下のビルドで一致しているのを確認しました。 https://ci.appveyor.com/project/sakuraeditor/sakura/build/1.0.456 |
debug 配下のファイルを UTF-8 (BOM付) に変換
該当フォルダ内の文字コードをすべて UTF-8 (BOM付) に変換しました。
確認方法
WinMerge で変更前と変更後を比較すると、文字コード以外の変更が無いことが確認できます。
関連 Issues
ソースコードのUnicode化 #112
特記事項
これまでの経緯から、マルチバイトを含む文字列リテラルについてもファイルのエンコーディングを変更したところで悪影響が起こらないように感じています。
今回の変更ではマルチバイトを含む文字列リテラルを含むファイルも含めてファイルエンコーディングの変更を行っています。