コピー代入のメソッド名を書き間違っているのを訂正

[berryzplus]

PR の目的

コピー代入のメソッド名を書き間違っているのを訂正します。

カテゴリ

• その他

メソッド名の書き間違いですが、実害はないので「その他」としています。
「不具合修正」にすると変な話になる気がするので。

PR の背景

CNativeW には文字列をコピーするためのメソッドがたくさんあります。

• CNativeW::SetNativeData(const CNativeW& rhs)
  • 指定された CNativeW をコピーするメソッド。
• CNativeW::SetString(const WCHAR* rhs)
  • 指定された C-String をコピーするメソッド。

このPRで提案する修正箇所は、
引数で指定された C-String をコピーする処理に、
指定された CNativeW をコピーするメソッドを使っています。

現状、CNativeW のコンストラクタはimplicitなのでコンパイルエラーにはなりませんが、メソッドの利用方法が誤っています。

他の修正と混ぜると説明がめんどくさそうなので先出しで修正提案しておく次第です。

PR のメリット

• CNativeW::SetNativeData(const CNativeW& rhs)メソッドの誤用箇所を潰せます。
  • 幸いなことに、誤用はここだけっす。

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

• 改修箇所の効果確認を実施することがかなり難しいです。
  • 詳細は影響範囲の項を参照。
  • 机上確認できる「単なる誤り」なので「えいやっ！」でやってもいい気はします。

PR の影響範囲

• アプリ（＝サクラエディタ）の機能に影響はありません。
• 修正箇所は PPA(=Poor Pascal for Application) マクロのコントローラクラスです。
  • PPAマクロを動かすには、ppa.dll の動作環境を整える必要があります。
  • ppa.dll には 64bit 版がないので、動作は 32bit 版のみとなります。
  • 改修箇所のコードを動かすには、マクロからppaスクリプトを実行する必要があります。

関連チケット

参考資料

[AppVeyorBot]

✅ Build sakura 1.0.2689 completed (commit 64d92d4b15 by @berryzplus)

[beru]

問題無いと思います。動作確認はしていません。

[berryzplus]

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

[berryzplus]

ん？なんかマージできん…

/azp run

[berryzplus]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).