Skip to content

C: Implement realloc for hb_allocator_T#1322

Merged
marcoroth merged 1 commit intomainfrom
hb-alloctor-realloc
Mar 7, 2026
Merged

C: Implement realloc for hb_allocator_T#1322
marcoroth merged 1 commit intomainfrom
hb-alloctor-realloc

Conversation

@marcoroth
Copy link
Owner

@marcoroth marcoroth commented Mar 7, 2026

This pull request adds realloc to the hb_allocator_T interface and implements all backend-specific realloc functions.

Currently, our hb_array_T and hb_buffer_T implementations use realloc for growing beyond their current capacity. In order to also migrate hb_array and hb_buffer to the allocator infrastructure we need a way to also support realloc in the interface.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 7, 2026

npx https://pkg.pr.new/@herb-tools/formatter@1322
npx https://pkg.pr.new/@herb-tools/language-server@1322
npx https://pkg.pr.new/@herb-tools/linter@1322

commit: 916c718

@github-actions
Copy link

github-actions bot commented Mar 7, 2026

🌿 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 916c718


✅ Preview deployment has been cleaned up.

@marcoroth marcoroth merged commit 1ce90c7 into main Mar 7, 2026
46 checks passed
@marcoroth marcoroth deleted the hb-alloctor-realloc branch March 7, 2026 13:03
marcoroth added a commit that referenced this pull request Mar 7, 2026
With the new `hb_allocator_realloc` implemented in #1322 we can now port
`hb_array` to use the `hb_allocator_T` infrastructure.

Resolves #1291
marcoroth added a commit that referenced this pull request Mar 7, 2026
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
@marcoroth marcoroth added this to the v1.0.0 milestone Mar 8, 2026
@marcoroth marcoroth mentioned this pull request Mar 9, 2026
4 tasks
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.

1 participant