optimizations #13

Closed

artemis wants to merge 2 commits from artemis/breezewiki:optimizations into main

artemis commented

2023-03-25 01:50:57 +00:00

First-time contributor

couple string handling optimizations i found with the flamegraph stuff. it helps a lil bit. minecraft/Block goes from 650 to 350ms on my machine.

artemis added 2 commits 2023-03-25 01:50:58 +00:00

2694eb7245 use faster string split

dd1b672c4d precompile regexp patterns

cadence commented

2023-04-01 11:19:08 +00:00

Owner

Really helpful, thanks!! I manually merged this into 3f1946a and f5529ed.

I also added my own commit ba6c5be which optimises the pre-processing regular expression a fair bit, but that step is still very slow, and it would be better for me to fix it in the html-parsing library since I've already vendored that. Will have to do this at some point...

Really helpful, thanks!! I manually merged this into 3f1946a and f5529ed. I also added my own commit ba6c5be which optimises the pre-processing regular expression a fair bit, but that step is still very slow, and it would be better for me to fix it in the html-parsing library since I've already vendored that. Will have to do this at some point...

cadence closed this pull request

2023-04-01 11:19:08 +00:00

Pull request closed

Please reopen this pull request to perform a merge.

No reviewers

No Label

No Milestone

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cadence/breezewiki#13

No description provided.