Skip to content

perf: rewrite config parser to double the startup speed - #2508

Merged
sxyazi merged 1 commit into
mainfrom
pr-96465633
Mar 21, 2025
Merged

sxyazi merged 1 commit into
mainfrom
pr-96465633

Conversation

@sxyazi

@sxyazi sxyazi commented Mar 21, 2025

Copy link
Copy Markdown
Owner

Startup time matters!

Before - db0300c:

Started in 32.108125ms

Now - this PR:

Started in 16.268708ms

Both are measured in release mode, test script:

Details

--- a/yazi-fm/src/main.rs
+++ b/yazi-fm/src/main.rs
@@ -10,6 +10,7 @@ yazi_macro::mod_flat!(context executor logs panic root router signals term);

 #[tokio::main]
 async fn main() -> anyhow::Result<()> {
+       let now = std::time::Instant::now();
        Panic::install();
        yazi_shared::init();

@@ -33,5 +34,6 @@ async fn main() -> anyhow::Result<()> {
        yazi_core::init();

        yazi_dds::serve();
+       println!("Started in {:?}", now.elapsed());
        app::App::serve().await
 }

@sxyazi
sxyazi merged commit 7389360 into main Mar 21, 2025
@sxyazi
sxyazi deleted the pr-96465633 branch March 21, 2025 17:45
@M00NJ M00NJ mentioned this pull request Apr 9, 2025
1 of 3 tasks
Lemi0002 pushed a commit to Lemi0002/yazi that referenced this pull request Apr 10, 2025
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Apr 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant