-
Notifications
You must be signed in to change notification settings - Fork 163
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
編集ウインドウへの参照をリファクタリングする #1744
Merged
sanomari
merged 11 commits into
sakura-editor:master
from
sanomari:feature/fix_reference_of_CEditWnd
Oct 19, 2021
Merged
編集ウインドウへの参照をリファクタリングする #1744
sanomari
merged 11 commits into
sakura-editor:master
from
sanomari:feature/fix_reference_of_CEditWnd
Oct 19, 2021
Commits on Oct 17, 2021
-
Configuration menu - View commit details
-
Copy full SHA for 3ca63d7 - Browse repository at this point
Copy the full SHA 3ca63d7View commit details -
冗長なメンバ変数 CEditVIew::m_pcEditWnd を削除する
CEditWndのインスタンスにはグローバル変数のようにアクセスできる。このため、コンストラクタでCEditWndのインスタンスを要求してクラス内外でのアクセスに流用する構造は無駄と考えられる。
Configuration menu - View commit details
-
Copy full SHA for 67f8ac8 - Browse repository at this point
Copy the full SHA 67f8ac8View commit details -
Configuration menu - View commit details
-
Copy full SHA for d83ad79 - Browse repository at this point
Copy the full SHA d83ad79View commit details -
CEditVIew::m_pcEditWnd置換2 パターン置換49箇所
置換パターン \b((m_)?pc?)?(Edit)?View(Dst)?->m_pcEditWnd->
Configuration menu - View commit details
-
Copy full SHA for 2e458ad - Browse repository at this point
Copy the full SHA 2e458adView commit details -
Configuration menu - View commit details
-
Copy full SHA for 37b1cb2 - Browse repository at this point
Copy the full SHA 37b1cb2View commit details -
Configuration menu - View commit details
-
Copy full SHA for e64af20 - Browse repository at this point
Copy the full SHA e64af20View commit details -
Configuration menu - View commit details
-
Copy full SHA for 000c066 - Browse repository at this point
Copy the full SHA 000c066View commit details -
CEditDoc::m_pcEditWnd置換2 パターン置換66箇所
\b((m_)?pc?)?(Edit)?Doc(Ref)?->m_pcEditWnd->
Configuration menu - View commit details
-
Copy full SHA for 2bd655f - Browse repository at this point
Copy the full SHA 2bd655fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 889939f - Browse repository at this point
Copy the full SHA 889939fView commit details -
Configuration menu - View commit details
-
Copy full SHA for f477971 - Browse repository at this point
Copy the full SHA f477971View commit details -
CEditWndのインスタンスはクラスのstatic変数に保持されるので、グローバル変数に持つと二重保持になり無駄です。
Configuration menu - View commit details
-
Copy full SHA for 4b341c4 - Browse repository at this point
Copy the full SHA 4b341c4View commit details
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.