Skip to content

C: Migrate hb_buffer to use hb_allocator_T infrastructure#1326

Merged
marcoroth merged 3 commits intomainfrom
hb-buffer-allocator
Mar 7, 2026
Merged

C: Migrate hb_buffer to use hb_allocator_T infrastructure#1326
marcoroth merged 3 commits intomainfrom
hb-buffer-allocator

Conversation

@marcoroth
Copy link
Owner

@marcoroth marcoroth commented Mar 7, 2026 •
edited
Loading

With the new hb_allocator_realloc implemented in #1322 we can now port hb_buffer to use the hb_allocator_T infrastructure.

Follow up on #1323

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 7, 2026 •
edited
Loading 

npx https://pkg.pr.new/@herb-tools/formatter@1326

npx https://pkg.pr.new/@herb-tools/language-server@1326

npx https://pkg.pr.new/@herb-tools/linter@1326

commit: 7f055a0

@github-actions
Copy link

github-actions bot commented Mar 7, 2026 •
edited
Loading

🌿 Interactive Playground and Documentation Preview

A preview deployment has been built for this pull request. Try out the changes live in the interactive playground:

🌱 Grown from commit 7f055a0

✅ Preview deployment has been cleaned up.

@marcoroth marcoroth merged commit 676351d into main Mar 7, 2026
32 checks passed
@marcoroth marcoroth deleted the hb-buffer-allocator branch March 7, 2026 17:59
@marcoroth marcoroth mentioned this pull request Mar 7, 2026
Merged
marcoroth pushed a commit that referenced this pull request Mar 7, 2026
@timkaechele
C: Migrate hb_narray to use new allocator system (#1327)
95a11e3 
This MR migrates the `hb_narray` implementation to use the
`hb_allocator` interface for allocating memory.

Related #1323 and #1326
@marcoroth marcoroth added this to the v1.0.0 milestone Mar 8, 2026
@marcoroth marcoroth mentioned this pull request Mar 9, 2026
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

c c-extension cpp node optimization parser performance refactor rust wasm

Projects

None yet

Milestone

v1.0.0

Development

Successfully merging this pull request may close these issues.

1 participant

@marcoroth