Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

builtin: reduce allocations in s.index_kmp/1 and s.replace/2 #21561

Merged
merged 4 commits into from
May 25, 2024

Conversation

spytheman
Copy link
Member

  • builtin: reduce allocations for s.index_kmp/1 and s.replace_once/2
  • builtin: reduce allocations for s.replace/2
  • builtin: restore s.replace_once/2 from master

@spytheman spytheman changed the title builtin reduce allocations 7 builtin: reduce allocations in s.index_kmp/1 and s.replace/2 May 24, 2024
@spytheman spytheman force-pushed the builtin_reduce_allocations_7 branch from 69c04f0 to 92e4dfc Compare May 24, 2024 17:16
@spytheman spytheman merged commit c9e6a12 into vlang:master May 25, 2024
69 checks passed
@spytheman spytheman deleted the builtin_reduce_allocations_7 branch May 25, 2024 00:25
spytheman added a commit to felipensp/v that referenced this pull request May 27, 2024
* master:
  cgen: fix array fixed initialization on struct from call (vlang#21568)
  testing: implement a separate `-show-asserts` option, for cleaner test output (`-stats` still works, and still shows both the compilation stats and the asserts) (vlang#21578)
  toml: fix `@[toml: ]`, support `@[skip]` (vlang#21571)
  os: fix debugger_present() for non Windows OSes (vlang#21573)
  builtin: simplify splint_nth methods (vlang#21563)
  net.http: change default http.Server listening address to :9009, to avoid conflicts with tools, that start their own http servers on 8080 like bytehound (vlang#21570)
  os: make minior improvement to C function semantics and related code (vlang#21565)
  io: cleanup prefix_and_suffix/1 util function (vlang#21562)
  os: remove mut declarions for unchanged vars in `os_nix.c.v` (vlang#21564)
  builtin: reduce allocations in s.index_kmp/1 and s.replace/2 (vlang#21561)
  ci: shorten path used for unix domain socket tests (to fit in Windows path limits)
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.

None yet

1 participant