レイアウト処理の負荷を下げる為、コメントが色分け表示対象外になる機会を増やす #1052
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
CColor_LineComment::Disp と CColor_BlockComment::Disp メソッドの実装を更新 タイプ別設定 『カラー』プロパティのコメントのリストアイテムのチェックが付いていても始点・終端記号が入力されていない場合は false を返すようにした。 リストアイテムのチェックはデフォルト設定では付いているので、従来はコメントの始点・終端記号が空の場合でも true を返していた。 CColorStrategyPool::OnChangeSetting で Disp メソッドが false を返すと 色分け表示対象外になり CColorStrategyPool::CheckColorMODE において色開始処理が呼ばれない為にレイアウト処理の負荷が軽くなる。
beru
changed the title
|
✅ Build sakura 1.0.2262 completed (commit 43484fc7be by @beru) |
berryzplus
reviewed
Sep 22, 2019
| @@ -38,7 +38,7 @@ class CLineComment | |||
| void CopyTo( const int n, const wchar_t* buffer, int nCommentPos ); // 行コメントデリミタをコピーする | |||
| bool Match( int nPos, const CStringRef& cStr ) const; // 行コメントに値するか確認する | |||
|
|
|||
| const wchar_t* getLineComment( const int n ){ | |||
| const wchar_t* getLineComment( const int n ) const{ | |||
There was a problem hiding this comment.
変更の趣旨と関係ないように思うんですが、変更には賛成です。
constなデータを参照するのに非constメソッドとしていることに違和感を感じました。
There was a problem hiding this comment.
Disp は const メソッドなのでそこから呼び出すには const る必要があるはずです。
There was a problem hiding this comment.
すんません。
無関係に思う ⇒ 関係大アリです。
了解っす 😢
|
✅ Build sakura 1.0.2264 completed (commit bdc6cec81a by @) |
|
レビューありがとうございました。Mergeします。 |
HoppingTappy
pushed a commit
to HoppingTappy/sakura
that referenced
this pull request
Jun 16, 2020
レイアウト処理の負荷を下げる為、コメントが色分け表示対象外になる機会を増やす
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 の背景
CColor_LineComment::DispとCColor_BlockComment::Dispメソッドの実装を更新して、タイプ別設定 『カラー』プロパティのコメントのリストアイテムのチェックが付いていても始点・終端記号が入力されていない場合は false を返すようにしました。リストアイテムのチェックはデフォルト設定では付いているので、従来はコメントの始点・終端記号が空の場合でも true が返されていました。
CColorStrategyPool::OnChangeSettingでDispメソッドが false を返すと 色分け表示対象外になりCColorStrategyPool::CheckColorMODEにおいて色開始処理が呼ばれない為にレイアウト処理の負荷が軽くなります。PR のメリット
従来に比べてレイアウト処理の負荷が下がる機会が増えます。
PR のデメリット (トレードオフとかあれば)
Disp メソッドの実装がやや複雑になります。コメントを書く事で対策しました。
タイプ別設定 『カラー』プロパティのコメントのリストアイテムのチェックを既に外している場合はそれから比べて処理負荷の低減がされません。またタイプ別設定によってはコメントを有効にして使うと思うのでその場合も処理負荷の低減がされません。
かなり大きい文書を扱っている時でないとレイアウト処理から呼び出される
CColorStrategyPool::CheckColorMODEの処理時間が体感出来るほどに膨らむことは無いかもしれません。PR の影響範囲
CColorStrategyPoolクラス