Windows 10 SDKのメッセージクラッカーを使えるようにする #1742
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Windows10 SDKに定義されているプロトタイプを流用できるようにして開発効率を上げる。 サクラエディタはWinUIもATLもMFCも使わないのでメッセージハンドラ関数を完全に独自定義していた。 完全独自だとメッセージ仕様からwParamとlParamの扱い方を個別に判断する必要があって無駄。
変更前: CViewFont::DeleteFont(); 変更後: CViewFont::DeleteFonts( void ); SDKのマクロとの衝突を回避するため。 変更前: CViewFont::CreateFont(const LOGFONT* plf); 変更後: CViewFont::CreateFonts( const LOGFONT* plf ); Deleteと名称を揃えるため。
|
Kudos, SonarCloud Quality Gate passed! |
|
✅ Build sakura 1.0.3955 completed (commit e81ca7e705 by @sanomari) |
berryzplus
approved these changes
Oct 11, 2021
| @@ -66,8 +66,8 @@ class CViewFont{ | |||
| } | |||
|
|
|||
| private: | |||
| void CreateFont(const LOGFONT *plf); | |||
| void DeleteFont(); | |||
| void CreateFonts( const LOGFONT *plf ); | |||
There was a problem hiding this comment.
Suggested change
| void CreateFonts( const LOGFONT *plf ); | |
| void CreateFonts( const LOGFONT &lf ); |
という指摘はしないほうが良さそうです。
PRの問題ではなくてCViewFontクラスの設計的な問題だから。
|
ん？メッセージトラッカー？ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR の目的
メッセージクラッカーを使えるようにして開発効率を上げます。
カテゴリ
PR の背景
Windows APIでプログラムをしたい人のためにメッセージクラッカーというものがあります。
https://www.keicode.com/windows/win02.php
そのままでは使えなかったので使えるようにするための修正とセットで導入提案する次第です。
PR のメリット
既定のメッセージクラッカーを流用できるようになるので、開発効率が上がります。
PR のデメリット (トレードオフとかあれば)
プログラミングの自由度（？）が下がります。
仕様・動作説明
サクラエディタの仕様・機能に影響を与える変更じゃないです。
メッセージクラッカーのヘッダーファイル(windowsx.h)に定義されているマクロ関数と名前が衝突する関数をリネームします。
ただし、名前が衝突した関数 DeleteFont は複数のフォントハンドルを解放する関数でした。
PR の影響範囲
メッセージクラッカーのヘッダーファイル(windowsx.h)に定義されているマクロ関数と同名の関数を定義できなくなります。
テスト内容
テスト1
手順
関連 issue, PR
参考資料