Skip to content

Move warmUp to earliest stage#1330

Merged
cyanzhong merged 1 commit intomainfrom
develop
Apr 5, 2026
Merged

Move warmUp to earliest stage#1330
cyanzhong merged 1 commit intomainfrom
develop

Conversation

@cyanzhong
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Moves the editor reuse pool warm-up earlier in the app startup sequence so the preloaded editor/WebView is initialized before the application delegate lifecycle begins.

Changes:

  • Call EditorReusePool.shared.warmUp() from Application.main() right after Bundle.swizzleInfoDictionaryOnce.
  • Remove the applicationWillFinishLaunching(_:) implementation that previously performed the warm-up.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
MarkEditMac/Sources/Main/Application/Application.swift Triggers editor reuse pool warm-up earlier during startup initialization.
MarkEditMac/Sources/Main/Application/AppDelegate.swift Removes warm-up from applicationWillFinishLaunching(_:) to avoid duplicate/late initialization.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cyanzhong cyanzhong merged commit 4d18591 into main Apr 5, 2026
5 checks passed
cyanzhong added a commit that referenced this pull request Apr 5, 2026
cyanzhong added a commit that referenced this pull request Apr 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants