Skip to content

Rename cap to capacity - #851

Merged
alejandro-vaz merged 2 commits into
servo:v2from
Divyansh089:rename-cap-to-capacity
Nov 3, 2026
Merged

alejandro-vaz merged 2 commits into
servo:v2from
Divyansh089:rename-cap-to-capacity

Conversation

@Divyansh089

Copy link
Copy Markdown
Contributor

renames the variables cap, old_cap and double_cap to capacity, old_capacity and double_capacity, as part of #639

closes #850

@Divyansh089

Copy link
Copy Markdown
Contributor Author

left unchanged:

  • RawSmallVec::INLINE_CAP, which is public under the internals feature
  • the bench/fuzz constants INLINE_CAP, MAX_CAP, CAP and the fuzz function test_with_inline_cap

if wanted they can rename too

@alejandro-vaz alejandro-vaz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

thanks @Divyansh089

@alejandro-vaz

Copy link
Copy Markdown
Collaborator

no need to rename those, the one on rawsmallvec is going to be removed with its simplification, and the bencher and fuzzer need a proper refactor on their own

@Divyansh089

Divyansh089 commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor Author

any other issue on which i can work on

@alejandro-vaz

Copy link
Copy Markdown
Collaborator

look at your emails

@alejandro-vaz
alejandro-vaz added this pull request to the merge queue Nov 3, 2026
Merged via the queue into servo:v2 with commit fd31ed6 Nov 3, 2026
8 checks passed
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.

rename cap to capacity

2 participants