btool 配下のソースコードを UTF-8 (BOM付き) に変換

[kobake]

sakura-editor 配下のソースコードのほとんどがエンコーディングを UTF-8 に変換完了しました。
一部だけエンコーディングが食い違うのにも違和感があるため、残りの btool 配下のソースコードも変換しておきます。

cd btool
nkf --overwrite --oc=UTF-8-BOM *.cpp
nkf --overwrite --oc=UTF-8-BOM *.h

確認方法

通常の Visual Studio ではこれらのファイルはビルドされません。
Makefile からのビルド時に btool はビルドされますが、Makefile では入力ファイルのエンコーディングを指定する箇所があるため、そのままではビルドは通りません。

一時的に用意した以下の Makefile により、btool 配下のファイルがビルドされることは確認できます。
https://github.com/kobake/sakura/blob/makefile-utf8/sakura_core/Makefile

※ただし、サクラエディタ本体内の auto にしたイテレータのところでビルド失敗するので、全ビルド成功には至りませんでした。これは自分の環境の gcc が古いせいかもです。

[berryzplus]

LGTMです。

一応、変更箇所だと表示された行をGitHubページ上で眺めて、差異がなさそうであることを確認しました。

このフォルダにあるものは、リソースコンパイルのためのものです。

brcの仕様は知りませんが、サクラエディタ本体がbcc32でビルドできなくなった以上保守する意味はないと考えられます。

windresにはshiftjisのダメ文字を正しく処理できない不具合があるので、rccvtで前処理してやる必要があります。rcの文字コードがutf8になった今、不要になったツールと言えます。

フォルダごと削除でもいいと思いますけどね。
とりあえず、このPRはこのまま通してよいと考えます。

[k-takata]

windresにはshiftjisのダメ文字を正しく処理できない不具合があるので、rccvtで前処理してやる必要があります。rcの文字コードがutf8になった今、不要になったツールと言えます。

試したことはないのですが、今どきのバージョンであれば、-c オプションでコードページを指定すれば、shiftjisでも通りませんかね？
逆に、今回utf-8に変更したことで、-c 65001が必要ではないでしょうか。→ #306

[m-tmatma]

この PR 何待ち?

[berryzplus]

修正待ち。
Windres のパラメータにUTF8指定を付ける対応が残ってます。

[berryzplus]

ちょっと正確じゃなかったかも。

確かベタ書きのコード指定が含まれる部分があって、単純変換じゃなくて修正が必要、というかutf8化することによりいらなくなるかも。

そんな話があったような気がします。

[berryzplus]

詳しくは #306 見てください。
このフォルダ2ファイルありますが、片方は使ってなくて、片方は要らなくなりそう、って状況です。

あ、2セットの、間違いです。

[kobake]

#306 対応を行いました。
もしかしてこれで btool の存在自体が不要になりますかね？
そうだとしたら別PRで btool 消す対応しちゃおうと思いますがどうでしょうか。

[berryzplus]

不要ファイルとして削除する方向に異論ないようなのでapprove取り下げておきます。

[berryzplus]

#306 対応を行いました。
もしかしてこれで btool の存在自体が不要になりますかね？

その認識です。
現状は MinGW でのビルドが通らなくなってますけどねｗ

そうだとしたら別PRで btool 消す対応しちゃおうと思いますがどうでしょうか。

今消すのはなんか違うような・・・。
#309 のMinGW対応が終わるまでは残しておいていい気がします。

[kobake]

この PR をどうしたいのか分からないです。そもそも将来的にもマージしないという判断であればちゃんとクローズしてください。

[berryzplus]

#351 マージ後にフォルダごと削除する予定なので閉じます。

[kobake]

はい。脳内スタックがひとつ消えて良かったです。