UNICODE一本化対応で見落としていたA版専用処理を削りたい

[berryzplus]

PR の目的

UNICODE一本化対応で見落としていたA版専用処理を削って、コードベースをシンプルにします。

カテゴリ

• リファクタリング(未使用関数の除去、です。)

PR の背景

• CShareData.cpp のリソース文字列置換関数を削除。
  • 言語切替時にメモリに展開したリソース文字列を置換するためのもの。
  • リソースに含まれる文字列のW⇒A変換処理を削ったので現在未使用。
  • CNativeA::Replace_j はこの処理のためだけに存在していたので一緒に削る。
• CLayoutMgr の未使用の関数宣言を削除。
  • 内部データのUNICODE化はだいぶ前に完了しているので現在未使用。

PR のメリット

PR のデメリット (トレードオフとかあれば)

• とくにありません。

まぁ、強いて言うと「まだ他にもある（かも」ってことですが、新たに見つかったらそれはその時対応すればいいと思っています。

PR の影響範囲

• アプリ（＝サクラエディタ）の機能に影響はありません。
• 未使用関数の除去なので、実用上の影響は無視できる程度と思われます。
  • 原理的にはソースコードが減る分だけコンパイル時間が短縮されるはずですが、人間が感知できるほどの差はなさそうに思います。（検証するメリットを感じないので検証してません。
  • コンパイル済みobjectのリンク工程において、参照されないシンボルはリンク対象から外されます。関数削除によって「参照されない」→「ない」の変化が生じますが、この影響も非常に軽微なものと考えられます。（検証するメリットを感じないので検証してません。

関連チケット

#1034 Unicode版のみのサポートなので、ANSI/UNICODE両対応を廃止したい
#1038 Unicode一本化対応の残課題を拾い集めてissueを立てる refactoring
#1139 CDataProfileの未使用コード(ANSI版)を削除する

参考資料

[AppVeyorBot]

✅ Build sakura 1.0.2691 completed (commit 08988abe9b by @berryzplus)

[m-tmatma]

azure のビルドが走ってなかったので手動で起動

[beru]

問題無いと思います。

[m-tmatma]

githubアプリだと21チェック通っているのに　web だと　4 チェックしか通ってないと表示される

[berryzplus]

レビューありがとうございます。
マージしちゃいます。