Skip to content

Adjust capacity of location children#2197

Merged
soutaro merged 1 commit intoruby:masterfrom
ksss:fix-capa
Dec 24, 2024
Merged

Adjust capacity of location children#2197
soutaro merged 1 commit intoruby:masterfrom
ksss:fix-capa

Conversation

@ksss
Copy link
Collaborator

@ksss ksss commented Dec 23, 2024

I believe this is an oversight in #1994 modifications.
There is almost no change in performance.

@soutaro soutaro added this to the RBS 3.8 milestone Dec 23, 2024
@soutaro soutaro self-assigned this Dec 23, 2024
Copy link
Member

@soutaro soutaro left a comment

Choose a reason for hiding this comment

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

The prev code is not optimal but works, as rbs_loc_add_required_child and rbs_loc_add_optional_child automatically resize the capacity if needed.

@soutaro soutaro added this pull request to the merge queue Dec 24, 2024
Merged via the queue into ruby:master with commit 4c77a2b Dec 24, 2024
19 checks passed
@ksss ksss deleted the fix-capa branch December 24, 2024 02:34
@soutaro soutaro added the Released PRs already included in the released version label Dec 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Released PRs already included in the released version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants