Skip to content

perf: improve generate_namespace_definition#1986

Merged
IWANABETHATGUY merged 2 commits intorolldown:mainfrom
shulaoda:perf/improve-generate-namespace-definition
Aug 15, 2024
Merged

perf: improve generate_namespace_definition#1986
IWANABETHATGUY merged 2 commits intorolldown:mainfrom
shulaoda:perf/improve-generate-namespace-definition

Conversation

@shulaoda
Copy link
Copy Markdown
Member

Description

Related to #1861

// TODO do not call the `generate_caller` function twice.
let final_code =
format!("this{}", parts.iter().map(|&part| generate_caller(part)).collect::<String>());

@shulaoda
Copy link
Copy Markdown
Member Author

@7086cmd cc

@netlify
Copy link
Copy Markdown

netlify bot commented Aug 15, 2024

Deploy Preview for rolldown-rs canceled.

Name Link
🔨 Latest commit 33ed7da
🔍 Latest deploy log https://app.netlify.com/sites/rolldown-rs/deploys/66bdc86d72a5c6000884c5f6

@shulaoda shulaoda marked this pull request as draft August 15, 2024 05:39
@shulaoda shulaoda marked this pull request as ready for review August 15, 2024 05:45
Copy link
Copy Markdown
Contributor

@7086cmd 7086cmd left a comment

Choose a reason for hiding this comment

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

Nice.

@IWANABETHATGUY IWANABETHATGUY added this pull request to the merge queue Aug 15, 2024
Merged via the queue into rolldown:main with commit d90802f Aug 15, 2024
@shulaoda shulaoda deleted the perf/improve-generate-namespace-definition branch August 15, 2024 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants